From a32fcb3765f9604c8e1ffec14e61e8789ee6d174 Mon Sep 17 00:00:00 2001
From: Thomas Woerner <twoerner@redhat.com>
Date: Thu, 4 May 2023 17:35:26 +0200
Subject: [PATCH] ansible_freeipa_module.py: Calm down ansible-test on print
and sys.exit
The function exit_raw_json is a replacement for AnsibleModule.exit_json
without flterting out values for no_log parameters.
Ansible added checks for pylint to forbid print and also sys.exit and
fails with ansible-bad-function. As the check is not known outside of
ansible-test, the disable line needed also W0012:
# pylint: disable=W0012,ansible-bad-function
---
plugins/module_utils/ansible_freeipa_module.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/plugins/module_utils/ansible_freeipa_module.py b/plugins/module_utils/ansible_freeipa_module.py
index ff7b06dc..c5f8d7f2 100644
--- a/plugins/module_utils/ansible_freeipa_module.py
+++ b/plugins/module_utils/ansible_freeipa_module.py
@@ -747,8 +747,8 @@ def exit_raw_json(module, **kwargs):
contains sensible data, it will be appear in the logs.
"""
module.do_cleanup_files()
- print(jsonify(kwargs))
- sys.exit(0)
+ print(jsonify(kwargs)) # pylint: disable=W0012,ansible-bad-function
+ sys.exit(0) # pylint: disable=W0012,ansible-bad-function
def __get_domain_validator():
--
GitLab