zuul/zuul-jobs
Code Issues Proposed changes

Merge "Do not set buildset_fact if it's not present in results.json"

Browse Source
This commit is contained in:
Zuul 2020-04-28 16:22:28 +00:00 committed by Gerrit Code Review
commit 6fb73060ec
2 changed files with 18 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
roles/build-container-image/tasks/main.yaml
View File

@ -1,9 +1,16 @@
- name: Check for results.json
stat:
path: "{{ zuul.executor.work_root }}/results.json"
register: result_json_stat
# This can be removed if we add this functionality to Zuul directly # This can be removed if we add this functionality to Zuul directly
- name: Load information from zuul_return - name: Load information from zuul_return
when: buildset_registry is not defined
set_fact: set_fact:
buildset_registry: "{{ (lookup('file', zuul.executor.work_root + '/results.json') | from_json)['buildset_registry'] }}" buildset_registry: "{{ (lookup('file', zuul.executor.work_root + '/results.json') | from_json)['buildset_registry'] }}"
ignore_errors: true when:
- buildset_registry is not defined
- result_json_stat.stat.exists
- "'buildset_registry' in (lookup('file', zuul.executor.work_root + '/results.json') | from_json)"
- name: Set container filename arg - name: Set container filename arg
set_fact: set_fact:

11
roles/build-docker-image/tasks/main.yaml
View File

@ -1,9 +1,16 @@
- name: Check for results.json
stat:
path: "{{ zuul.executor.work_root }}/results.json"
register: result_json_stat
# This can be removed if we add this functionality to Zuul directly # This can be removed if we add this functionality to Zuul directly
- name: Load information from zuul_return - name: Load information from zuul_return
when: buildset_registry is not defined
set_fact: set_fact:
buildset_registry: "{{ (lookup('file', zuul.executor.work_root + '/results.json') | from_json)['buildset_registry'] }}" buildset_registry: "{{ (lookup('file', zuul.executor.work_root + '/results.json') | from_json)['buildset_registry'] }}"
ignore_errors: true when:
- buildset_registry is not defined
- result_json_stat.stat.exists
- "'buildset_registry' in (lookup('file', zuul.executor.work_root + '/results.json') | from_json)"
- name: Build docker images - name: Build docker images
include_tasks: build.yaml include_tasks: build.yaml