Milan Zamazal has posted comments on this change. Change subject: migrations: change convergence schedule from time to iterations ......................................................................
Patch Set 4: Code-Review+1 To clarify the last Tomáš's comment: We primarily care about small VMs. There is a good probability to catch at least some iterations with them so the algorithm should generally work on them. As of my concern about continuous but too small progress without reaching the desired amount of memory, it's more likely to happen with large VMs. But we don't care about them that much as their migrations are expected to take long time anyway. So this patch should correspond to our goals. (Please correct me if there is anything wrong above.) -- To view, visit https://gerrit.ovirt.org/56558 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6f87c954031842c35c99888c228a34ec7f19d800 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjeli...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <mskri...@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: No _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches