diff --git a/changelogs/fragments/3404-redfish_utils-skip-manager-network-check.yml b/changelogs/fragments/3404-redfish_utils-skip-manager-network-check.yml new file mode 100644 index 0000000000..9b17761ff2 --- /dev/null +++ b/changelogs/fragments/3404-redfish_utils-skip-manager-network-check.yml @@ -0,0 +1,2 @@ +bugfixes: + - redfish_utils module utils - if a manager network property is not specified in the service, attempt to change the requested settings (https://github.com/ansible-collections/community.general/issues/3404/). diff --git a/plugins/module_utils/redfish_utils.py b/plugins/module_utils/redfish_utils.py index 55686b2f50..32a16932cf 100644 --- a/plugins/module_utils/redfish_utils.py +++ b/plugins/module_utils/redfish_utils.py @@ -2762,7 +2762,9 @@ class RedfishUtils(object): if isinstance(set_value, dict): for subprop in payload[property].keys(): if subprop not in target_ethernet_current_setting[property]: - return {'ret': False, 'msg': "Sub-property %s in nic_config is invalid" % subprop} + # Not configured already; need to apply the request + need_change = True + break sub_set_value = payload[property][subprop] sub_cur_value = target_ethernet_current_setting[property][subprop] if sub_set_value != sub_cur_value: @@ -2776,7 +2778,9 @@ class RedfishUtils(object): for i in range(len(set_value)): for subprop in payload[property][i].keys(): if subprop not in target_ethernet_current_setting[property][i]: - return {'ret': False, 'msg': "Sub-property %s in nic_config is invalid" % subprop} + # Not configured already; need to apply the request + need_change = True + break sub_set_value = payload[property][i][subprop] sub_cur_value = target_ethernet_current_setting[property][i][subprop] if sub_set_value != sub_cur_value: