- Sep 30, 2022
-
-
Rafael Guterres Jeffman authored
ipaautomountkey: Fix documentation sections and agument spec
-
Rafael Guterres Jeffman authored
ipaautomountlocation: Fix documentation sections and agument spec
-
Rafael Guterres Jeffman authored
ipaautomember: Fix documentation sections and agument spec
-
Thomas Woerner authored
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: str` needs to be set for string parameters - `type: list` needs to be set for list parameters - `elements: str` needs to be given for list of string parameters - `authors` 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 - `default=.*` needs to be dropped if `required=True` is also given The `copyright` date is extended with `-2022`.
-
Thomas Woerner authored
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: str` needs to be set for string parameters - `required` tags need to be fixed according to the `argument_spec` - `aliases` tag need to be consistent to `argument_spec` - `authors` needs to be given with the github user also: `Name (@user)` - `extends_documentation_fragment: ipamodule_base_docs` needs added and `ipaadmin_*` parameters need to be removed - `choices` needs to be replaced with `aliases` to match `argument_spec` argument_spec - `no_log=False` or `no_log=True` needs to be set for all parameters that have `key` in the name or for dicts also in one the key names - `required=None` needs to be replaced by `required=False` The `copyright` date is extended with `-2022`.
-
Thomas Woerner authored
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: str` needs to be set for string parameters - `elements: str` needs to be given for list of string parameters - `suboptions` instead of `options` needs to be used for dict parameters - `authors` needs to be given with the github user also: `Name (@user)` - `required` tag was fixed according to the argument_spec arguemnt_spec - `elements="str"` needs to be added to all list of string parameters - `no_log=False` or `no_log=True` needs to be set for all parameters that have `key` in the name or for dicts also in one the key names. The `copyright` date is extended with `-2022`.
-
- Sep 27, 2022
-
-
Rafael Guterres Jeffman authored
Fix plugins for ansible fake execution test
-
Rafael Guterres Jeffman authored
ipabackup_get_backup_dir: Fix for ansible-test fake execution test
-
Rafael Guterres Jeffman authored
ipasmartcard_client_get_vars: Fix for ansible-test fake execution test
-
Rafael Guterres Jeffman authored
ipasmartcard_server_get_vars: Fix for ansible-test fake execution test
-
Thomas Woerner authored
ERROR: Found 2 pylint issue(s) which need to be resolved: ERROR: tests/sanity/ignore-2.12.txt:3:1: ansible-test: Ignoring 'ansible-bad-import-from' on 'plugins/module_utils/ansible_freeipa_module.py' is unnecessary ERROR: tests/sanity/ignore-2.12.txt:5:1: ansible-test: Ignoring 'ansible-format-automatic-specification' on 'plugins/module_utils/ansible_freeipa_module.py' is unnecessary
-
Thomas Woerner authored
ansible_freeipa_module is providing netaddr and also DNSName, therefore it is not needed to have own imports in the module. These own imports would need an addional try exception clause to be able to pass the ansible-test fake execution test.
-
Thomas Woerner authored
All imports that are only available after installing IPA need to be in a try exception clause to be able to pass the fake execution test. If the imports can not be done, all used and needed attributes are defined with the value None, MODULE_IMPORT_ERROR is set to the import error and fail_json is called.
-
Thomas Woerner authored
All imports that are only available after installing IPA need to be in a try exception clause to be able to pass the fake execution test. The old workaround "if 'ansible.executor' in sys.modules:" is not working with this test anymore. If the imports can not be done, all used and needed attributes are defines with the value None. A check has been added to IPAAnsibleModule.__init__ to make sure that it fails if the imports have not been done successfully.
-
Thomas Woerner authored
All imports that are only available after installing IPA need to be in a try exception clause to be able to pass the fake execution test. If the imports can not be done, all used and needed attributes are defined with the value None, MODULE_IMPORT_ERROR is set to the import error and fail_json is called.
-
Thomas Woerner authored
All imports that are only available after installing IPA need to be in a try exception clause to be able to pass the fake execution test. If the imports can not be done, all used and needed attributes are defined with the value None, MODULE_IMPORT_ERROR is set to the import error and fail_json is called.
-
Thomas Woerner authored
All imports that are only available after installing IPA need to be in a try exception clause to be able to pass the fake execution test. If the imports can not be done, all used and needed attributes are defined with the value None, MODULE_IMPORT_ERROR is set to the import error and fail_json is called.
-
Rafael Guterres Jeffman authored
Re-enable dnsforwardzone tests
-
- Sep 16, 2022
-
-
Rafael Guterres Jeffman authored
ansible_freeipa_module: Remove deprecated FreeIPABaseModule
-
Rafael Guterres Jeffman authored
As the latest versions of FreeIPA contain fixes to set the port on DNS forwarders, this patch re-enables the upstream tests.
-
Rafael Guterres Jeffman authored
Update instuctions on Azure template varible files to reflect the current test configuration behavior.
-
Thomas Woerner authored
ansible-deprecated-no-collection-name is not needed anymore for plugins/module_utils/ansible_freeipa_module.py with the removal of FreeIPABaseModule class.
-
Thomas Woerner authored
The FreeIPABaseModule class has been maked deprecated with ansible-freeipa version 1.5.0. It is not used in the code any more therefore it is time to finally remove it.
-
- Sep 09, 2022
-
-
Rafael Guterres Jeffman authored
utils/ansible-freeipa.spec.in: Sync with Fedora rawhide spec file
-
Thomas Woerner authored
ipaconfig: Fix example playbook titles.
-
Thomas Woerner authored
ipaconfig: Add support for SID related attributes.
-
Rafael Guterres Jeffman authored
Since FreeIPA 4.9.8 the 'config_mod' command has parameters to enable and configure SIDs, and set the Netbios name. This patch adds the following parameters to ipaconfig plugin: enable_sids: New users and groups automatically get a SID assigned add_sids: Add SIDs for existing users and groups netbios_name: NetBIOS name of the IPA domain Both add_sids and netbios_name requires 'enable_sid: yes'. 'enable_sid' and 'netbios_name' are returned when querying IPA configuration. 'add_sids' always generate SIDs for users and groups, so, muiltiple executions of the playbook with 'add_sids: yes' will return 'changed', even if users and groups SIDs are not modified. A new test playbook is available: tests/config/test_config_sid.yml New examples playbooks are available: playbooks/config/change-ipa-domain-netbios-name.yml playbooks/config/generate-users-groups-sids.yml Fixes: #781 Related: https://bugzilla.redhat.com/show_bug.cgi?id=2069174 Related: https://bugzilla.redhat.com/show_bug.cgi?id=2069184
-
Rafael Guterres Jeffman authored
-
- Sep 08, 2022
-
-
Thomas Woerner authored
ipabackup: Fix order of ipabackup_name parameter evaluation.
-
Thomas Woerner authored
upstream CI: Force retrieval of ansible-freeipa master.
-
Rafael Guterres Jeffman authored
This patch forces the addition of a remote repository pointing to the main ansible-freeipa repo, and fetch its contents before confaring the modified files. The remote repository is removed after the modified file list is generated.
-
Rafael Guterres Jeffman authored
This patch forces the addition of a remote repository pointing to the main ansible-freeipa repo, and fetch its contents before confaring the modified files. The remote repository is removed after the modified file list is generated.
-
Rafael Guterres Jeffman authored
When performing a backup with 'state:present', if 'ipabackup_name' is provided, the backup will be performed, but the role with return an error since 'ipabackup_name' should not be set for this state. This patch moves the parameter evaluation to be performed before the actual backup is performed, so that the backup is not performed and an error is reported.
-
Thomas Woerner authored
The spec file was updated and was lacking several updates.
-
- Sep 07, 2022
-
-
Thomas Woerner authored
ipabackup: Add playbook tests for ipabackup.
-
- Sep 06, 2022
-
-
Thomas Woerner authored
upstream CI: Ensure 'master' branch is available for set_test_modules
-
Rafael Guterres Jeffman authored
If the repository is setup in a way that master branch is not available for comparing the current HEAD against it, the comparison will fail and not module/role will be scheduled for testing. This patch forces fetching 'master' from ansible-freeipa repository, allowing the comparison to be performed.
-
Thomas Woerner authored
upstream CI: run PR tests only for affected plugins
-
Rafael Guterres Jeffman authored
A new test playbook for ipabackup role can be found at: tests/role_backup/test_backup.yml The test is not yet complete, as 'state: restored' is not tested.
-
- Sep 05, 2022
-
-
Rafael Guterres Jeffman authored
ipaserver/ipareplica: Add isatty method to AnsibleModuleLog
-