8 Commits

Author	SHA1	Message	Date
Thomas Woerner	d7c02d1347	Improve jinja2 spacing: Remove space between join and () ... This change removes the space between join and (): "join ()" to "join()"	2023-01-17 11:51:38 +01:00
Thomas Woerner	473ed03e26	host*: Use FQCN for ansible.builtin ... Use Fully Qualified Collection Name (FQCN) for ansible builtins. This is ansible.builtin.set_fact instead of set_fact for example and aplies for all actions that are part of ansible.builtin. All the replaced ansible.builtins: assert, command, copy, debug, fail, fetch, file, import_playbook, import_tasks, include_role, include_tasks, include_vars, package, set_fact, shell, slurp, stat, systemd	2022-12-20 13:50:23 +01:00
Thomas Woerner	b04f9f58f7	test_host_ipaddresses.yml: Use result.failed also for failed_when ... For failed_when result.failed should be used to make sure that the task fails if there was an error.	2021-05-26 16:10:41 +02:00
Grzegorz Grasza	7a23531047	Use ansible_facts variable ... Without this change the "Import variables specific to distribution" tasks fail with "Could not find file on the Ansible Controller..." on environments with inject facts disabled. This changes the tests to run with ansible with inject_facts_as_vars = false and fixes other roles and playbooks.	2021-03-19 13:55:44 +01:00
Sergio Oliveira Campos	7a6036f7cf	Replaced groups.ipaserver[0] by ansible_fqdn. ... Since we are using docker for running the tests we can no longer rely on groups.ipaserver[0] as the managed host hostname.	2020-07-27 18:00:24 -03:00
Thomas Woerner	d3c6b976ba	tests: Unite admin passwords ... The tests have been using MyPassword123 and also SomeADMINpassword within the tasks of the tests. SomeADMINpassword should be used everywhere.	2020-02-26 12:42:31 +01:00
Thomas Woerner	22d8784da2	ipahost: Do not fail on missing DNS or zone when no IP address given ... If no IP address is given and either DNS is not configured or if the zone is not found then ipahost may not fail in dnsrecord_find. The error happened for example by ensuring the absence of a host that is not part of the domain or for a host that has been added with force and is using a domain that is not served by the DNS server in the domain. It also happened if there was no DNS server in the domain at all. A new test case has been added to test_host_ipaddresses.yml The fix requires ipalib_errors provided by ansible_freeipa_module. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1804838	2020-02-20 13:16:44 +01:00
Thomas Woerner	167c76311d	ipahost: Add support for several IP addresses and also to change them ... ipahost was so far ignoring IP addresses when the host already existed. This happened because host_mod is not providing functionality to do this. Now ipaddress is a list and it is possible to ensure a host with several IP addresses (these can be IPv4 and IPv6). Also it is possible to ensure presence and absence of IP addresses for an exising host using action member. There are no IP address conclict checks as this would lead into issues with updating an existing host that already is using a duplicate IP address for example for round-robin (RR). Also this might lead into issues with ensuring a new host with several IP addresses in this case. Also to ensure a list of hosts with changing the IP address of one host to another in the list would result in issues here. New example playbooks have been added: playbooks/host/host-present-with-several-ip-addresses.yml playbooks/host/host-member-ipaddresses-absent.yml playbooks/host/host-member-ipaddresses-present.yml A new test has been added for verification: tests/host/test_host_ipaddresses.yml Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1783976 https://bugzilla.redhat.com/show_bug.cgi?id=1783979	2020-02-13 13:59:20 +01:00