consistency group api needs to return volume type ID

Bug #1489957 reported by Richard Hagarty
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Low
Deepti Ramakrishna

Bug Description

Volume type is required to create a consistency group, but the API to get consistency groups does not include this value.

Horizon needs this value for its "Add volumes to consistency group" panel so that it can filter the "available volumes" list to only show volumes that match the volume type associated with the consistency group.

Tags: cg
Changed in cinder:
assignee: nobody → Deepti Ramakrishna (dramakri)
tags: added: cg
Xing Yang (xing-yang)
Changed in cinder:
status: New → Triaged
Revision history for this message
Richard Hagarty (richard-hagarty) wrote :

@Deepti Ramakrishna - any progress on this defect?

Revision history for this message
Glenn W (glenn-p-williamson) wrote :

We have been working on recreating this item but ran into two issues,

https://bugs.launchpad.net/cinder/+bug/1503965
https://bugs.launchpad.net/cinder/+bug/1503968

Deepti will provide further updates later.

Revision history for this message
Alex O'Rourke (alex-orourke) wrote :

Those two bugs are not related to this bug in particular.

I would be interested in taking this over if needed.

Revision history for this message
Xing Yang (xing-yang) wrote :

Note that these two bugs below are marked as Invalid. Please do not work on them. This bug report we are in right now is valid. Thanks.

https://bugs.launchpad.net/cinder/+bug/1503965
https://bugs.launchpad.net/cinder/+bug/1503968

Changed in cinder:
importance: Undecided → Low
Revision history for this message
Glenn W (glenn-p-williamson) wrote :

Got it, will close with Deepti shortly

Revision history for this message
Deepti Ramakrishna (dramakri) wrote :

Hi Richard,

I have already spent some time on this bug. Can I take a couple more days time? If I don't fix it in next few days, I will update you and unassign myself. Is that okay?

Thanks,
Deepti

Revision history for this message
Richard Hagarty (richard-hagarty) wrote :

That's fine Deepti.

But please let us know if you have any issues. We are really looking forward to using this functionality in Horizon.

Thanks,
Rich

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

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

Reviewed: https://review.openstack.org/245624
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=926c6f9fb5da28cc09499d7539581581e825e62c
Submitter: Jenkins
Branch: master

commit 926c6f9fb5da28cc09499d7539581581e825e62c
Author: Deepti Ramakrishna <email address hidden>
Date: Thu Nov 19 00:43:49 2015 -0800

    CG API should return volume type IDs

    The create method of consistency group API requires a list of
    volume type ids; volumes belonging to only these types can be added
    to the consistency group. The get method of the consistency group API
    does not, however, return the volume type id information.

    Horizon needs this value for its "Add volumes to consistency group"
    panel so that it can filter the "available volumes" list to only
    show volumes that match the volume type associated with the consistency
    group.

    This was fixed by propagating the volume type id information in
    the api layer (this information is stored in the db, it just wasn't
    being propagated).

    APIImpact
    The get method of the consistency group API now returns the volume
    type ids information.

    DocImpact
    The screenshots in the admin guide (see
    http://docs.openstack.org/admin-guide-cloud/blockstorage-consistency-groups.html)
    should be updated to show this new field.

    Change-Id: Icc3206c67227af38a7bf89ea0145942f750b506b
    Closes-Bug: #1489957

Changed in cinder:
status: In Progress → Fix Committed
Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/cinder 8.0.0.0b1

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

Thierry Carrez (ttx)
Changed in cinder:
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.