test_host_certificate.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:19 +02:00
parent aa57aa56f4
commit 399a376451

View File

@@ -31,7 +31,7 @@
name: "{{ 'test.' + ipaserver_domain }}"
force: yes
register: result
failed_when: not result.changed
failed_when: not result.changed or result.failed
- name: Host test cert members present
ipahost:
@@ -43,7 +43,7 @@
- "{{ lookup('file', 'cert3.b64', rstrip=False) }}"
action: member
register: result
failed_when: not result.changed
failed_when: not result.changed or result.failed
- name: Host test cert members present again
ipahost:
@@ -55,7 +55,7 @@
- "{{ lookup('file', 'cert3.b64', rstrip=False) }}"
action: member
register: result
failed_when: result.changed
failed_when: result.changed or result.failed
- name: Host test cert members absent
ipahost:
@@ -68,7 +68,7 @@
state: absent
action: member
register: result
failed_when: not result.changed
failed_when: not result.changed or result.failed
- name: Host test cert members absent again
ipahost:
@@ -81,7 +81,7 @@
state: absent
action: member
register: result
failed_when: result.changed
failed_when: result.changed or result.failed
- name: Host test absent
ipahost:
@@ -89,7 +89,7 @@
name: "{{ 'test.' + ipaserver_domain }}"
state: absent
register: result
failed_when: not result.changed
failed_when: not result.changed or result.failed
- name: Host test absent again
ipahost:
@@ -97,7 +97,7 @@
name: "{{ 'test.' + ipaserver_domain }}"
state: absent
register: result
failed_when: result.changed
failed_when: result.changed or result.failed
- name: Remove certificate files.
shell: