Change in vdsm[ovirt-3.6]: periodic: make VmDispatcher ignore TooManyTasks

fromani at redhat.com fromani at redhat.com
Tue Nov 24 12:14:36 UTC 2015


Francesco Romani has submitted this change and it was merged.

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


periodic: make VmDispatcher ignore TooManyTasks

VmDispatcher is meant to make N independent operations
from one (VM bulk sampling), being N the number of active VMs.

To fulfil this claim, if one of the N independent calls fails with
TooManyTasks, we go on attempting running the rest of the calls, even
though it is highly likely that once one TooManyTasks error is
encountered, many subsequent dispatch() attempts would fail.

Backport-To: 3.6
Change-Id: Iad2d186327f8607aaeb41ca132a9a8a0806869cf
Signed-off-by: Francesco Romani <fromani at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/41657
Continuous-Integration: Jenkins CI
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/48409
Reviewed-by: Milan Zamazal <mzamazal at redhat.com>
---
M tests/periodicTests.py
M vdsm/virt/periodic.py
2 files changed, 44 insertions(+), 2 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Francesco Romani: Verified; Looks good to me, approved
  Milan Zamazal: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iad2d186327f8607aaeb41ca132a9a8a0806869cf
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Milan Zamazal <mzamazal at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list