Remove imageutils from oslo-incubator and move to using imageutils in oslo_utils

Bug #1517861 reported by Ivan Kolodyazhny
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
High
Jay Bryant

Bug Description

Cidner should not use imageutils from oslo-incubator

Ivan Kolodyazhny (e0ne)
Changed in cinder:
assignee: nobody → Ivan Kolodyazhny (e0ne)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

Changed in cinder:
status: New → In Progress
Eric Harney (eharney)
summary: - Move umageutils from oslo-incubator to cinder
+ Move imageutils from oslo-incubator to cinder
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (master)

Change abandoned by Ivan Kolodyazhny (<email address hidden>) on branch: master
Review: https://review.openstack.org/247480
Reason: It will be moved to oslo.utils: https://review.openstack.org/#/c/249107/

Jay Bryant (jsbryant)
summary: - Move imageutils from oslo-incubator to cinder
+ Remove imageutils from oslo-incubator and move to using imageutils in
+ oslo_utils
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

Changed in cinder:
assignee: Ivan Kolodyazhny (e0ne) → Jay Bryant (jsbryant)
Jay Bryant (jsbryant)
Changed in cinder:
importance: Undecided → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/290875
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=388a131c3dd720939a1fa697a8b54a1a33457f64
Submitter: Jenkins
Branch: master

commit 388a131c3dd720939a1fa697a8b54a1a33457f64
Author: Jay S Bryant <email address hidden>
Date: Tue Mar 8 05:22:39 2016 -0600

    Remove remaining oslo-incubator code from Cinder

    Most of the parts for oslo-incubator have moved to
    oslo libraries. Cinder only had imageutils.py and
    _i18n.py remaining in cinder/openstack/common. I
    was to able remove both of these remaining incubator
    components.

    oslo_utils now has imageutils available. I moved
    cinder/image/image_utils.py to use the new
    imageutils from oslo_utils and updated all the other
    places that were using the old imageutils import.

    _i18n was only being used in scheduler/base_filter.py
    which appears to have been accidentally left when the
    scheduler was previously pulled out of incubator. I
    fixed this oversight.

    Finally, this patch deletes cinder/openstack/common
    cinder/openstack and the openstack-common.conf file.

    Change-Id: I8fe0b5d6c278715620f93f682289df7ae60dbeba
    Closes-bug: 1517861

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