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 <rgolan(a)redhat.com>
---
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 <rgolan(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>