test_hosts_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:20 +02:00
parent 399a376451
commit d95029bbc0

View File

@@ -23,7 +23,7 @@
- name: "{{ 'test.' + ipaserver_domain }}" - name: "{{ 'test.' + ipaserver_domain }}"
force: yes force: yes
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Generate self-signed certificates. - name: Generate self-signed certificates.
shell: shell:
@@ -46,7 +46,7 @@
- "{{ lookup('file', 'cert3.b64', rstrip=False) }}" - "{{ lookup('file', 'cert3.b64', rstrip=False) }}"
action: member action: member
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Host test cert members present again - name: Host test cert members present again
ipahost: ipahost:
@@ -59,7 +59,7 @@
- "{{ lookup('file', 'cert3.b64', rstrip=False) }}" - "{{ lookup('file', 'cert3.b64', rstrip=False) }}"
action: member action: member
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed
- name: Host test cert members absent - name: Host test cert members absent
ipahost: ipahost:
@@ -73,7 +73,7 @@
state: absent state: absent
action: member action: member
#register: result #register: result
#failed_when: not result.changed #failed_when: not result.changed or result.failed
- name: Host test cert members absent again - name: Host test cert members absent again
ipahost: ipahost:
@@ -87,7 +87,7 @@
state: absent state: absent
action: member action: member
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed
- name: Host test absent - name: Host test absent
ipahost: ipahost:
@@ -96,7 +96,7 @@
- name: "{{ 'test.' + ipaserver_domain }}" - name: "{{ 'test.' + ipaserver_domain }}"
state: absent state: absent
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Remove certificate files. - name: Remove certificate files.
shell: shell: