Change in ovirt-engine[master]: engine : ovirt-node enter installing status when the install...

mkublin at redhat.com mkublin at redhat.com
Tue Apr 9 13:15:40 UTC 2013


Michael Kublin has submitted this change and it was merged.

Change subject: engine : ovirt-node enter installing status when the install fails
......................................................................


engine : ovirt-node enter installing status when the install fails

When ovirt-node installation fails because of missing iso file
the status of the host is left as Installing.

This patch adds VdsDynamic data to compensation context if can do action
of InstalllVdsCommand fails. The status is reverted back to the previous
status in compensation logic. The error messages are added to the
return values CanDoActionMessages so the message can be displayed
in cli

Change-Id: I2d74e946cb68d2b9dbc11fc5c6bed9f63ba35fe2
Bug-Url: https://bugzilla.redhat.com/947982
Signed-off-by: Ravi Nori <rnori at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsCommand.java
1 file changed, 16 insertions(+), 0 deletions(-)

Approvals:
  Michael Kublin: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/13622
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I2d74e946cb68d2b9dbc11fc5c6bed9f63ba35fe2
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpastern at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list