Duplicate FlavorNotFound handled in server create API

Bug #1256427 reported by Matt Riedemann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Matt Riedemann

Bug Description

The FlavorNotFound exception is handled in multiple places in the server create API, the first being:

https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L951

and the latter being:

https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L970

They both return a 400 response and only the former is used, so remove the latter.

This is both in the v2 and v3 API.

Matt Riedemann (mriedem)
Changed in nova:
importance: Undecided → Low
assignee: nobody → Matt Riedemann (mriedem)
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/59239

Changed in nova:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/59239
Committed: http://github.com/openstack/nova/commit/2d2e30dba5c571c05dd4191c1b023fe4014c1751
Submitter: Jenkins
Branch: master

commit 2d2e30dba5c571c05dd4191c1b023fe4014c1751
Author: Matt Riedemann <email address hidden>
Date: Wed Nov 27 09:05:06 2013 -0800

    Remove duplicate FlavorNotFound exception handling in server create API

    The v2 and v3 openstack server create APIs have duplicate handling for
    the FlavorNotFound exception. This patch removes the duplicate handling
    which is found in a generic catch-all block for each API.

    Closes-Bug: #1256427

    Change-Id: I78344cd588178c501398147f3afcd418e0744b0d

Changed in nova:
status: In Progress → Fix Committed
Changed in nova:
milestone: none → icehouse-1
Thierry Carrez (ttx)
Changed in nova:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: icehouse-1 → 2014.1
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.