Correctly build params dict in recordset

Change If8fda40780050d271c9d869d8959ef569644fd88 unintentionally broke
our integration tests. This patch fixes the code and tests to make
everything pass again.

Change-Id: Ief8d1f9e1eec13a2d435e96a0d70e31a2b4431f2

ci/roles/recordset/tasks/main.yml

@@ -48,7 +48,7 @@
 - name: Assert recordset not changed
   assert:
     that:
-      - recordset is changed
+      - recordset is not changed
 
 - name: Assert recordset fields
   assert:
@@ -84,13 +84,13 @@
     zone: "{{ dns_zone.zone.name }}"
     name: "{{ recordset.recordset.name }}"
     state: absent
-  register: recordset
+  register: deleted_recordset
 
 - name: Verify recordset deletion
   assert:
     that:
-    - recordset is successful
+    - deleted_recordset is successful
-    - recordset is changed
+    - deleted_recordset is changed
 
 - name: Delete unexistent recordset
   openstack.cloud.recordset:
@@ -98,12 +98,12 @@
     zone: "{{ dns_zone.zone.name }}"
     name: "{{ recordset.recordset.name }}"
     state: absent
-  register: recordset
+  register: deleted_recordset
 
 - name: Verify recordset deletion
   assert:
     that:
-    - recordset is not changed
+    - deleted_recordset is not changed
 
 - name: Delete dns zone
   openstack.cloud.dns_zone:

plugins/modules/recordset.py

@@ -184,6 +184,10 @@ class DnsRecordsetModule(OpenStackModule):
 
     def _needs_update(self, params, recordset):
         for k in ('description', 'records', 'ttl', 'type'):
+            if k not in params:
+                continue
+            if k not in recordset:
+                return True
             if params[k] is not None and params[k] != recordset[k]:
                 return True
         return False
@@ -204,10 +208,10 @@ class DnsRecordsetModule(OpenStackModule):
         description = self.params['description']
         ttl = self.params['ttl']
         params = {
-            description: description,
+            'description': description,
-            records: records,
+            'records': records,
-            type: recordset_type.upper(),
+            'type': recordset_type.upper(),
-            ttl: ttl,
+            'ttl': ttl,
         }
         return {k: v for k, v in params.items() if v is not None}