Francesco Romani has posted comments on this change. Change subject: migration: downtime: start thread after 1st iter ......................................................................
Patch Set 4: (1 comment) https://gerrit.ovirt.org/#/c/59220/4/vdsm/virt/migration.py File vdsm/virt/migration.py: Line 580: iterationCount) Line 581: if iterationCount == 1: Line 582: # it does not make sense to do any adjustments before Line 583: # first iteration. Line 584: self.downtime_thread.start() considering this branch is in deep stabilization mode, do we want a tunable to restore the old behaviour? Line 585: Line 586: lastDataRemaining = dataRemaining Line 587: Line 588: if (now - lastProgressTime) > progress_timeout: -- To view, visit https://gerrit.ovirt.org/59220 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I661c44d9dd7bdb9f1ec3a355038cfa8966946011 Gerrit-PatchSet: 4 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: Martin Polednik <mpoled...@redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com> Gerrit-Reviewer: Milan Zamazal <mzama...@redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjeli...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org