From 1a597c2b036ca5912dbd108363c36b0c298956d3 Mon Sep 17 00:00:00 2001
From: "Jesse Pretorius (odyssey4me)" <jesse@odyssey4.me>
Date: Thu, 30 Jan 2020 16:17:23 +0000
Subject: [PATCH] Bump minimum openstacksdk version when using
 os_network/dns_domain

With the addition of [a], the minimum openstacksdk version needs to
be bumped to include [b], or the os_network module will return the
error:

TypeError: create_network() got an unexpected keyword argument 'dns_domain'

To handle this, we fail the module if the dns_domain argument is used
and the minimum openstacksdk version for that argument is not met.

[a] https://github.com/ansible/ansible/commit/6c74e29618a6872bc0da66e4992cfcd9cebf6acc
[b] https://github.com/openstack/openstacksdk/commit/a3e846e2b9301f4ae725e97b21b4313bfcc81d10

Fixes: #64495
Fixes: #64841
Change-Id: I8c41e505d5c1949ddc535c78933da06405c47299
---
 .../64495-os_network-openstacksdk-min.yml     |  3 +++
 plugins/modules/os_network.py                 | 24 +++++++++++--------
 2 files changed, 17 insertions(+), 10 deletions(-)
 create mode 100644 changelogs/fragments/64495-os_network-openstacksdk-min.yml

diff --git a/changelogs/fragments/64495-os_network-openstacksdk-min.yml b/changelogs/fragments/64495-os_network-openstacksdk-min.yml
new file mode 100644
index 00000000..7fd35acb
--- /dev/null
+++ b/changelogs/fragments/64495-os_network-openstacksdk-min.yml
@@ -0,0 +1,3 @@
+bugfixes:
+  - bump the minimum openstacksdk version when os_network
+    uses the dns_domain argument
diff --git a/plugins/modules/os_network.py b/plugins/modules/os_network.py
index e31357f6..163ecd11 100644
--- a/plugins/modules/os_network.py
+++ b/plugins/modules/os_network.py
@@ -77,7 +77,7 @@ options:
      type: int
    dns_domain:
      description:
-       -  The DNS domain value to set.
+       -  The DNS domain value to set. Requires openstacksdk>=0.29.
           Network will use Openstack defaults if this option is
           not provided.
 requirements:
@@ -193,12 +193,18 @@ def main():
     provider_physical_network = module.params['provider_physical_network']
     provider_network_type = module.params['provider_network_type']
     provider_segmentation_id = module.params['provider_segmentation_id']
-    project = module.params.get('project')
-    port_security_enabled = module.params.get('port_security_enabled')
-    mtu = module.params.get('mtu')
-    dns_domain = module.params.get('dns_domain')
+    project = module.params['project']
 
-    sdk, cloud = openstack_cloud_from_module(module)
+    net_create_kwargs = {
+        'port_security_enabled': module.params['port_security_enabled'],
+        'mtu_size': module.params['mtu']
+    }
+
+    if module.params['dns_domain'] is not None:
+        sdk, cloud = openstack_cloud_from_module(module, min_version='0.29.0')
+        net_create_kwargs['dns_domain'] = module.params['dns_domain']
+    else:
+        sdk, cloud = openstack_cloud_from_module(module)
     try:
         if project is not None:
             proj = cloud.get_project(project)
@@ -224,13 +230,11 @@ def main():
                 if project_id is not None:
                     net = cloud.create_network(name, shared, admin_state_up,
                                                external, provider, project_id,
-                                               port_security_enabled=port_security_enabled,
-                                               mtu_size=mtu, dns_domain=dns_domain)
+                                               **net_create_kwargs)
                 else:
                     net = cloud.create_network(name, shared, admin_state_up,
                                                external, provider,
-                                               port_security_enabled=port_security_enabled,
-                                               mtu_size=mtu, dns_domain=dns_domain)
+                                               **net_create_kwargs)
                 changed = True
             else:
                 changed = False