From 09f68fc6595ab078210fa1846f620dd4ad4cba66 Mon Sep 17 00:00:00 2001 From: Christopher Snyder Date: Tue, 9 Apr 2019 03:36:51 -0400 Subject: [PATCH] redhat_subscription: Fix usage of ConfigParser (#54815) --- ...edhat_subscription_use_strings_for_yum_plugin_configs.yaml | 2 ++ lib/ansible/modules/packaging/os/redhat_subscription.py | 4 ++-- 2 files changed, 4 insertions(+), 2 deletions(-) create mode 100644 changelogs/fragments/redhat_subscription_use_strings_for_yum_plugin_configs.yaml diff --git a/changelogs/fragments/redhat_subscription_use_strings_for_yum_plugin_configs.yaml b/changelogs/fragments/redhat_subscription_use_strings_for_yum_plugin_configs.yaml new file mode 100644 index 0000000000..942abe2356 --- /dev/null +++ b/changelogs/fragments/redhat_subscription_use_strings_for_yum_plugin_configs.yaml @@ -0,0 +1,2 @@ +bugfixes: +- redhat_subscription - For compatibility using the redhat_subscription module on hosts set to use a python 3 interpreter, use string values when updating yum plugin configuration files. diff --git a/lib/ansible/modules/packaging/os/redhat_subscription.py b/lib/ansible/modules/packaging/os/redhat_subscription.py index aa41bde915..fa1beb5728 100644 --- a/lib/ansible/modules/packaging/os/redhat_subscription.py +++ b/lib/ansible/modules/packaging/os/redhat_subscription.py @@ -260,9 +260,9 @@ class RegistrationBase(object): cfg.read([tmpfile]) if enabled: - cfg.set('main', 'enabled', 1) + cfg.set('main', 'enabled', '1') else: - cfg.set('main', 'enabled', 0) + cfg.set('main', 'enabled', '0') fd = open(tmpfile, 'w+') cfg.write(fd)