From 14eedb2956a304d582da63e57b7663c61eb7a7a4 Mon Sep 17 00:00:00 2001 From: Bill Dodd Date: Tue, 29 Oct 2019 08:16:49 -0500 Subject: [PATCH] Do not PATCH Account Enabled prop if not supported (#62617) * do not PATCH Account Enabled prop if not supported * add changelog fragment --- ...617-fix-redfish-enable-account-if-enabled-prop-missing.yaml | 3 +++ lib/ansible/module_utils/redfish_utils.py | 2 +- 2 files changed, 4 insertions(+), 1 deletion(-) create mode 100644 changelogs/fragments/62617-fix-redfish-enable-account-if-enabled-prop-missing.yaml diff --git a/changelogs/fragments/62617-fix-redfish-enable-account-if-enabled-prop-missing.yaml b/changelogs/fragments/62617-fix-redfish-enable-account-if-enabled-prop-missing.yaml new file mode 100644 index 00000000000..d98558fde58 --- /dev/null +++ b/changelogs/fragments/62617-fix-redfish-enable-account-if-enabled-prop-missing.yaml @@ -0,0 +1,3 @@ +--- +bugfixes: +- redfish_command - fix EnableAccount if Enabled property is not present in Account resource (https://github.com/ansible/ansible/issues/59822) diff --git a/lib/ansible/module_utils/redfish_utils.py b/lib/ansible/module_utils/redfish_utils.py index b8c64048761..0df6be9f922 100644 --- a/lib/ansible/module_utils/redfish_utils.py +++ b/lib/ansible/module_utils/redfish_utils.py @@ -904,7 +904,7 @@ class RedfishUtils(object): uri = response['uri'] data = response['data'] - if data.get('Enabled'): + if data.get('Enabled', True): # account already enabled, nothing to do return {'ret': True, 'changed': False}