snapshot metadata mapper is not exist in v2 APIRouter

Bug #1227799 reported by Saurabh
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Saurabh

Bug Description

cinder/api/v2/router.py doesn't have mapper for snapshot metadata, but cinder/api/v2/snapshot_metadata.py exist with all actions.

So either we add mapper or delete snapshot_metadata.py.

Tags: api
Changed in cinder:
assignee: nobody → Saurabh (sourabh-chordiya)
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/47440

Changed in cinder:
status: New → In Progress
Revision history for this message
Ghanshyam Mann (ghanshyammann) wrote :

Snapshot metadata API was implemented in in blueprint- https://blueprints.launchpad.net/cinder/+spec/update-snap-metadata . But looks like mapping in V2 router class was missed somehow.

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

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

commit dee4c862f241a6f12fdb6f0e7c48b32063ef6fe5
Author: saurabh <email address hidden>
Date: Thu Sep 19 23:37:35 2013 +0530

    Added mapper for snapshot_metadata

    cinder/api/v2/snapshot_metadata.py exist with all actions
    but mapper missing for it.

    Closes-bug: 1227799

    Change-Id: I9a95c68a8de620626042951fa15a555621ebbc17

Changed in cinder:
status: In Progress → Fix Committed
tags: added: api
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → havana-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: havana-rc1 → 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.