commit 408ef8f84a698f764aa5d769d6d01fd9340de2e5
Author: Matt Riedemann <email address hidden>
Date: Mon Mar 25 13:16:42 2019 -0400
Error out migration when confirm_resize fails
If anything fails and raises an exception during
confirm_resize, the migration status is stuck in
"confirming" status even though the instance status
may be "ERROR".
This change adds the errors_out_migration decorator
to the confirm_resize method to make sure the migration
status is "error" if an error is raised.
In bug 1821594 it was the driver.confirm_migration
method that raised some exception, so a unit test is
added here which simulates a similar scenario.
This only partially closes the bug because we are still
leaking allocations on the source node resource provider
since _delete_allocation_after_move is not called. That
will be dealt with in a separate patch.
Reviewed: https:/ /review. openstack. org/647546 /git.openstack. org/cgit/ openstack/ nova/commit/ ?id=408ef8f84a6 98f764aa5d769d6 d01fd9340de2e5
Committed: https:/
Submitter: Zuul
Branch: master
commit 408ef8f84a698f7 64aa5d769d6d01f d9340de2e5
Author: Matt Riedemann <email address hidden>
Date: Mon Mar 25 13:16:42 2019 -0400
Error out migration when confirm_resize fails
If anything fails and raises an exception during
confirm_resize, the migration status is stuck in
"confirming" status even though the instance status
may be "ERROR".
This change adds the errors_ out_migration decorator
to the confirm_resize method to make sure the migration
status is "error" if an error is raised.
In bug 1821594 it was the driver. confirm_ migration
method that raised some exception, so a unit test is
added here which simulates a similar scenario.
This only partially closes the bug because we are still allocation_ after_move is not called. That
leaking allocations on the source node resource provider
since _delete_
will be dealt with in a separate patch.
Change-Id: Ic7d78ad43a2bad 7f932c22c98944a ccbbed9e9e2
Partial-Bug: #1821594