attachmnet_get_by_host|instance shouldn't require a volume_id

Bug #1632433 reported by John Griffith
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
John Griffith

Bug Description

If I want to get attachments by instance, or by attached_host I shouldn't have to specify a volume-id.

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/385122

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

Reviewed: https://review.openstack.org/385122
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=6f174b412696bfa6262a5bea3ac42f45efbbe2ce
Submitter: Jenkins
Branch: master

commit 6f174b412696bfa6262a5bea3ac42f45efbbe2ce
Author: John Griffith <email address hidden>
Date: Tue Oct 11 12:40:53 2016 -0600

    Remove volid from attachments_get_by_host|instance

    Attachments_get_by_host|instance should be just that, associating
    them with a specified volume-id doesn't really solve the problem.

    If that relationship is needed a simple get_by_volume will work with
    some inspection.

    This patch removes the volume_id arg from those get methods. A follow
    up patch will add get_all with filters for more complex relationships.

    Change-Id: Ic5ffdced96fdf780cce2a1227c5f2a599860f0ca
    Closes-Bug: #1632433

Changed in cinder:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 10.0.0.0b1

This issue was fixed in the openstack/cinder 10.0.0.0b1 development milestone.

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.