airship/shipyard
Code Issues Proposed changes

Updating Docker Gate use of zuul.newrev

Browse Source 
- Zuul updated ansible to 2.7, no longer allows missing variables.
- Using default value when it isn't available.

Based on Aaron Sheffield's PS for Pegleg: https://review.openstack.org/#/c/645631/

Change-Id: I02495bc793021b429e0be62ecac0ed45e930484c
This commit is contained in:
Aaron Sheffield 2019-03-22 12:53:55 -05:00
parent 041cbcbafe
commit 12de088b9d
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
tools/gate/roles/build-images/tasks/airship-shipyard.yaml
View File

@ -28,7 +28,7 @@
environment: environment:
BRANCH: "{{ zuul.branch }}" BRANCH: "{{ zuul.branch }}"
CHANGE: "{{ zuul.change }}" CHANGE: "{{ zuul.change }}"
COMMIT: "{{ zuul.newrev }}" COMMIT: "{{ zuul.newrev | default('') }}"
PATCHSET: "{{ zuul.patchset }}" PATCHSET: "{{ zuul.patchset }}"
register: image_tags register: image_tags
@ -111,7 +111,7 @@
DOCKER_REGISTRY: "quay.io" DOCKER_REGISTRY: "quay.io"
IMAGE_PREFIX: "airshipit" IMAGE_PREFIX: "airshipit"
IMAGE_TAG: "{{ item }}" IMAGE_TAG: "{{ item }}"
COMMIT: "{{ zuul.newrev }}" COMMIT: "{{ zuul.newrev | default('') }}"
PUSH_IMAGE: "true" PUSH_IMAGE: "true"
with_items: "{{ image_tags.stdout_lines }}" with_items: "{{ image_tags.stdout_lines }}"