[libvirt] [PATCH 3/5] qemu: migrate: Save MigratePerform error in MigrateFinish.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This way we won't squash the original error report in
MigratePerform, as is done for P2P and Tunneled migration.

Signed-off-by: Cole Robinson <crobinso@xxxxxxxxxx>
---
 src/qemu/qemu_driver.c |    8 ++++++++
 1 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index daa6f94..fc6b9fa 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -7670,8 +7670,12 @@ qemudDomainMigrateFinish2 (virConnectPtr dconn,
     virDomainObjPtr vm;
     virDomainPtr dom = NULL;
     virDomainEventPtr event = NULL;
+    virErrorPtr origerr = NULL;
     int newVM = 1;
 
+    /* Migration failed. Save the current error so nothing squashes it */
+    origerr = virSaveLastError();
+
     qemuDriverLock(driver);
     vm = virDomainFindByName(&driver->domains, dname);
     if (!vm) {
@@ -7765,6 +7769,10 @@ endjob:
         vm = NULL;
 
 cleanup:
+    if (origerr) {
+        virSetError(origerr);
+        virFreeError(origerr);
+    }
     if (vm)
         virDomainObjUnlock(vm);
     if (event)
-- 
1.6.5.2

--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]