Alon Bar-Lev has posted comments on this change.

Change subject: hosted-engine: allow to deploy hosted-engine
......................................................................


Patch Set 18:

(1 comment)

https://gerrit.ovirt.org/#/c/38547/18/src/plugins/ovirt-host-deploy/hosted-engine/packages.py
File src/plugins/ovirt-host-deploy/hosted-engine/packages.py:

Line 91:         self.services.state(
Line 92:             name='ovirt-ha-agent',
Line 93:             state=False,
Line 94:         )
Line 95:         if self.environment[odeploycons.HostedEngineEnv.ENABLE]:
so this is the only issue left, if you not enabling this means we disable it, 
we should also invalidate the configuration, so manual start will not create 
undesired side effect.
Line 96:             self.services.state(
Line 97:                 name=service,
Line 98:                 state=True,
Line 99:             )


-- 
To view, visit https://gerrit.ovirt.org/38547
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia07992ccab2f745879c8d3d777e45b524bbdf6f8
Gerrit-PatchSet: 18
Gerrit-Project: ovirt-host-deploy
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Doron Fediuck <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Martin Sivák <[email protected]>
Gerrit-Reviewer: Roy Golan <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to