Flavors v2,v3/flavors/​{flavor_id}​ API should not return deleted flavor

Bug #1246017 reported by Tiago Mello
34
This bug affects 9 people
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
Unassigned

Bug Description

Currently, if you pass a flavor_id of a deleted for the v2/flavors/​{flavor_id}​ API call, it will return the flavor even though it has been deleted. The API shouldn't return a deleted flavor unless it is explicity requested for with any other parameter.

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

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

Changed in nova:
status: New → In Progress
tags: added: api
Revision history for this message
Christopher Yeoh (cyeoh-0) wrote :

I don't think this is a straigtforward fix.

Deleted flavors may still be in use by running instances so its quite a valid request to retreive information about them. At the very least we need the ability to specify that we want to show information for a deleted flavor.

Its unclear whether this is intentional behavior or not (being able to show deleted flavors) so I think its probably worth bringing up on the mailing list to see if we should change it for V2.

For V3 I think we need some wider changes to the flavors API, and am planning on bringing it up at summit

Changed in nova:
milestone: icehouse-1 → icehouse-2
Thierry Carrez (ttx)
Changed in nova:
milestone: icehouse-2 → icehouse-3
Thierry Carrez (ttx)
Changed in nova:
milestone: icehouse-3 → icehouse-rc1
Tracy Jones (tjones-i)
Changed in nova:
milestone: icehouse-rc1 → next
tags: added: icehouse-rc-potential
Thierry Carrez (ttx)
tags: added: icehouse-backport-potential
removed: icehouse-rc-potential
Revision history for this message
Tracy Jones (tjones-i) wrote :

this is an old bug with only an abandoned review. please reopen if you intend to fix

Changed in nova:
status: In Progress → Incomplete
Revision history for this message
Sean Dague (sdague) wrote :

The deleted flavor information should be stored in the instance system metadata, so it should be fine.

Changed in nova:
assignee: Tiago Rodrigues de Mello (timello) → nobody
milestone: next → none
Sean Dague (sdague)
Changed in nova:
status: Incomplete → 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.