containerized-undercloud-upgrades: dlrn tasks don't run

Bug #1768997 reported by Emilien Macchi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
Critical
Ronelle Landy

Bug Description

With the new shiny CI job that test upgrades from an undercloud (not containerized) on Queens, which is then upgraded to a containerized Rocky undercloud, quickstart never runs the Ansible tasks that build deloreans package for the project that we're testing. e.g python-tripleoclient.

You can see the "build or not to build" thing:
http://logs.openstack.org/67/566067/1/check/tripleo-ci-centos-7-containerized-undercloud-upgrades/c84191e/job-output.txt.gz#_2018-05-03_14_03_01_699534

That is skipped (why?). So because we skip it, we never run delorean so we don't actually test the change in the project.

After some debugging, we know that it happens for any project.

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

root cause is that with this job and any upgrade job atm. The variable "release" may not match the the release in the zuul change.

In this case the job starts with release "queens" but is triggered from a patch on master

- name: Clone and build the specific DLRN projects
  include: dlrn-build.yml artg_change={{ item }}
  with_items: '{{ artg_change_list|default([]) }}'
  when:
    - item.project not in artg_skipped_projects
    - release in item.branch

The correct solution to this issue is complex and is being worked on now in [2]

[1]https://github.com/openstack/tripleo-quickstart-extras/blob/master/roles/build-test-packages/tasks/main.yml#L139
[2]https://trello.com/c/cuKevn28/728-sprint-13-upgrades-goals

Changed in tripleo:
importance: High → Critical
Revision history for this message
wes hayutin (weshayutin) wrote :
Revision history for this message
wes hayutin (weshayutin) wrote :

trying w/o the when condition https://review.openstack.org/#/c/566393/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-quickstart-extras (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/568946

Revision history for this message
Ronelle Landy (rlandy) wrote :

We have some initial test results using a patch based on the releases script integration into toci-quickstart - see logs: http://logs.openstack.org/31/568931/2/check/tripleo-ci-centos-7-containerized-undercloud-upgrades/9cae57f/logs/undercloud/home/zuul/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-quickstart-extras (master)

Reviewed: https://review.openstack.org/568946
Committed: https://git.openstack.org/cgit/openstack/tripleo-quickstart-extras/commit/?id=d13dc308fae4a12657238f0fd5afed03eeabb3c2
Submitter: Zuul
Branch: master

commit d13dc308fae4a12657238f0fd5afed03eeabb3c2
Author: Wes Hayutin <email address hidden>
Date: Wed May 16 15:45:06 2018 -0400

    execute the build / install of zuul changes in upgrade

    At the moment the build-test-packages and install-built-repo
    are only called in the n version not the n+1. If the
    dependency is master it will not get installed, however
    a queens patch would.

    Once we flip the upgrade role we need to recheck
    for zuul changes on the n+1 version and rebuild them
    if they are found

    Related-Bug: #1768997
    Change-Id: I1d1401753f4576317805bc4c556b37f71faa60ed

Changed in tripleo:
milestone: rocky-2 → rocky-3
Ronelle Landy (rlandy)
Changed in tripleo:
status: Triaged → Fix Released
assignee: nobody → Ronelle Landy (rlandy)
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.