Change in ovirt-engine[master]: core: cancel migration better error handling

rgolan at redhat.com rgolan at redhat.com
Thu Jan 31 09:10:31 UTC 2013


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



More information about the Engine-commits mailing list