Need to include 'metadata' in Cinder's VolumeSnapshot object in Horizon API

Bug #1542468 reported by Vivek Agrawal
18
This bug affects 4 people
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Wishlist
Vivek Agrawal

Bug Description

The VolumeSnapshot object in 'horizon/openstack_dashboard/api/cinder.py'. This object is accessed by multiple views in Horizon. The object does not have metadata populated for the snapshot.

Changed in horizon:
assignee: nobody → Vivek Agrawal (vivek-agrawal)
Revision history for this message
Vivek Agrawal (vivek-agrawal) wrote :

In case I am extending horizon functionality and have stored some metadata information for snapshot_metadata. Then the key values of metadata are not accessible to Horizon through the VolumeSnapshots object of cinder.py in horizon API. If we have a list of snapshots then we need to iterate and get snapshot metadata separately which is not a good idea. This functionality was present in earlier release.

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

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

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

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on horizon (master)

Change abandoned by David Lyle (<email address hidden>) on branch: master
Review: https://review.openstack.org/281629
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Changed in horizon:
importance: Undecided → Wishlist
milestone: none → ongoing
Changed in horizon:
milestone: ongoing → next
Changed in horizon:
assignee: Vivek Agrawal (vivek-agrawal) → Timur Sufiev (tsufiev-x)
Timur Sufiev (tsufiev-x)
Changed in horizon:
assignee: Timur Sufiev (tsufiev-x) → Vivek Agrawal (vivek-agrawal)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/279667
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=a05883d88a22042a795e48e167c3e2c746f2c89d
Submitter: Jenkins
Branch: master

commit a05883d88a22042a795e48e167c3e2c746f2c89d
Author: Vivek Agrawal <email address hidden>
Date: Thu Feb 11 13:34:52 2016 -0800

    Addition of metadata for snapshot in VolumeSnapshot object

    When accessing the snapshot object in Horizon codebase, we do not have
    access to the metadata of the snapshot. This is because the
    VolumeSnapshot object does not have metadata as a member. So added
    'metadata' to the VolumeSnapshot object. The fix should provide access
    to metadata.
    Added test data and test cases for snapshot object with metadata.

    Closes-Bug: 1542468

    Change-Id: I4ae2e7a74d3ddbff8493b42f8f86c7002229cf8c

Changed in horizon:
status: In Progress → Fix Released
Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/horizon 10.0.0.0b2

This issue was fixed in the openstack/horizon 10.0.0.0b2 development milestone.

Changed in horizon:
status: Fix Released → Confirmed
status: Confirmed → Fix Released
Changed in horizon:
milestone: next → newton-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.