Delorean fails to detect ZUUL_BRANCH when building rpms

Bug #1735620 reported by Emilien Macchi
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
High
wes hayutin

Bug Description

If you submit a patch in triploe-ci (which doesn't have stable branch), with a Depends-On on TripleO Heat Templates master, newton/ocata jobs will fail.
In fact, Delorean tries to build the package by using the distgit from master, with the THT code from newton.

See: http://logs.openstack.org/03/523903/7/check-tripleo/tripleo-ci-centos-7-ovb-ha-oooq-newton/2e7c996/logs/delorean_logs/8e/16/8e168a8ddf7ba98fd8f12f5ee0ec1bca0231192f_dev/build.log.txt.gz

This is the patch where you can see the problem: https://review.openstack.org/#/c/523903/

openstack-tripleo-heat-templates-5.3.7-0.20171201030248.8e168a8.el7 - 8e168a8 is the latest commit in stable/newton although the delorean is using the spec file from master, it tried to run " cp -ar common" which doesn't exist in the newton distgit...

Tags: ci quickstart
Ronelle Landy (rlandy)
tags: added: quickstart
tags: removed: quickstart
Revision history for this message
wes hayutin (weshayutin) wrote :

Looking at the review.. I only see CI repos in the changes. AFAICT delorean should not be building any packages for this change. https://review.openstack.org/#/c/523903/

Changed in tripleo:
assignee: nobody → wes hayutin (weshayutin)
tags: added: ci quickstart
Revision history for this message
Javier Peña (jpena-c) wrote :

The issue is in the tripleo-quickstart-extras playbook, see [1]. All changes in the dependency matrix are using "master" as a release, so artg_release is set to master by the playbook (see [2]) and DLRN is configured to use master as a release instead of Newton or Ocata.

I think we could fix it by adding a variable to force the release, and set it accordingly in certain jobs. However, I'm not familiar enough with the playbooks, so I'd appreciate some more insights.

About why THT is built, it looks like Zuul made it a related change, see [3] where it includes https://review.openstack.org/#/c/524346 as a change in Zuul.

[1] - https://github.com/openstack/tripleo-quickstart-extras/blob/master/roles/build-test-packages/tasks/main.yml#L119-L124
[2] - http://logs.openstack.org/03/523903/8/check-tripleo/tripleo-ci-centos-7-ovb-ha-oooq-newton/7190c49/job-output.txt.gz#_2017-12-01_06_04_24_895944
[3] - http://logs.openstack.org/03/523903/8/check-tripleo/tripleo-ci-centos-7-ovb-ha-oooq-newton/7190c49/job-output.txt.gz#_2017-12-01_06_04_24_399019

Changed in tripleo:
milestone: queens-3 → queens-rc1
Changed in tripleo:
milestone: queens-rc1 → rocky-1
Changed in tripleo:
milestone: rocky-1 → rocky-2
Changed in tripleo:
milestone: rocky-2 → rocky-3
Changed in tripleo:
milestone: rocky-3 → rocky-rc1
Changed in tripleo:
milestone: rocky-rc1 → stein-1
Changed in tripleo:
milestone: stein-1 → stein-2
Changed in tripleo:
milestone: stein-2 → stein-3
Changed in tripleo:
milestone: stein-3 → stein-rc1
Changed in tripleo:
milestone: stein-rc1 → train-1
Changed in tripleo:
milestone: train-1 → train-2
Changed in tripleo:
milestone: train-2 → train-3
Changed in tripleo:
milestone: train-3 → ussuri-1
Revision history for this message
Daniel Bengtsson (damani42) wrote :

Is this still an issue?

wes hayutin (weshayutin)
Changed in tripleo:
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.