freeipa / ansible-freeipa

Ansible roles and modules for FreeIPA
GNU General Public License v3.0
482 stars 230 forks source link

ipahostgroup: Fix idempotence issues due to capitalization #1202

Closed rjeffman closed 5 months ago

rjeffman commented 6 months ago

ipahostgroup parameters 'host', 'hostgroup', 'membermanager_user' and 'membermanager_group' must be compared in a case insensitive manner and stored as lower case strings.

This patch fixes the comparison and storage of this parameters, and change the handling of members to use the same structure as in newer modules.

Two new tests files were added:

tests/hostgroup/test_hostgroup_case_insensitive.yml
tests/hostgroup/test_hostgroup_membermanager_case_insensitive.yml

Fixes #1196

rjeffman commented 5 months ago

/azp run CheckPR

azure-pipelines[bot] commented 5 months ago
Azure Pipelines successfully started running 1 pipeline(s).