Release 7.6.1 (#501)

* Release 7.6.1

* Release 7.6.1

* updated changelog changes

* updated changelog

.github/workflows/ansible-test.yml

@@ -157,7 +157,7 @@ jobs:
           run: pip install https://github.com/ansible/ansible/archive/${{ matrix.ansible-version }}.tar.gz --disable-pip-version-check
 
         - name: Install ansible lint
-          run: pip install ansible-lint pytest --disable-pip-version-check
+          run: pip install ansible-lint==6.16.2 pytest --disable-pip-version-check
 
         - name: Install ansible posix
           run: ansible-galaxy collection install ansible.posix

CHANGELOG.rst

@@ -5,6 +5,27 @@ Dell OpenManage Ansible Modules Release Notes
 .. contents:: Topics
 
 
+v7.6.1
+======
+
+Release Summary
+---------------
+
+Removed the dependency of community general collections.
+
+Minor Changes
+-------------
+
+- Updated the idrac_gather_facts role to use jinja template filters.
+
+Known Issues
+------------
+
+- idrac_redfish_storage_contoller - Issue(256164) - If incorrect value is provided for one of the attributes in the provided attribute list for controller configuration, then this module does not exit with error.
+- ome_device_network_services - Issue(212681) - The module does not provide a proper error message if unsupported values are provided for the following parameters- port_number, community_name, max_sessions, max_auth_retries, and idle_timeout.
+- ome_device_power_settings - Issue(212679) - The module displays the following message if the value provided for the parameter ``power_cap`` is not within the supported range of 0 to 32767, ``Unable to complete the request because PowerCap does not exist or is not applicable for the resource URI.``
+- ome_smart_fabric_uplink - Issue(186024) - Despite the module supported by OpenManage Enterprise Modular, it does not allow the creation of multiple uplinks of the same name. If an uplink is created using the same name as an existing uplink, the existing uplink is modified.
+
 v7.6.0
 ======
 

changelogs/changelog.yaml

@@ -1362,3 +1362,26 @@ releases:
           name: idrac_storage_controller
           namespace: null
     release_date: "2023-05-30"
+  7.6.1:
+    changes:
+      known_issues:
+        - idrac_redfish_storage_contoller - Issue(256164) - If incorrect value is provided
+          for one of the attributes in the provided attribute list for controller configuration,
+          then this module does not exit with error.
+        - ome_device_network_services - Issue(212681) - The module does not provide
+          a proper error message if unsupported values are provided for the following
+          parameters- port_number, community_name, max_sessions, max_auth_retries, and
+          idle_timeout.
+        - ome_device_power_settings - Issue(212679) - The module displays the following
+          message if the value provided for the parameter ``power_cap`` is not within
+          the supported range of 0 to 32767, ``Unable to complete the request because
+          PowerCap does not exist or is not applicable for the resource URI.``
+        - ome_smart_fabric_uplink - Issue(186024) - Despite the module supported by
+          OpenManage Enterprise Modular, it does not allow the creation of multiple
+          uplinks of the same name. If an uplink is created using the same name as an
+          existing uplink, the existing uplink is modified.
+      minor_changes:
+        - Updated the idrac_gather_facts role to use jinja template filters.
+      release_summary:
+        Removed the dependency of community general collections.
+    release_date: "2023-06-01"

galaxy.yml

@@ -1,6 +1,6 @@
 namespace: dellemc
 name: openmanage
-version: 7.6.0
+version: 7.6.1
 readme: README.md
 authors:
   - Jagadeesh N V <Jagadeesh.N.V@Dell.com>
@@ -38,7 +38,6 @@ tags:
 
 dependencies: {
   "ansible.posix": ">=1.5.4",
-  "community.general": ">=7.0.0",
   "ansible.utils": ">=2.10.2",
   "ansible.windows": ">=1.14.0"
 }

requirements.yml

@@ -1,6 +1,5 @@
 collections:
   - dellemc.openmanage
   - ansible.posix
-  - community.general
   - ansible.utils
   - ansible.windows

roles/idrac_gather_facts/tasks/get_metrics_info.yml

@@ -16,9 +16,7 @@
 
 - name: Get Power Supply Metrics ids
   ansible.builtin.set_fact:
-    power_metrics_ids: "{{ power_result.json.Members | json_query(jquery) }}"
-  vars:
-    jquery: "[*].Metrics"
+    power_metrics_ids: "{{ power_result.json.Members | selectattr('Metrics', 'defined') | map(attribute='Metrics') | flatten }}"
   no_log: true
 
 - name: Get Power Supply Metrics information.
@@ -69,9 +67,7 @@
 
 - name: Get Memory Metrics ids
   ansible.builtin.set_fact:
-    memory_metrics_ids: "{{ memory_result.json.Members | json_query(jquery) }}"
-  vars:
-    jquery: "[*].Metrics"
+    memory_metrics_ids: "{{ memory_result.json.Members | selectattr('Metrics', 'defined') | map(attribute='Metrics') | flatten }}"
   no_log: true
 
 - name: Get Memory Metrics information.
@@ -91,18 +87,14 @@
   delegate_to: "{{ lookup('ansible.builtin.env', 'RUNON', default='localhost') }}"
   no_log: true
 
-- name: Set j query
-  ansible.builtin.set_fact:
-    jquery: "[*].json"
-
 - name: Set Power Supply/Thermal/Memory Metrics facts
   ansible.builtin.set_fact:
     power_metrics:
-      "{{ power_metrics_result.results | json_query(jquery) | flatten |
+      "{{ power_metrics_result.results | selectattr('json', 'defined') | map(attribute='json') | flatten |
       ansible.utils.remove_keys(target=['@odata.context', '@odata.type', '@odata.id', 'DataSourceUri']) }}"
     thermal_metrics:
       "{{ thermal_result.json | ansible.utils.remove_keys(target=['@odata.context', '@odata.type', '@odata.id', 'DataSourceUri',
       'TemperatureReadingsCelsius@odata.count']) }}"
     memory_metrics:
-      "{{ memory_metrics_result.results | json_query(jquery) | flatten |
+      "{{ memory_metrics_result.results | selectattr('json', 'defined') | map(attribute='json') | flatten |
       ansible.utils.remove_keys(target=['@odata.context', '@odata.type', '@odata.id', 'DataSourceUri']) }}"

roles/idrac_gather_facts/tasks/get_physical_info.yml

@@ -17,9 +17,7 @@
 
 - name: Get all storage controller ids.
   ansible.builtin.set_fact:
-    storage_ids_list: "{{ disk_result.json | default('') | json_query(jquery) }}"
-  vars:
-    jquery: 'Members[*]."@odata.id"'
+    storage_ids_list: "{{ disk_result.json.Members | map('dict2items') | flatten | map(attribute='value') }}"
   no_log: true
 
 - name: Get PhysicalDisk information.
@@ -39,12 +37,9 @@
   delegate_to: "{{ lookup('ansible.builtin.env', 'RUNON', default='localhost') }}"
   no_log: true
 
-- name: Set query
-  ansible.builtin.set_fact:
-    jq: "[*].json.Drives"
-
 - name: Set Physical Disk facts
   ansible.builtin.set_fact:
     physical_disk:
-      "{{ result.results | json_query(jq) | flatten | ansible.utils.remove_keys(target=['@odata.context', '@odata.id', '@odata.type',
+      "{{ result.results | selectattr('json', 'defined') | map(attribute='json') | selectattr('Drives', 'defined') |
+      map(attribute='Drives') | flatten | ansible.utils.remove_keys(target=['@odata.context', '@odata.id', '@odata.type',
       'Actions', 'Assembly', 'Links', 'DellDriveSMARTAttributes', 'DellNVMeSMARTAttributes', 'Operations@odata.count']) }}"

roles/idrac_gather_facts/tasks/get_resource_id.yml

@@ -17,17 +17,13 @@
 
 - name: Get first System Id from the system response
   ansible.builtin.set_fact:
-    api_system: "{{ system_api_result.json | default('') | json_query(jquery) }}"
-  vars:
-    jquery: 'Members[0]."@odata.id"'
+    api_system: "{{ system_api_result.json.Members[0]['@odata.id'] | default('') }}"
   when: computer_system_id == ''
   no_log: true
 
 - name: Get all system Ids
   ansible.builtin.set_fact:
-    system_ids_list: "{{ system_api_result.json | default('') | json_query(jquery) }}"
-  vars:
-    jquery: 'Members[*]."@odata.id"'
+    system_ids_list: "{{ system_api_result.json.Members | map('dict2items') | flatten | map(attribute='value') }}"
   when: computer_system_id != ''
   no_log: true
 
@@ -61,17 +57,13 @@
 
 - name: Get first manager resource id from manager response.
   ansible.builtin.set_fact:
-    api_manager: "{{ manager_api_result.json | default('') | json_query(jquery) }}"
-  vars:
-    jquery: 'Members[0]."@odata.id"'
+    api_manager: "{{ manager_api_result.json.Members[0]['@odata.id'] | default('') }}"
   when: manager_id == ''
   no_log: true
 
 - name: Get all manager resource ids.
   ansible.builtin.set_fact:
-    manager_ids_list: "{{ manager_api_result.json | default('') | json_query(jquery) }}"
-  vars:
-    jquery: 'Members[*]."@odata.id"'
+    manager_ids_list: "{{ manager_api_result.json.Members | map('dict2items') | flatten | map(attribute='value') }}"
   when: manager_id != ''
   no_log: true
 
@@ -105,7 +97,5 @@
 
 - name: Get first chassis resource id from manager response.
   ansible.builtin.set_fact:
-    api_chassis: "{{ chassis_api_result.json | default('') | json_query(jquery) }}"
-  vars:
-    jquery: 'Members[0]."@odata.id"'
+    api_chassis: "{{ chassis_api_result.json.Members[0]['@odata.id'] | default('') }}"
   no_log: true

roles/idrac_gather_facts/tasks/get_system_info.yml

@@ -34,7 +34,5 @@
 - name: Set system facts
   ansible.builtin.set_fact:
     system:
-      "{{ system_result.json | json_query(jquery) | combine(os_result.json.Attributes) |
+      "{{ system_result.json.Oem.Dell.DellSystem | combine(os_result.json.Attributes) |
       ansible.utils.remove_keys(target=['@odata.context', '@odata.id', '@odata.type']) }}"
-  vars:
-    jquery: "Oem.Dell.DellSystem"

roles/idrac_gather_facts/tasks/get_virtual_disk_info.yml

@@ -17,9 +17,7 @@
 
 - name: Get all storage controller ids.
   ansible.builtin.set_fact:
-    storage_ids_list: "{{ virtual_disk_result.json | default('') | json_query(jquery) }}"
-  vars:
-    jquery: 'Members[*]."@odata.id"'
+    storage_ids_list: "{{ virtual_disk_result.json.Members | map('dict2items') | flatten | map(attribute='value') | default('') }}"
   no_log: true
 
 - name: Get Virtual Disk information.
@@ -39,12 +37,10 @@
   delegate_to: "{{ lookup('ansible.builtin.env', 'RUNON', default='localhost') }}"
   no_log: true
 
-- name: Set query
-  ansible.builtin.set_fact:
-    jq: "[*].json.Drives"
-
 - name: Set Virtual Disk facts
   ansible.builtin.set_fact:
     virtual_disk:
-      "{{ result.results | json_query(jq) | flatten | ansible.utils.remove_keys(target=['@odata.context', '@odata.type', '@odata.id', 'Actions',
-      'EncryptionTypes@odata.count', 'Identifiers@odata.count', 'Links', 'Operations@odata.count', 'DellVirtualDisk', 'DellVirtualDisk@Redfish.Deprecated']) }}"
+      "{{ result.results | selectattr('json', 'defined') | map(attribute='json') | selectattr('Members', 'defined') |
+      map(attribute='Members') | flatten | ansible.utils.remove_keys(target=['@odata.context', '@odata.type',
+      '@odata.id', 'Actions', 'EncryptionTypes@odata.count', 'Identifiers@odata.count', 'Links',
+      'Operations@odata.count', 'DellVirtualDisk', 'DellVirtualDisk@Redfish.Deprecated']) }}"