Change in ovirt-engine[master]: bootstrap: handle bootstrap messages using string member

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: handle bootstrap messages using string member
......................................................................


bootstrap: handle bootstrap messages using string member

Current implementation holds the VdsInstaller object in context of
InstallVdsCommand only to extract the error message out of it.

VdsInstaller is going to be retired so we move the reference to local
scope and store the error message in string at state scope.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=875528
Change-Id: I928db05fa3ca7cd5941f30af8036d598847ea7d2
Signed-off-by: Alon Bar-Lev <alonbl at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ApproveVdsCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InstallVdsCommand.java
2 files changed, 15 insertions(+), 11 deletions(-)

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I928db05fa3ca7cd5941f30af8036d598847ea7d2
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