Volume type extra specs delete always returns 202 status code

Bug #1090306 reported by Rohan on 2012-12-14
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
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 Edit Tag help
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) on 2013-03-11
tags: added: api
removed: ntt
Changed in cinder:
assignee: nobody → Mike Perez (thingee)

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

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) on 2013-07-17
Changed in cinder:
milestone: none → havana-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2013-10-17
Changed in cinder:
milestone: havana-2 → 2013.2
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers