Change in vdsm[master]: migrations: change convergence schedule from time to iterations

fromani at redhat.com fromani at redhat.com
Thu May 5 08:17:19 UTC 2016


Francesco Romani has posted comments on this change.

Change subject: migrations: change convergence schedule from time to iterations
......................................................................


Patch Set 4: Code-Review+2

The worst issue I'm aware of is indeed non-converging migrations, not slow migrations.
The current algorhitm is indeed deceptive: seems to be right, but takes in account the wrong metric (time, not iterations), thus does not actually do the proper thing.
Now we are heading in the right direction, waiting for libvirt 1.3 that will really allow us to fix thing.
Last but not least, this is a needed step for convergence schedule.
So I think we need this in 4.0. +2.

PS: besides all above, this code is starting to loudly beg for a general cleanup. But this is a different, old, story...

-- 
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 <tjelinek at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <mskrivan at redhat.com>
Gerrit-Reviewer: Milan Zamazal <mzamazal at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: No


More information about the vdsm-patches mailing list