diff --git a/changelogs/fragments/6883-redfish-utils-changing-variable-names-in-get-volume-inventory.yml b/changelogs/fragments/6883-redfish-utils-changing-variable-names-in-get-volume-inventory.yml new file mode 100644 index 0000000000..91932c4dd6 --- /dev/null +++ b/changelogs/fragments/6883-redfish-utils-changing-variable-names-in-get-volume-inventory.yml @@ -0,0 +1,2 @@ +bugfixes: + - refish_utils module utils - changing variable names to avoid issues occuring when fetching Volumes data (https://github.com/ansible-collections/community.general/pull/6883). diff --git a/plugins/module_utils/redfish_utils.py b/plugins/module_utils/redfish_utils.py index f807ac76a3..bd0959bed8 100644 --- a/plugins/module_utils/redfish_utils.py +++ b/plugins/module_utils/redfish_utils.py @@ -916,19 +916,19 @@ class RedfishUtils(object): response = self.get_request(self.root_uri + data['Controllers'][u'@odata.id']) if response['ret'] is False: return response - data = response['data'] + c_data = response['data'] - if data.get('Members') and data['Members']: - response = self.get_request(self.root_uri + data['Members'][0][u'@odata.id']) + if c_data.get('Members') and c_data['Members']: + response = self.get_request(self.root_uri + c_data['Members'][0][u'@odata.id']) if response['ret'] is False: return response - data = response['data'] + member_data = response['data'] - if data: - if 'Name' in data: - controller_name = data['Name'] + if member_data: + if 'Name' in member_data: + controller_name = member_data['Name'] else: - controller_id = data.get('Id', '1') + controller_id = member_data.get('Id', '1') controller_name = 'Controller %s' % controller_id volume_results = [] volume_list = []