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 <alo...@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 <alo...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Barak Azulay <bazu...@redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsl...@redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to