airship/airshipctl
Code Issues Proposed changes

Remove publish job dependency from build job

Browse Source 
With latest changes build images jobs uses zuul.change variable
which is not defined for merge event.
Also using build job as a parent for publish creates an unnessesarly
execution of make command which we repeat later under publish playbook.

That change removes that dependency.

Change-Id: I7e02c2f3ef2ccd8b9db9872b123222020f80664a
This commit is contained in:
Shturm, Svetlana 2021-04-19 14:32:55 -05:00
parent cac8564b59
commit 220f0673cb
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
playbooks/airship-airshipctl-publish-images.yaml
View File

@ -13,4 +13,5 @@
- hosts: all
become: yes
roles:
- docker-install
- airshipctl-publish-images

4
zuul.d/jobs.yaml
View File

@ -248,8 +248,8 @@
voting: false
- job:
name: airship-airshipctl-publish-image
parent: airship-airshipctl-build-image
post-run: playbooks/airship-airshipctl-publish-images.yaml
nodeset: airship-airshipctl-single-node
run: playbooks/airship-airshipctl-publish-images.yaml
irrelevant-files: *noncodefiles
secrets:
- name: airshipctl_image_repo_credentials