[tripleoclient] bash deploy ramdisk 'deprecation' changed behavior

Bug #1530883 reported by John Trowbridge
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
Critical
John Trowbridge
Liberty
Fix Released
Undecided
Unassigned

Bug Description

We backported the patch to deprecate the bash deploy ramdisk [1], however this patch also changes the default behavior of `openstack overcloud image upload` to use IPA. This behavior should not have changed on the stable/liberty branch, so we should revert the patch and only add the deprecation messages without the behavior change.

This is critical for RDO because the stable/liberty branch of IPA is missing some patches to work with the older iPXE images shipped with CentOS and RHEL.

[1] https://github.com/openstack/python-tripleoclient/commit/368ecbf78dee6f0b0c20f69db1f0793599af2a5c

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-tripleoclient (stable/liberty)

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/263289

John Trowbridge (trown)
Changed in tripleo:
assignee: nobody → John Trowbridge (trown)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/263351

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on python-tripleoclient (stable/liberty)

Change abandoned by John Trowbridge (<email address hidden>) on branch: stable/liberty
Review: https://review.openstack.org/263289
Reason: abandoned in favor of this compromise:
https://review.openstack.org/#/c/263351

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-tripleoclient (stable/liberty)

Reviewed: https://review.openstack.org/263351
Committed: https://git.openstack.org/cgit/openstack/python-tripleoclient/commit/?id=7f0190fe464ac8520949c11014a8c8bd712d3862
Submitter: Jenkins
Branch: stable/liberty

commit 7f0190fe464ac8520949c11014a8c8bd712d3862
Author: Dmitry Tantsur <email address hidden>
Date: Mon Jan 4 18:04:21 2016 +0100

    Allow to use the old deploy ramdisk

    While defaulting to IPA is a correct choice, for the transition period
    let's allow people to use the old ramdisk. Adds new option --old-deploy-image
    to the upload command and move the deprecation message to it.

    This change does not correspond to a Mitaka change, as in Mitaka the old
    ramdisk should not be supported.

    Change-Id: I311843ed95d23dfdc83d184a09c52f0f5e1b31df
    Closes-Bug: #1530883

tags: added: in-stable-liberty
Revision history for this message
Derek Higgins (derekh) wrote :

The compromise has been merged to liberty so this can be closed.

Changed in tripleo:
status: In Progress → Fix Committed
status: Fix Committed → 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.