Brick's iser.py has dependency on cinder

Bug #1210312 reported by Walt Boring
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
High
Walt Boring

Bug Description

brick/iser/iser.py has dependency on cinder exception.py as well as volume_utils.

Nothing in brick can have a dependency on cinder, or it won't work in other projects (nova, glance)

Tags: brick
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/40984

Changed in cinder:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/40984
Committed: http://github.com/openstack/cinder/commit/d1d6b808afec4a4d74d004dc48832eaaf1b415b1
Submitter: Jenkins
Branch: master

commit d1d6b808afec4a4d74d004dc48832eaaf1b415b1
Author: Walter A. Boring IV <email address hidden>
Date: Thu Aug 8 15:29:54 2013 -0700

    Remove Brick iser dependency on cinder

    This patch removes the brick iser.py's dependency
    on cinder exceptions and volume_utils.
    This required moving some exceptions out of cinder's
    exception.py that the iser.py raises. Also had
    to create a BrickException and refactor existing
    brick exceptions to use the new BrickException model.

    Fixes Bug #1210312

    Change-Id: I672375807fed4952e5321fbcd9b57b9ef369f68f

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: havana-3 → 2013.2
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.