- Feb 15, 2024
-
-
Rafael Guterres Jeffman authored
There was a space between the code block marker and the highlight hint in a playbook example.
-
- Dec 21, 2023
-
-
Thomas Woerner authored
ipagroup: Add support for renaming groups
-
- Dec 20, 2023
-
-
Rafael Guterres Jeffman authored
Fixes for ansible-lint 6.22.1
-
Rafael Guterres Jeffman authored
FreeIPA suports renaming groupobjects with the CLI parameter "rename", and this parameter was missing in ansible-freeipa ipagroup module. This patch adds support for a new state 'renamed' and the 'rename' parameter. Tests were updated to cope with the changes.
-
Thomas Woerner authored
- Replace outdated noqa 503 with noqa no-handler - Drop outdated and not needed noqa 505 for include_vars - Drop outdated noqa deprecated-command-syntax for ansible.builtin.shell using cmd tag These warnings have been reported by utils/lint_check.sh using ansible-lint 6.22.1.
-
Rafael Guterres Jeffman authored
Use module_defaults to improve reading test cases.
-
Thomas Woerner authored
ipauser: Add support for renaming users
-
Thomas Woerner authored
ipadnszone: Add support for per-zone privilege delegation
-
- Dec 19, 2023
-
-
Rafael Guterres Jeffman authored
FreeIPA suports renaming user objects with the CLI parameter "rename", and this parameter was missing in ansible-freeipa ipauser module. This patch adds support for a new state 'renamed' and the 'rename' parameter. Tests were updated to cope with the changes. Related to RHBZ#2234379, RHBZ#2234380 Fixes #1103
-
Rafael Guterres Jeffman authored
IPA DNS Zones management can be delegated by adding a "Manage DNS zone" permission. The CLI commands that manage these permissions are dnszone-add-permission and dnszone-remove-permission. The ansible-freeipa module ipadnszone did not have this capability, and it now support dnszone per-zone management delegation by setting the module parameter 'permission'. If set to 'true' the permission will be assigned to the zone, if set to false the permission will be removed.
-
Rafael Guterres Jeffman authored
ipaclient: Properly name automount_location var and add documentation
-
Thomas Woerner authored
Handle data type or empty string in module_utils
-
- Dec 15, 2023
-
-
Rafael Guterres Jeffman authored
Use the commom parameter type handling method for parameters that accept a value or an empty string.
-
Rafael Guterres Jeffman authored
Use the commom parameter type handling method for parameters that accept a value or an empty string.
-
Rafael Guterres Jeffman authored
Use the commom parameter type handling method for parameters that accept a value or an empty string.
-
Rafael Guterres Jeffman authored
Some parameters, in modules, have a specific data type, but allow the use of an empty string to clear the parameter. By providing a method to retrieve the parameter with the correct data type, or optionally an empty string, allows for consistency of parameter handling between different modules.
-
- Dec 14, 2023
-
-
Thomas Woerner authored
ipauser: Do not try to modify user when not changing password
-
- Dec 08, 2023
-
-
Rafael Guterres Jeffman authored
The parameter 'allow_empty_string' in 'module_params_get' is used to allow an item in a list to be an empty string. The problem is that the naming is misleading, as it is checking a list item rather than a string. This patch rename the parameter to 'allow_empty_list_item' so that it more clearly refers to list itens instead of standalone strings, and do not collide with future parameters that may test for empty strings which are not part of lists.
-
Thomas Woerner authored
ipaclient: Fix OTP error reporting
-
- Dec 07, 2023
-
-
Rafael Guterres Jeffman authored
Revert "[TEMP] Enable only idp, service and user module tests"
-
Thomas Woerner authored
This reverts commit de3c6c0a.
-
Thomas Woerner authored
ipareplica: Support inventory groups.ipaserver
-
Thomas Woerner authored
upstream ci: Increase timeout for PR tests
-
Rafael Guterres Jeffman authored
After the change for a single job to run PR tests, and if there is any change to ansible_module_utils, all the playbook tests are executed, and the result is a failure due to timeout. This PR increases the timeout so that a PR with changes to ansible_module_utils can have the tests executed.
-
Rafael Guterres Jeffman authored
ipaidp: Fix validation and reset of parameters
-
Thomas Woerner authored
Bump minimum ansible-lint version to 6.22
-
- Dec 06, 2023
-
-
Rafael Guterres Jeffman authored
By the first quarter of 2024, all collections must pass ansible-lint tests run with version 6.22.x. This PR ensure that all ansible-freeipa tests depending on ansible-lint use a valid version of it.
-
Rafael Guterres Jeffman authored
Merge pull request #1186 from t-woerner/test_host_random_conditional_statements_no_jinja2_templating test_host_random: No jinja2 templating in conditional statements
-
Rafael Guterres Jeffman authored
When deploying an IPA client with ipaclient, if an error occured while getting an OTP, no error message is logged, as the task that logs the error is not excuted due to the previous taks failure. By adding a 'rescue' section to the code block and moving the error reporting to this new section, we ensure that the proper error messages will be reported.
-
Thomas Woerner authored
With ansible-core 2.14.12 using jinja2 templating in conditional statements with ansible_facts['fqdn'] is marked as unsafe and results in a failure. The issues with using jinja2 templating in conditional statements for asserts have been solved and a new server_fqdn fact has been added for ansible_facts['fqdn'].
-
- Dec 05, 2023
-
-
Thomas Woerner authored
-
Thomas Woerner authored
The uri parameters auth_uri, dev_auth_uri, token_uri, userinfo_uri and keys_uri have not been validated before. Also the base_url was not normalized. The auth_uri, dev_auth_uri, token_uri and userinfo_uri need to be set for new entries, but might be empty or empty string for reset or updates. The ipaidpclientsecret needs to be decoded from binary string in find_idp result to not trigger no change ipd_mod calls. The code for validate_uri and base_url normalization has been copied from the ipaserver idp plugin. ansible_freeipa_module: urlparse from urllib.parse with a fallback to six.moves.urllib.parse is imported and also exported. urlparse is needed for validate_uri in ipaidp module. Resolves: RHEL-17954, RHEL-17955, RHEL-17957 and RHEL-17958
-
- Nov 28, 2023
-
-
Rafael Guterres Jeffman authored
If a playbook to ensure the existence of a user contains 'random:false' and 'update_password: always' is executed twice, the second execution will raise an exception due to "No modifications to perform", as there is actually nothing to modify. The fix for the issue is to remove 'random' if it is not set to true, as setting it to 'false' would have no effect on the user object. Related: https://issues.redhat.com/browse/RHEL-4934
-
Rafael Guterres Jeffman authored
test_pwpolicy: minlength parameter can be reset with empty string now
-
Thomas Woerner authored
The reset of the minlength parameter failed with an internal error so far. This has been fixed in IPA and therefore requires to fix the test in ansible-freeipa also. Related: https://pagure.io/freeipa/issue/9297
-
- Nov 24, 2023
-
-
Thomas Woerner authored
Reproduce upstream CI groups in developer's machine
-
Thomas Woerner authored
upstream CI: Build containers in parallel jobs
-
Thomas Woerner authored
Update ansible-lint and pylint versions
-
Thomas Woerner authored
ipahost: Remove dangling dns records during test setup
-
Rafael Guterres Jeffman authored
idoverride{user,group}: Fix delete_continue with state absent
-