Change in vdsm[master]: periodic: make VmDispatcher ignore TooManyTasks

fromani at redhat.com fromani at redhat.com
Tue Jun 30 10:36:26 UTC 2015


Francesco Romani has posted comments on this change.

Change subject: periodic: make VmDispatcher ignore TooManyTasks
......................................................................


Patch Set 9: Code-Review-1

(1 comment)

https://gerrit.ovirt.org/#/c/41657/9/vdsm/virt/periodic.py
File vdsm/virt/periodic.py:

Line 245:             except Exception:
Line 246:                 # we want to make sure to have VM UUID logged
Line 247:                 self._log.exception("while dispatching %s to VM '%s'",
Line 248:                                     self._create, vm_id)
Line 249: 
I still need the 'else' clause here.
Line 250:             try:
Line 251:                 self._executor.dispatch(op, self._timeout)
Line 252:             except executor.TooManyTasks:
Line 253:                 skipped.append(vm_id)


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iad2d186327f8607aaeb41ca132a9a8a0806869cf
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list