Alon Bar-Lev has posted comments on this change.

Change subject: gluster: starting glusterd after install
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File src/plugins/ovirt-host-deploy/gluster/packages.py
Line 81:         priority=plugin.Stages.PRIORITY_LOW
Line 82:     )
Line 83:     def _start(self):
Line 84:         self.logger.info(_('Starting gluster'))
Line 85:         self.services.state('glusterd', True)
But it is already installed on gluster, right?

We do not install it, only start, right?

If so, it should be in separate source.

And, it should only start if no reboot.

Refer to the way vdsmd is started.
Line 86: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I4f65ea6cbb54c844fb42ad1a14c61c438ac0865a
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-host-deploy
Gerrit-Branch: master
Gerrit-Owner: Aravinda VK <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Aravinda VK <[email protected]>
Gerrit-Reviewer: Sahina Bose <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to