From 0dc49d0706e757d159651e974cdb2e76a793027c Mon Sep 17 00:00:00 2001 From: Thomas Woerner Date: Fri, 28 Oct 2022 19:23:38 +0200 Subject: [PATCH 1/2] ipasmartcard_server_get_vars: Fix doc sections and agument spec ansible-test with ansible-2.14 is adding a lot of new tests to ensure that the documentation section and the agument spec is complete. Needed changes: DOCUMENTATION section - `suboptions` needs to be removed without arguments - `author` needs to be given with the github user also: `Name (@user)` --- .../library/ipasmartcard_server_get_vars.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/roles/ipasmartcard_server/library/ipasmartcard_server_get_vars.py b/roles/ipasmartcard_server/library/ipasmartcard_server_get_vars.py index 85897914..b34bec39 100644 --- a/roles/ipasmartcard_server/library/ipasmartcard_server_get_vars.py +++ b/roles/ipasmartcard_server/library/ipasmartcard_server_get_vars.py @@ -36,9 +36,8 @@ short_description: Get variables from ipaplatform and ipaserver and python interpreter. description: Get variables from ipaplatform and ipaserver and python interpreter. -options: author: - - Thomas Woerner + - Thomas Woerner (@t-woerner) ''' EXAMPLES = ''' From 903e002e85463e7540990e8854c3caee95bbd8e8 Mon Sep 17 00:00:00 2001 From: Thomas Woerner Date: Fri, 28 Oct 2022 19:24:16 +0200 Subject: [PATCH 2/2] ipasmartcard_server_validate_ca_certs: Fix doc sections and agument spec ansible-test with ansible-2.14 is adding a lot of new tests to ensure that the documentation section and the agument spec is complete. Needed changes: DOCUMENTATION section - `type: list` needs to be set for list parameters - `elements: str` needs to be given for list of string parameters - `required` tags need to be fixed according to the `argument_spec` - `author` needs to be given with the github user also: `Name (@user)` argument_spec - `elements="str"` needs to be added to all list of string parameters --- .../library/ipasmartcard_server_validate_ca_certs.py | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/roles/ipasmartcard_server/library/ipasmartcard_server_validate_ca_certs.py b/roles/ipasmartcard_server/library/ipasmartcard_server_validate_ca_certs.py index 7463efef..b776e584 100644 --- a/roles/ipasmartcard_server/library/ipasmartcard_server_validate_ca_certs.py +++ b/roles/ipasmartcard_server/library/ipasmartcard_server_validate_ca_certs.py @@ -41,9 +41,11 @@ options: description: List of files containing CA certificates for the service certificate files - required: yes + type: list + elements: str + required: no author: - - Thomas Woerner + - Thomas Woerner (@t-woerner) ''' EXAMPLES = ''' @@ -63,7 +65,8 @@ except ImportError: def main(): ansible_module = AnsibleModule( argument_spec=dict( - ca_cert_files=dict(required=False, type='list', default=[]), + ca_cert_files=dict(required=False, type='list', elements='str', + default=[]), ), supports_check_mode=False, )