commit 972d4e0eb391e83fe8d3020ff95db0e6a840a224
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.
Change-Id: Ic7d78ad43a2bad7f932c22c98944accbbed9e9e2
Partial-Bug: #1821594
(cherry picked from commit 408ef8f84a698f764aa5d769d6d01fd9340de2e5)
Reviewed: https:/ /review. openstack. org/649421 /git.openstack. org/cgit/ openstack/ nova/commit/ ?id=972d4e0eb39 1e83fe8d3020ff9 5db0e6a840a224
Committed: https:/
Submitter: Zuul
Branch: stable/stein
commit 972d4e0eb391e83 fe8d3020ff95db0 e6a840a224
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 64aa5d769d6d01f d9340de2e5)
Partial-Bug: #1821594
(cherry picked from commit 408ef8f84a698f7