Update catalog service for the new sdk

This patch do the following:

* Update catalog_service to use new openstacksdk
* Add catalog_service role to test catalog_service module

Change-Id: I6778f5e91cb0ead63cede28af0111d7ffbbf3ab1

.zuul.yaml

@@ -65,6 +65,7 @@
       tox_extra_args: &skip_broken_ci_roles >-
         address_scope
         auth
+        catalog_service
         client_config
         dns
         dns_zone_info

ci/roles/catalog_service/tasks/main.yml

@@ -0,0 +1,94 @@
+---
+- name: Delete service test
+  openstack.cloud.catalog_service:
+    cloud: "{{ cloud }}"
+    service_type: test
+    name: test
+    state: absent
+  register: service_delete
+
+- name: Assert changed is set to false
+  assert:
+    that:
+      - not service_delete.changed
+
+- name: Create a service for test
+  openstack.cloud.catalog_service:
+    cloud: "{{ cloud }}"
+    name: "test_service"
+    state: present
+    service_type: test_type
+    description: "Test service"
+  register: service_test
+
+- name: Verify returned values
+  assert:
+    that:
+      - item in service_test.service
+  loop:
+    - description
+    - id
+    - is_enabled
+    - links
+    - name
+    - type
+
+- name: Check if the service test was created successfully
+  openstack.cloud.catalog_service:
+    cloud: "{{ cloud }}"
+    service_type: test
+    name: test
+  register: service_created
+
+- name: Verify returned values
+  assert:
+    that:
+      - item in service_created.service
+  loop:
+    - description
+    - id
+    - is_enabled
+    - links
+    - name
+    - type
+
+- name: Update service test
+  openstack.cloud.catalog_service:
+    cloud: "{{ cloud }}"
+    service_type: test
+    description: "A new description"
+    is_enabled: False
+    name: test
+  register: service_test
+
+- name: Check if description and enabled were updated
+  assert:
+    that:
+      - service_test.service.description == "A new description"
+      - not (service_test.service.is_enabled|bool)
+
+- name: Delete service test
+  openstack.cloud.catalog_service:
+    cloud: "{{ cloud }}"
+    service_type: test
+    name: test
+    state: absent
+  register: service_deleted
+
+- name: Verify if service was deleted
+  assert:
+    that:
+      - service_deleted.changed
+
+- name: Delete service test again
+  openstack.cloud.catalog_service:
+    cloud: "{{ cloud }}"
+    service_type: test
+    name: test
+    state: absent
+  register: service_deleted
+
+- name: Assert changed is set to false
+  assert:
+    that:
+      - not service_deleted.changed

ci/run-collection.yml

@@ -6,6 +6,7 @@
   roles:
     - { role: address_scope, tags: address_scope }
     - { role: auth, tags: auth }
+    - { role: catalog_service, tags: catalog_service }
     - { role: client_config, tags: client_config }
     - { role: dns_zone_info, tags: dns_zone_info }
     - role: object_container

plugins/modules/catalog_service.py

@@ -21,16 +21,18 @@ options:
      description:
         - Description of the service
      type: str
-   enabled:
+   is_enabled:
      description:
         - Is the service enabled
      type: bool
      default: 'yes'
-   service_type:
+     aliases: ['enabled']
+   type:
      description:
         - The type of service
      required: true
      type: str
+     aliases: ['service_type']
    state:
      description:
        - Should the resource be present or absent.
@@ -51,21 +53,21 @@ EXAMPLES = '''
      cloud: mycloud
      state: present
      name: glance
-     service_type: image
+     type: image
      description: OpenStack Image Service
 # Delete a service
 - openstack.cloud.catalog_service:
      cloud: mycloud
      state: absent
      name: glance
-     service_type: image
+     type: image
 '''
 
 RETURN = '''
 service:
     description: Dictionary describing the service.
     returned: On success when I(state) is 'present'
-    type: complex
+    type: dict
     contains:
         id:
             description: Service ID.
@@ -75,7 +77,7 @@ service:
             description: Service name.
             type: str
             sample: "glance"
-        service_type:
+        type:
             description: Service type.
             type: str
             sample: "image"
@@ -83,10 +85,14 @@ service:
             description: Service description.
             type: str
             sample: "OpenStack Image Service"
-        enabled:
+        is_enabled:
             description: Service status.
             type: bool
             sample: True
+        links:
+            description: Link of the service
+            type: str
+            sample: http://10.0.0.1/identity/v3/services/0ae87
 id:
     description: The service ID.
     returned: On success when I(state) is 'present'
@@ -100,9 +106,9 @@ from ansible_collections.openstack.cloud.plugins.module_utils.openstack import O
 class IdentityCatalogServiceModule(OpenStackModule):
     argument_spec = dict(
         description=dict(default=None),
-        enabled=dict(default=True, type='bool'),
+        is_enabled=dict(default=True, aliases=['enabled'], type='bool'),
         name=dict(required=True),
-        service_type=dict(required=True),
+        type=dict(required=True, aliases=['service_type']),
         state=dict(default='present', choices=['absent', 'present']),
     )
 
@@ -111,10 +117,8 @@ class IdentityCatalogServiceModule(OpenStackModule):
     )
 
     def _needs_update(self, service):
-        if service.enabled != self.params['enabled']:
+        for parameter in ('is_enabled', 'description', 'type'):
-            return True
+            if service[parameter] != self.params[parameter]:
-        if service.description is not None and \
-           service.description != self.params['description']:
                 return True
         return False
 
@@ -132,46 +136,52 @@ class IdentityCatalogServiceModule(OpenStackModule):
 
     def run(self):
         description = self.params['description']
-        enabled = self.params['enabled']
+        enabled = self.params['is_enabled']
         name = self.params['name']
         state = self.params['state']
-        service_type = self.params['service_type']
+        type = self.params['type']
 
-        services = self.conn.search_services(
+        filters = {'name': name, 'type': type}
-            name_or_id=name, filters=dict(type=service_type))
 
+        services = list(self.conn.identity.services(**filters))
+
+        service = None
         if len(services) > 1:
             self.fail_json(
                 msg='Service name %s and type %s are not unique'
-                % (name, service_type))
+                % (name, type))
         elif len(services) == 1:
             service = services[0]
-        else:
-            service = None
 
         if self.ansible.check_mode:
             self.exit_json(changed=self._system_state_change(service))
 
+        args = {'name': name, 'enabled': enabled, 'type': type}
+        if description:
+            args['description'] = description
+
         if state == 'present':
             if service is None:
-                service = self.conn.create_service(
+                service = self.conn.identity.create_service(**args)
-                    name=name, description=description, type=service_type, enabled=True)
                 changed = True
             else:
                 if self._needs_update(service):
-                    service = self.conn.update_service(
+                    # The self.conn.update_service calls get_service that
-                        service.id, name=name, type=service_type, enabled=enabled,
+                    # checks if the service is duplicated or not. We don't need
-                        description=description)
+                    # to do it here because it was already checked above
+                    service = self.conn.identity.update_service(service,
+                                                                **args)
                     changed = True
                 else:
                     changed = False
-            self.exit_json(changed=changed, service=service, id=service.id)
+            service = service.to_dict(computed=False)
+            self.exit_json(changed=changed, service=service, id=service['id'])
 
         elif state == 'absent':
             if service is None:
                 changed = False
             else:
-                self.conn.delete_service(service.id)
+                self.conn.identity.delete_service(service)
                 changed = True
             self.exit_json(changed=changed)