test_host.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.
This commit is contained in:
Thomas Woerner
2021-05-20 13:16:20 +02:00
parent d95029bbc0
commit 1d61128c9c

View File

@@ -44,7 +44,7 @@
update_dns: yes update_dns: yes
reverse: no reverse: no
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Host "{{ host1_fqdn }}" present again - name: Host "{{ host1_fqdn }}" present again
ipahost: ipahost:
@@ -54,7 +54,7 @@
update_dns: yes update_dns: yes
reverse: no reverse: no
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed
- name: Host "{{ host2_fqdn }}" present - name: Host "{{ host2_fqdn }}" present
ipahost: ipahost:
@@ -64,7 +64,7 @@
update_dns: yes update_dns: yes
reverse: no reverse: no
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Host "{{ host2_fqdn }}" present again - name: Host "{{ host2_fqdn }}" present again
ipahost: ipahost:
@@ -74,7 +74,7 @@
update_dns: yes update_dns: yes
reverse: no reverse: no
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed
- name: Host "{{ host3_fqdn }}" present - name: Host "{{ host3_fqdn }}" present
ipahost: ipahost:
@@ -84,7 +84,7 @@
update_dns: yes update_dns: yes
reverse: no reverse: no
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Host "{{ host3_fqdn }}" present again - name: Host "{{ host3_fqdn }}" present again
ipahost: ipahost:
@@ -94,7 +94,7 @@
update_dns: yes update_dns: yes
reverse: no reverse: no
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed
- name: Host "{{ host4_fqdn }}" present - name: Host "{{ host4_fqdn }}" present
ipahost: ipahost:
@@ -104,7 +104,7 @@
update_dns: yes update_dns: yes
reverse: no reverse: no
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Host "{{ host4_fqdn }}" present again - name: Host "{{ host4_fqdn }}" present again
ipahost: ipahost:
@@ -114,7 +114,7 @@
update_dns: yes update_dns: yes
reverse: no reverse: no
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed
- name: Host "{{ host5_fqdn }}" present - name: Host "{{ host5_fqdn }}" present
ipahost: ipahost:
@@ -124,7 +124,7 @@
update_dns: yes update_dns: yes
reverse: no reverse: no
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Host "{{ host5_fqdn }}" present again - name: Host "{{ host5_fqdn }}" present again
ipahost: ipahost:
@@ -134,7 +134,7 @@
update_dns: yes update_dns: yes
reverse: no reverse: no
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed
- name: Host "{{ host6_fqdn }}" present - name: Host "{{ host6_fqdn }}" present
ipahost: ipahost:
@@ -144,7 +144,7 @@
update_dns: yes update_dns: yes
reverse: no reverse: no
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Host "{{ host6_fqdn }}" present again - name: Host "{{ host6_fqdn }}" present again
ipahost: ipahost:
@@ -154,7 +154,7 @@
update_dns: yes update_dns: yes
reverse: no reverse: no
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed
# disabled can only be checked with enabled hosts, all hosts above are # disabled can only be checked with enabled hosts, all hosts above are
# not enabled. # not enabled.
@@ -170,7 +170,7 @@
# - "{{ host6_fqdn }}" # - "{{ host6_fqdn }}"
# state: disabled # state: disabled
# register: result # register: result
# failed_when: not result.changed # failed_when: not result.changed or result.failed
# #
#- name: Hosts host1..host6 disabled again #- name: Hosts host1..host6 disabled again
# ipahost: # ipahost:
@@ -184,7 +184,7 @@
# - "{{ host6_fqdn }}" # - "{{ host6_fqdn }}"
# state: disabled # state: disabled
# register: result # register: result
# failed_when: result.changed # failed_when: result.changed or result.failed
- name: Hosts host1..host6 absent - name: Hosts host1..host6 absent
ipahost: ipahost:
@@ -199,7 +199,7 @@
update_dns: yes update_dns: yes
state: absent state: absent
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Hosts host1..host6 absent again - name: Hosts host1..host6 absent again
ipahost: ipahost:
@@ -214,5 +214,5 @@
update_dns: yes update_dns: yes
state: absent state: absent
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed