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(a)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(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>