Roy Golan has submitted this change and it was merged. Change subject: core: cancel migration better error handling ......................................................................
core: cancel migration better error handling Cancelling migration could fail because the VM already migrated to the destination host or the VM might crash on the src host. Those conditions results error return 1, "Virtual machine does not exist" from the host which is confusing. Instead the user is guided to try again and try to track the VM events logs. Change-Id: I546cf86c1c3eeb062b9558d3c82e25035c82efbb Bug-Url: https://bugzilla.redhat.com/878778 Signed-off-by: Roy Golan <[email protected]> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllErrors.java M backend/manager/modules/dal/src/main/resources/bundles/VdsmErrors.properties M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/CancelMigrateVDSCommand.java 3 files changed, 26 insertions(+), 0 deletions(-) Approvals: Roy Golan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11225 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I546cf86c1c3eeb062b9558d3c82e25035c82efbb Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
