Change in ovirt-engine[master]: host-deploy: sync node upgrade to vds deploy

yzaslavs at redhat.com yzaslavs at redhat.com
Tue Jul 30 15:09:02 UTC 2013


Yair Zaslavsky has submitted this change and it was merged.

Change subject: host-deploy: sync node upgrade to vds deploy
......................................................................


host-deploy: sync node upgrade to vds deploy

during the move to network processing within engine, the node upgrade
and vds deploy became out of sync, it could have been easily been solved
by adding handleError(e, VDSStatus.InstallFailed) at exception handler,
however better to sync the two.

Change-Id: Ie1c2733597f0d7408dd0b863b42412b479dfcd15
Signed-off-by: Alon Bar-Lev <alonbl at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InstallVdsCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/OVirtNodeUpgrade.java
2 files changed, 34 insertions(+), 39 deletions(-)

Approvals:
  Yair Zaslavsky: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie1c2733597f0d7408dd0b863b42412b479dfcd15
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list