Change in ovirt-engine[master]: bootstrap: split the logic of ovirt-node upgrade out

yzaslavs at redhat.com yzaslavs at redhat.com
Mon Nov 19 11:58:35 UTC 2012


Yair Zaslavsky has submitted this change and it was merged.

Change subject: bootstrap: split the logic of ovirt-node upgrade out
......................................................................


bootstrap: split the logic of ovirt-node upgrade out

Currently we have 4 separate bootstrap methods:
 1. Host.
 2. Node registration.
 3. Node installation.
 4. Node upgrade.

While there is something common between (1-3), node upgrade is totally
different sequence:

 1. Unlike bootstrap it is executing foreign code (not originated at
    engine machine.
 2. Unlike bootstrap it is non-customizable.

This change split the logic of the node upgrade out, to ease future
VdsInstaller retirement.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=875528
Change-Id: I22dee8f1238de198abba72f576fb4ee82c513f34
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
1 file changed, 48 insertions(+), 5 deletions(-)

Approvals:
  Alon Bar-Lev: Verified
  Yair Zaslavsky: Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I22dee8f1238de198abba72f576fb4ee82c513f34
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list