ansible-collections/community.general
1
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2025-10-23 04:24:00 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

module utils: update code to python3 (#10907)

Browse source 
* module utils: update code to python3

* add changelog frag
This commit is contained in:
Alexei Znamensky 2025-10-12 00:42:11 +13:00 • committed by GitHub
commit cc83188594
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
9 changed files with 83 additions and 77 deletions
Download patch file Download diff file Expand all files Collapse all files

9
changelogs/fragments/10907-2to3-mu.yml Normal file
View file

@ -0,0 +1,9 @@
minor_changes:
- csv module_utils plugin - use Python 3 idioms (https://github.com/ansible-collections/community.general/pull/10907).
- gitlab module_utils plugin - use Python 3 idioms (https://github.com/ansible-collections/community.general/pull/10907).
- homebrew module_utils plugin - use Python 3 idioms (https://github.com/ansible-collections/community.general/pull/10907).
- ilo_redfish_utils module_utils plugin - use Python 3 idioms (https://github.com/ansible-collections/community.general/pull/10907).
- redfish_utils module_utils plugin - use Python 3 idioms (https://github.com/ansible-collections/community.general/pull/10907).
- saslprep module_utils plugin - use Python 3 idioms (https://github.com/ansible-collections/community.general/pull/10907).
- utm_utils module_utils plugin - use Python 3 idioms (https://github.com/ansible-collections/community.general/pull/10907).
- vexata module_utils plugin - use Python 3 idioms (https://github.com/ansible-collections/community.general/pull/10907).

2
plugins/module_utils/csv.py
View file

@ -52,7 +52,7 @@ def initialize_dialect(dialect, **kwargs):
def read_csv(data, dialect, fieldnames=None):
BOM = to_native(u'\ufeff')
BOM = to_native('\ufeff')
data = to_native(data, errors='surrogate_or_strict')
if data.startswith(BOM):
data = data[len(BOM):]

5
plugins/module_utils/gitlab.py
View file

@ -10,10 +10,7 @@ from ansible.module_utils.basic import missing_required_lib
from ansible_collections.community.general.plugins.module_utils.version import LooseVersion
try:
from urlparse import urljoin
except ImportError:
from urllib.parse import urljoin # Python 3+
from urllib.parse import urljoin
import traceback

2
plugins/module_utils/homebrew.py
View file

@ -11,7 +11,7 @@ import re
def _create_regex_group_complement(s):
lines = (line.strip() for line in s.split("\n") if line.strip())
chars = filter(None, (line.split("#")[0].strip() for line in lines))
chars = [_f for _f in (line.split("#")[0].strip() for line in lines) if _f]
group = rf"[^{''.join(chars)}]"
return re.compile(group)

4
plugins/module_utils/ilo_redfish_utils.py
View file

@ -31,9 +31,9 @@ class iLORedfishUtils(RedfishUtils):
if data["Oem"]["Hpe"]["Links"]["MySession"]["@odata.id"]:
current_session = data["Oem"]["Hpe"]["Links"]["MySession"]["@odata.id"]
for sessions in data[u'Members']:
for sessions in data['Members']:
# session_list[] are URIs
session_list.append(sessions[u'@odata.id'])
session_list.append(sessions['@odata.id'])
# for each session, get details
for uri in session_list:
session = {}

128
plugins/module_utils/redfish_utils.py
View file

@ -396,7 +396,7 @@ class RedfishUtils(object):
# Insert the headers (Content-Disposition and Content-Type)
if 'filename' in fields[form]:
name = os.path.basename(fields[form]['filename']).replace('"', '\\"')
write_buffer(body, u'Content-Disposition: form-data; name="%s"; filename="%s"' % (to_text(form), to_text(name)))
write_buffer(body, 'Content-Disposition: form-data; name="%s"; filename="%s"' % (to_text(form), to_text(name)))
else:
write_buffer(body, 'Content-Disposition: form-data; name="%s"' % form)
write_buffer(body, 'Content-Type: %s' % fields[form]['mime_type'])
@ -562,9 +562,9 @@ class RedfishUtils(object):
data = response['data']
self.firmware_uri = self.software_uri = None
if 'FirmwareInventory' in data:
self.firmware_uri = data['FirmwareInventory'][u'@odata.id']
self.firmware_uri = data['FirmwareInventory']['@odata.id']
if 'SoftwareInventory' in data:
self.software_uri = data['SoftwareInventory'][u'@odata.id']
self.software_uri = data['SoftwareInventory']['@odata.id']
return {'ret': True}
def _find_chassis_resource(self):
@ -696,12 +696,12 @@ class RedfishUtils(object):
return response
data = response['data']
for log_svcs_entry in data.get('Members', []):
response = self.get_request(self.root_uri + log_svcs_entry[u'@odata.id'])
response = self.get_request(self.root_uri + log_svcs_entry['@odata.id'])
if response['ret'] is False:
return response
_data = response['data']
if 'Entries' in _data:
log_svcs_uri_list.append(_data['Entries'][u'@odata.id'])
log_svcs_uri_list.append(_data['Entries']['@odata.id'])
# For each entry in LogServices, get log name and all log entries
for log_svcs_uri in log_svcs_uri_list:
@ -744,15 +744,15 @@ class RedfishUtils(object):
return response
data = response['data']
for log_svcs_entry in data[u'Members']:
for log_svcs_entry in data['Members']:
response = self.get_request(self.root_uri + log_svcs_entry["@odata.id"])
if response['ret'] is False:
return response
_data = response['data']
# Check to make sure option is available, otherwise error is ugly
if "Actions" in _data:
if "#LogService.ClearLog" in _data[u"Actions"]:
self.post_request(self.root_uri + _data[u"Actions"]["#LogService.ClearLog"]["target"], {})
if "#LogService.ClearLog" in _data["Actions"]:
self.post_request(self.root_uri + _data["Actions"]["#LogService.ClearLog"]["target"], {})
if response['ret'] is False:
return response
return {'ret': True}
@ -807,14 +807,14 @@ class RedfishUtils(object):
# Loop through Members and their StorageControllers
# and gather properties from each StorageController
if data[u'Members']:
for storage_member in data[u'Members']:
storage_member_uri = storage_member[u'@odata.id']
if data['Members']:
for storage_member in data['Members']:
storage_member_uri = storage_member['@odata.id']
response = self.get_request(self.root_uri + storage_member_uri)
data = response['data']
if key in data:
controllers_uri = data[key][u'@odata.id']
controllers_uri = data[key]['@odata.id']
response = self.get_request(self.root_uri + controllers_uri)
if response['ret'] is False:
@ -822,9 +822,9 @@ class RedfishUtils(object):
result['ret'] = True
data = response['data']
if data[u'Members']:
for controller_member in data[u'Members']:
controller_member_uri = controller_member[u'@odata.id']
if data['Members']:
for controller_member in data['Members']:
controller_member_uri = controller_member['@odata.id']
response = self.get_request(self.root_uri + controller_member_uri)
if response['ret'] is False:
return response
@ -875,16 +875,16 @@ class RedfishUtils(object):
if 'Storage' in data:
# Get a list of all storage controllers and build respective URIs
storage_uri = data[u'Storage'][u'@odata.id']
storage_uri = data['Storage']['@odata.id']
response = self.get_request(self.root_uri + storage_uri)
if response['ret'] is False:
return response
result['ret'] = True
data = response['data']
if data[u'Members']:
for controller in data[u'Members']:
controller_list.append(controller[u'@odata.id'])
if data['Members']:
for controller in data['Members']:
controller_list.append(controller['@odata.id'])
for c in controller_list:
uri = self.root_uri + c
response = self.get_request(uri)
@ -894,7 +894,7 @@ class RedfishUtils(object):
controller_name = 'Controller 1'
storage_id = data['Id']
if 'Controllers' in data:
controllers_uri = data['Controllers'][u'@odata.id']
controllers_uri = data['Controllers']['@odata.id']
response = self.get_request(self.root_uri + controllers_uri)
if response['ret'] is False:
@ -902,8 +902,8 @@ class RedfishUtils(object):
result['ret'] = True
cdata = response['data']
if cdata[u'Members']:
controller_member_uri = cdata[u'Members'][0][u'@odata.id']
if cdata['Members']:
controller_member_uri = cdata['Members'][0]['@odata.id']
response = self.get_request(self.root_uri + controller_member_uri)
if response['ret'] is False:
@ -921,8 +921,8 @@ class RedfishUtils(object):
controller_name = 'Controller %s' % sc_id
drive_results = []
if 'Drives' in data:
for device in data[u'Drives']:
disk_uri = self.root_uri + device[u'@odata.id']
for device in data['Drives']:
disk_uri = self.root_uri + device['@odata.id']
response = self.get_request(disk_uri)
data = response['data']
@ -952,8 +952,8 @@ class RedfishUtils(object):
result['ret'] = True
data = response['data']
for controller in data[u'Members']:
controller_list.append(controller[u'@odata.id'])
for controller in data['Members']:
controller_list.append(controller['@odata.id'])
for c in controller_list:
uri = self.root_uri + c
@ -967,7 +967,7 @@ class RedfishUtils(object):
sc_id = data.get('Id', '1')
controller_name = 'Controller %s' % sc_id
drive_results = []
for device in data[u'Devices']:
for device in data['Devices']:
drive_result = {}
for property in properties:
if property in device:
@ -1005,7 +1005,7 @@ class RedfishUtils(object):
if 'Storage' in data:
# Get a list of all storage controllers and build respective URIs
storage_uri = data[u'Storage'][u'@odata.id']
storage_uri = data['Storage']['@odata.id']
response = self.get_request(self.root_uri + storage_uri)
if response['ret'] is False:
return response
@ -1013,8 +1013,8 @@ class RedfishUtils(object):
data = response['data']
if data.get('Members'):
for controller in data[u'Members']:
controller_list.append(controller[u'@odata.id'])
for controller in data['Members']:
controller_list.append(controller['@odata.id'])
for idx, c in enumerate(controller_list):
uri = self.root_uri + c
response = self.get_request(uri)
@ -1023,13 +1023,13 @@ class RedfishUtils(object):
data = response['data']
controller_name = 'Controller %s' % str(idx)
if 'Controllers' in data:
response = self.get_request(self.root_uri + data['Controllers'][u'@odata.id'])
response = self.get_request(self.root_uri + data['Controllers']['@odata.id'])
if response['ret'] is False:
return response
c_data = response['data']
if c_data.get('Members') and c_data['Members']:
response = self.get_request(self.root_uri + c_data['Members'][0][u'@odata.id'])
response = self.get_request(self.root_uri + c_data['Members'][0]['@odata.id'])
if response['ret'] is False:
return response
member_data = response['data']
@ -1052,13 +1052,13 @@ class RedfishUtils(object):
volume_list = []
if 'Volumes' in data:
# Get a list of all volumes and build respective URIs
volumes_uri = data[u'Volumes'][u'@odata.id']
volumes_uri = data['Volumes']['@odata.id']
response = self.get_request(self.root_uri + volumes_uri)
data = response['data']
if data.get('Members'):
for volume in data[u'Members']:
volume_list.append(volume[u'@odata.id'])
for volume in data['Members']:
volume_list.append(volume['@odata.id'])
for v in volume_list:
uri = self.root_uri + v
response = self.get_request(uri)
@ -1075,9 +1075,9 @@ class RedfishUtils(object):
# Get related Drives Id
drive_id_list = []
if 'Links' in data:
if 'Drives' in data[u'Links']:
for link in data[u'Links'][u'Drives']:
drive_id_link = link[u'@odata.id']
if 'Drives' in data['Links']:
for link in data['Links']['Drives']:
drive_id_link = link['@odata.id']
drive_id = drive_id_link.rstrip('/').split('/')[-1]
drive_id_list.append({'Id': drive_id})
volume_result['Linked_drives'] = drive_id_list
@ -1364,7 +1364,7 @@ class RedfishUtils(object):
data = response['data']
for users in data.get('Members', []):
user_list.append(users[u'@odata.id']) # user_list[] are URIs
user_list.append(users['@odata.id']) # user_list[] are URIs
# for each user, get details
for uri in user_list:
@ -1644,8 +1644,8 @@ class RedfishUtils(object):
result['ret'] = True
data = response['data']
for sessions in data[u'Members']:
session_list.append(sessions[u'@odata.id']) # session_list[] are URIs
for sessions in data['Members']:
session_list.append(sessions['@odata.id']) # session_list[] are URIs
# for each session, get details
for uri in session_list:
@ -1674,8 +1674,8 @@ class RedfishUtils(object):
return {'ret': True, 'changed': False, 'msg': "There are no active sessions"}
# loop to delete every active session
for session in data[u'Members']:
response = self.delete_request(self.root_uri + session[u'@odata.id'])
for session in data['Members']:
response = self.delete_request(self.root_uri + session['@odata.id'])
if response['ret'] is False:
return response
@ -1776,8 +1776,8 @@ class RedfishUtils(object):
else:
uri = None
for member in data[u'Members']:
fw_uri = self.root_uri + member[u'@odata.id']
for member in data['Members']:
fw_uri = self.root_uri + member['@odata.id']
# Get details for each software or firmware member
response = self.get_request(fw_uri)
if response['ret'] is False:
@ -2084,7 +2084,7 @@ class RedfishUtils(object):
return response
result['ret'] = True
data = response['data']
for attribute in data[u'Attributes'].items():
for attribute in data['Attributes'].items():
bios_attributes[attribute[0]] = attribute[1]
result["entries"] = bios_attributes
return result
@ -2340,14 +2340,14 @@ class RedfishUtils(object):
# Check the attributes
for attr_name, attr_value in attributes.items():
# Check if attribute exists
if attr_name not in data[u'Attributes']:
if attr_name not in data['Attributes']:
# Remove and proceed to next attribute if this isn't valid
attrs_bad.update({attr_name: attr_value})
del attrs_to_patch[attr_name]
continue
# If already set to requested value, remove it from PATCH payload
if data[u'Attributes'][attr_name] == attr_value:
if data['Attributes'][attr_name] == attr_value:
del attrs_to_patch[attr_name]
warning = ""
@ -2492,8 +2492,8 @@ class RedfishUtils(object):
data = response['data']
# Checking if fans are present
if u'Fans' in data:
for device in data[u'Fans']:
if 'Fans' in data:
for device in data['Fans']:
fan = {}
for property in properties:
if property in device:
@ -2568,7 +2568,7 @@ class RedfishUtils(object):
result['ret'] = True
data = response['data']
if "Temperatures" in data:
for sensor in data[u'Temperatures']:
for sensor in data['Temperatures']:
sensor_result = {}
for property in properties:
if property in sensor:
@ -2610,8 +2610,8 @@ class RedfishUtils(object):
result['ret'] = True
data = response['data']
for cpu in data[u'Members']:
cpu_list.append(cpu[u'@odata.id'])
for cpu in data['Members']:
cpu_list.append(cpu['@odata.id'])
for c in cpu_list:
cpu = {}
@ -2660,8 +2660,8 @@ class RedfishUtils(object):
result['ret'] = True
data = response['data']
for dimm in data[u'Members']:
memory_list.append(dimm[u'@odata.id'])
for dimm in data['Members']:
memory_list.append(dimm['@odata.id'])
for m in memory_list:
dimm = {}
@ -2730,8 +2730,8 @@ class RedfishUtils(object):
result['ret'] = True
data = response['data']
for nic in data[u'Members']:
nic_list.append(nic[u'@odata.id'])
for nic in data['Members']:
nic_list.append(nic['@odata.id'])
for n in nic_list:
nic = self.get_nic(n)
@ -2786,8 +2786,8 @@ class RedfishUtils(object):
result['ret'] = True
data = response['data']
for virtualmedia in data[u'Members']:
virtualmedia_list.append(virtualmedia[u'@odata.id'])
for virtualmedia in data['Members']:
virtualmedia_list.append(virtualmedia['@odata.id'])
for n in virtualmedia_list:
virtualmedia = {}
@ -2953,8 +2953,8 @@ class RedfishUtils(object):
return response
data = response['data']
virt_media_list = []
for member in data[u'Members']:
virt_media_list.append(member[u'@odata.id'])
for member in data['Members']:
virt_media_list.append(member['@odata.id'])
resources, headers = self._read_virt_media_resources(virt_media_list)
# see if image already inserted; if so, nothing to do
@ -3068,8 +3068,8 @@ class RedfishUtils(object):
return response
data = response['data']
virt_media_list = []
for member in data[u'Members']:
virt_media_list.append(member[u'@odata.id'])
for member in data['Members']:
virt_media_list.append(member['@odata.id'])
resources, headers = self._read_virt_media_resources(virt_media_list)
# find the VirtualMedia resource to eject
@ -3136,7 +3136,7 @@ class RedfishUtils(object):
data = response['data']
if 'Power' in data:
power_uri = data[u'Power'][u'@odata.id']
power_uri = data['Power']['@odata.id']
else:
continue

6
plugins/module_utils/saslprep.py
View file

@ -50,11 +50,11 @@ def mapping_profile(string):
if in_table_c12(c):
# map non-ASCII space characters
# (that can be mapped) to Unicode space
tmp.append(u' ')
tmp.append(' ')
else:
tmp.append(c)
return u"".join(tmp)
return "".join(tmp)
def is_ral_string(string):
@ -160,7 +160,7 @@ def saslprep(string):
# "This profile specifies using Unicode normalization form KC."
string = normalize('NFKC', string)
if not string:
return u''
return ''
# RFC4013: 2.3. Prohibited Output.
# RFC4013: 2.4. Bidirectional Characters.

2
plugins/module_utils/utm_utils.py
View file

@ -183,7 +183,7 @@ class UTM:
result = None
if response is not None:
results = json.loads(response.read())
result = next(iter(filter(lambda d: d['name'] == module.params.get('name'), results)), None)
result = next(iter([d for d in results if d['name'] == module.params.get('name')]), None)
return info, result
def _clean_result(self, result):

2
plugins/module_utils/vexata.py
View file

@ -20,7 +20,7 @@ VXOS_VERSION = None
def get_version(iocs_json):
if not iocs_json:
raise Exception('Invalid IOC json')
active = filter(lambda x: x['mgmtRole'], iocs_json)
active = [x for x in iocs_json if x['mgmtRole']]
if not active:
raise Exception('Unable to detect active IOC')
active = active[0]