From 220f0673cbb4454907a3708bd603ba19bd18a99c Mon Sep 17 00:00:00 2001 From: "Shturm, Svetlana" Date: Mon, 19 Apr 2021 14:32:55 -0500 Subject: [PATCH] Remove publish job dependency from build job 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 --- playbooks/airship-airshipctl-publish-images.yaml | 1 + zuul.d/jobs.yaml | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/playbooks/airship-airshipctl-publish-images.yaml b/playbooks/airship-airshipctl-publish-images.yaml index 791af65d6..68203f5e6 100644 --- a/playbooks/airship-airshipctl-publish-images.yaml +++ b/playbooks/airship-airshipctl-publish-images.yaml @@ -13,4 +13,5 @@ - hosts: all become: yes roles: + - docker-install - airshipctl-publish-images diff --git a/zuul.d/jobs.yaml b/zuul.d/jobs.yaml index 5b19182eb..c367c9a03 100644 --- a/zuul.d/jobs.yaml +++ b/zuul.d/jobs.yaml @@ -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