Francesco Romani has submitted this change and it was merged. Change subject: periodic: always re-schedule operations ......................................................................
periodic: always re-schedule operations when one periodic.Operation() try to dispatch itself to the executor, it could get TooManyTasks. In this case it should keep rescheduling itself. In general periodic.Operation should alwauys run until explicitely disabled, using their stop() method. Change-Id: Icb5836ca768c88316896e233d756a986e3111d8e Bug-Url: https://bugzilla.redhat.com/1364925 Backport-To: 4.0 Backport-To: 3.6 Signed-off-by: Francesco Romani <from...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/61685 Reviewed-by: Milan Zamazal <mzama...@redhat.com> Continuous-Integration: Jenkins CI Reviewed-by: Dan Kenigsberg <dan...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/61889 Reviewed-on: https://gerrit.ovirt.org/62401 --- M tests/periodicTests.py M vdsm/virt/periodic.py 2 files changed, 37 insertions(+), 3 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/62401 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icb5836ca768c88316896e233d756a986e3111d8e Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Milan Zamazal <mzama...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org
