[RFE] Use stable branch of agent when testing stable branches

Bug #1561756 reported by Jim Rollenhagen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Wishlist
Jim Rollenhagen

Bug Description

Currently all tests, whether master or stable, uses the master build of IPA to test. Stable branches should use the correct version of the agent tarball. The changes this will involve:

* Change agent post job to upload the ramdisk with the branch name appended (at http://tarballs.openstack.org/ironic-python-agent/coreos/files/). This may get tricky as the artifacts to be uploaded is expressed in YAML, and project-config is not branched: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic-python-agent-jobs.yaml
* Change devstack to fetch the ramdisk for the correct branch. Note that this will need to be backported across devstack stable branches.

tags: added: rfe
Changed in ironic:
status: New → Confirmed
importance: Undecided → Wishlist
tags: added: gate
tags: added: rfe-approved
removed: rfe
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic (master)

Fix proposed to branch: master
Review: https://review.openstack.org/297479

Changed in ironic:
assignee: nobody → Jim Rollenhagen (jim-rollenhagen)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/299317

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

Reviewed: https://review.openstack.org/297479
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=8e15d6a1388f49026c4d35ba87219a49ccc3c134
Submitter: Jenkins
Branch: master

commit 8e15d6a1388f49026c4d35ba87219a49ccc3c134
Author: Jim Rollenhagen <email address hidden>
Date: Thu Mar 24 17:45:15 2016 -0700

    Allow fetching IPA ramdisk with branch name

    This adds branch name to the default URLs for IPA images, ensuring that
    devstack runs will use the correct branch of IPA for tests.

    It introduces a variable IPA_DOWNLOAD_BRANCH to accomplish this, which
    defaults to master. This default will be set to the branch name on
    stable branches, e.g. stable/mitaka. It is modified to match our URL
    scheme for agent images, e.g. stable-mitaka.

    Change-Id: I3ec6dafb9522b5fe79e0c6681c2578d2a723380a
    Closes-Bug: #1561756

Changed in ironic:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (stable/mitaka)

Reviewed: https://review.openstack.org/299317
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=6510b8f86b10e92e2462fd4796bd6ec7ad710c4b
Submitter: Jenkins
Branch: stable/mitaka

commit 6510b8f86b10e92e2462fd4796bd6ec7ad710c4b
Author: Jim Rollenhagen <email address hidden>
Date: Thu Mar 24 17:45:15 2016 -0700

    Allow fetching IPA ramdisk with branch name

    This adds branch name to the default URLs for IPA images, ensuring that
    devstack runs will use the correct branch of IPA for tests.

    It introduces a variable IPA_DOWNLOAD_BRANCH to accomplish this, which
    defaults to master. This default will be set to the branch name on
    stable branches, e.g. stable/mitaka. It is modified to match our URL
    scheme for agent images, e.g. stable-mitaka.

    This cherry-pick is modified to set the default to stable/mitaka.

    Change-Id: I3ec6dafb9522b5fe79e0c6681c2578d2a723380a
    Closes-Bug: #1561756
    (cherry picked from commit 8e15d6a1388f49026c4d35ba87219a49ccc3c134)

tags: added: in-stable-mitaka
Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/ironic 5.1.1

This issue was fixed in the openstack/ironic 5.1.1 release.

Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/ironic 6.0.0

This issue was fixed in the openstack/ironic 6.0.0 release.

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.