Volume type extra specs delete always returns 202 status code

Bug #1090306 reported by Rohan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Cian O'Driscoll

Bug Description

Branch : master

Api : DELETE http://<ip>:8776/v1/<tenant_id>/types/<vol_type_id>/extra_specs/<spec_name>

Issue : The above api http call always returns http status code of 202 , even if the <spec_name> does not exist, in which case it should return http status code 404.

Tags: api
Revision history for this message
John Griffith (john-griffith) wrote :

Most of the extensions just always return the 202 as you've pointed out. We could probably use some clean-up and better reporting here but I think this will be a V2 API task as opposed to changing all of the behaviors in V1.

Changed in cinder:
importance: Undecided → Low
status: New → Triaged
importance: Low → Undecided
Mike Perez (thingee)
tags: added: api
removed: ntt
Changed in cinder:
assignee: nobody → Mike Perez (thingee)
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/31044

Changed in cinder:
assignee: Mike Perez (thingee) → Cian O'Driscoll (dricco)
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

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

commit de84fd7b1539893bf48ed8b36db1bfdeaf05d7a9
Author: Cian O'Driscoll <email address hidden>
Date: Thu May 30 13:00:54 2013 +0000

    Return 404 from delete of extra spec if not found

    API service now returns a 404 if the volume type that the
    user is trying to delete doesn't exist.

    Fixed Bug 1090306.

    Change-Id: I3c1d4fdac1c2492f532ab4dbfd5c7aaf0ffa6fa4

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