Nir Soffer has posted comments on this change. Change subject: jobs: Drop jobs.stop ......................................................................
Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/63937/1/lib/vdsm/jobs.py File lib/vdsm/jobs.py: Line 162 Line 163 Line 164 Line 165 Line 166 I would leave empty stop for consistency, so we can clean up the mess in vdsm, starting and stopping all modules using generic code, without special treatment for each one. -- To view, visit https://gerrit.ovirt.org/63937 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ifa1dc49472f429543a5335747c8e7a4b13bb3112 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/admin/lists/[email protected]
