OSAPI confirm/revertResize error message unhelpful

Bug #856527 reported by Brian Waldon
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Brian Waldon
Diablo
Fix Released
Undecided
Unassigned

Bug Description

The error message returned when trying to confirmResize or revertResize on an instance that hasn't been migrated is not helpful. The code currently wraps exception.MigrationNotFoundByStatus with a blank HTTPBadRequest. Adding a message specific to this case would be much better.

Brian Waldon (bcwaldon)
Changed in nova:
status: New → Triaged
Thierry Carrez (ttx)
Changed in nova:
importance: Undecided → Low
Brian Waldon (bcwaldon)
Changed in nova:
status: Triaged → In Progress
assignee: nobody → Brian Waldon (bcwaldon)
Revision history for this message
Brian Waldon (bcwaldon) wrote :
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : A change has been merged to openstack/nova

Reviewed: https://review.openstack.org/662
Committed: http://github.com/openstack/nova/commit/261b4111d481562760321bfc83d64ba35e981b5b
Submitter: Jenkins
Branch: master

 status fixcommitted
 done

commit 261b4111d481562760321bfc83d64ba35e981b5b
Author: Brian Waldon <email address hidden>
Date: Mon Sep 26 13:57:34 2011 -0400

    Explicit errors on confirm/revertResize failures

    Fixing bug 856527

    Change-Id: Ib9be618596ade2e9a899ecdc10ec9f61bf06958a

Changed in nova:
status: In Progress → Fix Committed
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix merged to nova (stable/diablo)

Reviewed: https://review.openstack.org/1130
Committed: http://github.com/openstack/nova/commit/ec2bfe91b7bf7424eb34ad371ec25eec62cd3d24
Submitter: Jenkins
Branch: stable/diablo

 status fixcommitted
 done

commit ec2bfe91b7bf7424eb34ad371ec25eec62cd3d24
Author: Brian Waldon <email address hidden>
Date: Mon Sep 26 13:57:34 2011 -0400

    Explicit errors on confirm/revertResize failures

    Fixing bug 856527

    (cherry picked from commit 261b4111d481562760321bfc83d64ba35e981b5b)

    Change-Id: I3664875a3d7f0da5603bb23ca01b990ed08eed68

Thierry Carrez (ttx)
Changed in nova:
milestone: none → essex-1
Thierry Carrez (ttx)
Changed in nova:
status: Fix Committed → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote : Please test proposed package

Hello Brian, or anyone else affected,

Accepted nova into oneiric-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

tags: added: verification-needed
Thierry Carrez (ttx)
Changed in nova:
milestone: essex-1 → 2012.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.