Target host in nova DB got updated to new compute while migration failed

Bug #1653718 reported by Sahid Orentino
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
In Progress
Undecided
Sahid Orentino

Bug Description

During live-migration if the process goes in a unpredictable situation for example if QEMU suffers an issue Nova could still consider the migration has succeeded even if not.

The VM on source node can be still registered even if stopped. In the worst scenario, the operator could start it and so two VMs running could share the same disk.

We should fix that issue, do not consider a migration to have succeeded if it was not the case. For that we should handle the return migrateToURI*

[1] http://git.openstack.org/cgit/openstack/nova/tree/nova/virt/libvirt/driver.py#n6303
[2] http://git.openstack.org/cgit/openstack/nova/tree/nova/virt/libvirt/migration.py#n223

Changed in nova:
assignee: nobody → sahid (sahid-ferdjaoui)
status: New → In Progress
Revision history for this message
Daniel Berrange (berrange) wrote :

Driving any logic off the return value of migrateToURI* is really a bad idea. When migrateToURI* fails, there are many different states the VM can be in on either the src/dst host, and the migrateToURI return status gives no useful info about which applies. Migration might even have 100% succeeded, despite migrateToURI returning an error. This is why Nova stopped looking at migrateToURI error status and switched to the job info records, to solve https://bugs.launchpad.net/nova/+bug/1414065

Revision history for this message
Sean Dague (sdague) wrote :

There are no currently open reviews on this bug, changing the status back to the previous state and unassigning. If there are active reviews related to this bug, please include links in comments.

Changed in nova:
status: In Progress → New
assignee: sahid (sahid-ferdjaoui) → nobody
Revision history for this message
Sean Dague (sdague) wrote :

Found open reviews for this bug in gerrit, setting to In Progress.

review: https://review.openstack.org/409261 in branch: master
review: https://review.openstack.org/409260 in branch: master

Changed in nova:
status: New → In Progress
assignee: nobody → sahid (sahid-ferdjaoui)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (master)

Change abandoned by Sean Dague (<email address hidden>) on branch: master
Review: https://review.openstack.org/409261
Reason: This review is > 4 weeks without comment, and is not mergable in it's current state. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by sahid (<email address hidden>) on branch: master
Review: https://review.openstack.org/409260

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by sahid (<email address hidden>) on branch: master
Review: https://review.openstack.org/409261

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.