zuul/zuul-jobs
Code Issues Proposed changes

container-logs: use failed_when: instead of ignore_errors:

Browse Source 
Change-Id: I3b59c126b8d1dff38eef40003e3c63846dec959a
This commit is contained in:
Albin Vass 2020-04-27 20:53:45 +02:00
parent 605fb74921
commit a7da59703f
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
roles/collect-container-logs/tasks/main.yaml
View File

@ -1,7 +1,7 @@
- name: List containers - name: List containers
command: "{{ container_command }} ps -a --format '{{ '{{ .Names }}' }}'" command: "{{ container_command }} ps -a --format '{{ '{{ .Names }}' }}'"
register: docker_containers register: docker_containers
ignore_errors: true failed_when: false
- name: Create container log dir - name: Create container log dir
file: file:
@ -17,7 +17,7 @@
shell: "{{ container_command }} logs {{ zj_container_name }} &> {{ ansible_user_dir }}/zuul-output/logs/{{ container_command }}/{{ zj_container_name }}.txt" shell: "{{ container_command }} logs {{ zj_container_name }} &> {{ ansible_user_dir }}/zuul-output/logs/{{ container_command }}/{{ zj_container_name }}.txt"
args: args:
executable: /bin/bash executable: /bin/bash
ignore_errors: true failed_when: false
- name: Open container logs permissions - name: Open container logs permissions
file: file: