test_users_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:43 +02:00
parent e0defaaebe
commit b1edf574d7

View File

@@ -41,7 +41,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: User test cert members present again - name: User test cert members present again
ipauser: ipauser:
@@ -54,7 +54,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: User test cert members absent - name: User test cert members absent
ipauser: ipauser:
@@ -68,7 +68,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: User test cert members absent again - name: User test cert members absent again
ipauser: ipauser:
@@ -82,7 +82,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: User test absent - name: User test absent
ipauser: ipauser:
@@ -91,7 +91,7 @@
- name: test - name: test
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: