Nir Soffer has uploaded a new change for review. Change subject: executor: Remove unneeded exception ......................................................................
executor: Remove unneeded exception _ExecutorStopped is not needed now sine NotRunning has the same meaning. Change-Id: I41db76f77eabe1343a3b130e08d24e54ee1d710f Signed-off-by: Nir Soffer <[email protected]> --- M lib/vdsm/executor.py 1 file changed, 2 insertions(+), 6 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/42/30742/1 diff --git a/lib/vdsm/executor.py b/lib/vdsm/executor.py index f6ab7cf..9e2bb5f 100644 --- a/lib/vdsm/executor.py +++ b/lib/vdsm/executor.py @@ -115,7 +115,7 @@ self._log.debug('Worker %s started' % worker.name) try: self._loop() - except _ExecutorStopped: + except NotRunning: self._log.debug('Worker %s stopped' % worker.name) except _TaskDiscarded: self._log.debug('Worker %s was discarded' % worker.name) @@ -137,7 +137,7 @@ while self._running and self._tasks.empty: self._cond.wait() if not self._running: - raise _ExecutorStopped() + raise NotRunning() return self._tasks.get() @@ -209,10 +209,6 @@ def clear(self): self._tasks.clear() - - -class _ExecutorStopped(Exception): - pass class _TaskDiscarded(Exception): -- To view, visit http://gerrit.ovirt.org/30742 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I41db76f77eabe1343a3b130e08d24e54ee1d710f Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
