Darshan N has posted comments on this change.

Change subject: gluster: Restart nrpe nervice during host deploy.
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.ovirt.org/#/c/28890/2/src/plugins/ovirt-host-deploy/gluster/nrpe.py
File src/plugins/ovirt-host-deploy/gluster/nrpe.py:

Line 136:     @plugin.event(
Line 137:         stage=plugin.Stages.STAGE_CLOSEUP,
Line 138:         condition=lambda self: (
Line 139:             self._modified
Line 140:         )
> this should be modified to:
Done
Line 141:     )
Line 142:     def _closeup(self):
Line 143:         if self.services.exists('nrpe'):
Line 144:             self.logger.info(_('Restarting nrpe service'))


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7d1164d97ce9b95de0107990733dc7ecb1fe3974
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-host-deploy
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnara...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Darshan N <dnara...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to