diff --git a/changelogs/fragments/65302-dnf-msg-return.yml b/changelogs/fragments/65302-dnf-msg-return.yml new file mode 100644 index 00000000000000..9202a3dfe270d9 --- /dev/null +++ b/changelogs/fragments/65302-dnf-msg-return.yml @@ -0,0 +1,3 @@ +bugfixes: + - dnf module - Ensure the modules exit_json['msg'] response is + always string, not sometimes a tuple. diff --git a/lib/ansible/modules/packaging/os/dnf.py b/lib/ansible/modules/packaging/os/dnf.py index cf796bf688664f..c548452eb6c46a 100644 --- a/lib/ansible/modules/packaging/os/dnf.py +++ b/lib/ansible/modules/packaging/os/dnf.py @@ -1165,7 +1165,7 @@ def ensure(self): else: response['changed'] = True if failure_response['failures']: - failure_response['msg'] = 'Failed to install some of the specified packages', + failure_response['msg'] = 'Failed to install some of the specified packages' self.module.fail_json(**failure_response) if self.module.check_mode: response['msg'] = "Check mode: No changes made, but would have if not in check mode" @@ -1195,7 +1195,7 @@ def ensure(self): response['results'].append("Removed: {0}".format(package)) if failure_response['failures']: - failure_response['msg'] = 'Failed to install some of the specified packages', + failure_response['msg'] = 'Failed to install some of the specified packages' self.module.exit_json(**response) self.module.exit_json(**response) except dnf.exceptions.DepsolveError as e: