Alon Bar-Lev has posted comments on this change.

Change subject: bootstrap: rewrite bootstrap using the new ovirt-host-deploy 
package
......................................................................


Patch Set 11: (3 inline comments)

....................................................
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsDeploy.java
Line 291:      */
Line 292:     private Callable _customizationDialog[] = new Callable[] {
Line 293:         new Callable<Object>() { public Object call() throws 
Exception {
Line 294:             if (
Line 295:                 (Boolean)_parser.cliEnvironmentGet(
Well, that's true, but currently I will settle with cast failure in log with 
that line.
Line 296:                     VdsmEnv.OVIRT_NODE
Line 297:                 )
Line 298:             ) {
Line 299:                 _messages.post(


Line 297:                 )
Line 298:             ) {
Line 299:                 _messages.post(
Line 300:                     InstallerMessages.Severity.INFO,
Line 301:                     "Host is hypervisor"
OK, I will print ovirt-node, I hope nobody wish to change this for branding.
Line 302:                 );
Line 303:                 _setNode();
Line 304:             }
Line 305:             return null;


Line 473:             if (_customizationShouldAbort) {
Line 474:                 _parser.cliAbort();
Line 475:             }
Line 476:             else {
Line 477:                 _customizationDialog[_customizationIndex++].call();
Does not crash, it will hang up the channel as expected, just like any 
communication exception we get.
Line 478:             }
Line 479:         }
Line 480:         catch (SoftError e) {
Line 481:             log.errorFormat(


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

Gerrit-MessageType: comment
Gerrit-Change-Id: If78c62601231f4729ca95da7653907b37856d672
Gerrit-PatchSet: 11
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: Doron Fediuck <dfedi...@redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsl...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@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