Ironic

IPA carrying legacy code

Bug #1404362 reported by Jim Rollenhagen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Undecided
Jim Rollenhagen
Ironic 2015.1.0 "kilo"

Bug Description

Ironic-python-agent carries a bunch of legacy code that is no longer used. Let's remove it.

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

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

Changed in ironic:
assignee: nobody → Jim Rollenhagen (jim-rollenhagen)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

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

Reviewed: https://review.openstack.org/143167
Committed: https://git.openstack.org/cgit/openstack/ironic-python-agent/commit/?id=5585d96b8a6fae7aa1d7f484f6fe3a61e522ee7f
Submitter: Jenkins
Branch: master

commit 5585d96b8a6fae7aa1d7f484f6fe3a61e522ee7f
Author: Jim Rollenhagen <email address hidden>
Date: Fri Dec 19 11:36:38 2014 -0800

    Remove configdrive builder

    We're leaving this up to Nova these days, remove the unused code
    here.

    Change-Id: I2c4ac1bffa59146ae54c8b758f62314454aa8369
    Partial-Bug: #1404362

Changed in ironic:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/143168
Committed: https://git.openstack.org/cgit/openstack/ironic-python-agent/commit/?id=bf0129d81b14be3594db3863389f308f2bc62c7f
Submitter: Jenkins
Branch: master

commit bf0129d81b14be3594db3863389f308f2bc62c7f
Author: Jim Rollenhagen <email address hidden>
Date: Fri Dec 19 11:49:10 2014 -0800

    Remove /mnt/sys support

    This code was here to allow IPA to be ran inside a docker container
    with /sys bind-mounted in to /mnt/sys. IPA no longer can run inside
    a docker container anyway, so remove this.

    Change-Id: Ia6a9c2e49870b7a74087e3fed2fc42d5b8b6b81c
    Closes-Bug: #1404362

Thierry Carrez (ttx)
Changed in ironic:
milestone: none → kilo-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ironic:
milestone: kilo-2 → 2015.1.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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