ipaconfig: Fix fail_json calls.

Ansible's fail_json() method required that the message paramater was passed with a keyword parameter, rather than a positional one. Although this seems to work with ansible-core 2.13+, it might not work with previous versions of Ansible. This patch fixes the behaviour for all supported Ansible versions.
freeipa · Sep 21, 2022 · eaf1dcd · eaf1dcd
1 parent 433d109
commit eaf1dcd
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/plugins/modules/ipaconfig.py b/plugins/modules/ipaconfig.py
@@ -482,13 +482,14 @@ def main():
             required_sid = any([netbios_name, add_sids])
             if required_sid and not enable_sid:
                 ansible_module.fail_json(
-                    "'enable-sid: yes' required for 'netbios_name' "
-                    "and 'add-sids'."
+                    msg="'enable-sid: yes' required for 'netbios_name' "
+                        "and 'add-sids'."
                 )
             if enable_sid:
                 if not has_enable_sid:
-                    ansible_module.fail_json(
-                        "This version of IPA does not support 'enable-sid'.")
+                    ansible_module.fail_json(msg=(
+                        "This version of IPA does not support 'enable-sid'."
+                    ))
                 if (
                     netbios_name
                     and netbios_name == get_netbios_name(ansible_module)