diff --git a/roles/ipaclient/library/ipaclient_setup_firefox.py b/roles/ipaclient/library/ipaclient_setup_firefox.py index 276fe61548f84c91be9f5872150eaced40d60573..2852e19d97c861f278386897628edf8fd03b87ba 100644 --- a/roles/ipaclient/library/ipaclient_setup_firefox.py +++ b/roles/ipaclient/library/ipaclient_setup_firefox.py @@ -5,7 +5,7 @@ # # Based on ipa-client-install code # -# Copyright (C) 2017 Red Hat +# Copyright (C) 2017-2022 Red Hat # see file 'COPYING' for use and warranty information # # This program is free software; you can redistribute it and/or modify @@ -40,14 +40,16 @@ description: options: domain: description: Primary DNS domain of the IPA deployment + type: str required: yes firefox_dir: description: Specify directory where Firefox is installed (for example '/usr/lib/firefox') + type: str required: no author: - - Thomas Woerner + - Thomas Woerner (@t-woerner) ''' EXAMPLES = ''' @@ -63,20 +65,21 @@ RETURN = ''' from ansible.module_utils.basic import AnsibleModule from ansible.module_utils.ansible_ipa_client import ( - setup_logging, sysrestore, paths, options, configure_firefox + setup_logging, check_imports, sysrestore, paths, options, configure_firefox ) def main(): module = AnsibleModule( argument_spec=dict( - domain=dict(required=True), - firefox_dir=dict(required=False), + domain=dict(required=True, type='str'), + firefox_dir=dict(required=False, type='str'), ), - supports_check_mode=True, + supports_check_mode=False, ) module._ansible_debug = True + check_imports(module) setup_logging() domain = module.params.get('domain')