Francesco Romani has posted comments on this change. Change subject: migration: Enable lazy setting of incoming/outgoing limits ......................................................................
Patch Set 25: (2 comments) https://gerrit.ovirt.org/#/c/53305/25//COMMIT_MSG Commit Message: PS25, Line 11: t is best time for the engine to update them not sure this is always true. Let's pretend Engine decreases the bound, so (most likely) the new migration will be queued on source - exactly because the bound limti was lowered. This works, but feels a bit awkward. Is really that what Engine wants? https://gerrit.ovirt.org/#/c/53305/25/vdsm/API.py File vdsm/API.py: PS25, Line 592: not incomingLimit I believe 'incomingLimit is None' is both a bit cheaper and clearer. -- To view, visit https://gerrit.ovirt.org/53305 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I79ab97f15788e4024c94d051e4aade713d760acf Gerrit-PatchSet: 25 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Martin Betak <mbe...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbe...@redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches