Change in vdsm[master]: vdsm: Reduce the migration progress timeout

vfeenstr at redhat.com vfeenstr at redhat.com
Tue Jan 7 07:29:38 UTC 2014


Vinzenz Feenstra has posted comments on this change.

Change subject: vdsm: Reduce the migration progress timeout
......................................................................


Patch Set 8:

(1 comment)

....................................................
File lib/vdsm/config.py.in
Line 54:             'recognized by kvm/qemu if a coma separated list given then a '
Line 55:             'NIC per device will be created.'),
Line 56: 
Line 57:         ('migration_timeout', '300',
Line 58:             'Migration Source Host: Uses at least 1/2 of this value for VMs '
It's relevant to this patch because it was wrong before and I am modifying the other part of it. It's not clear over the list of patches what I am removing if I am leaving this wrong documentation in place.

And this crazy looking logic for whatever reason is what has been implemented, I am only documentation as it is right now.
However if you look at the needed by part of this patchset you'll see that there is a bunch more which in the end will change all of this documentation and I hope it's more clear in the end.
Line 59:             'with less than 2 GiB of RAM otherwise 1/4 of this value per GiB '
Line 60:             'RAM for calculating the delay before setting/increasing the '
Line 61:             'migration_downtime.'
Line 62:             'Migration Destination Host: Maximum time the destination waits '


-- 
To view, visit http://gerrit.ovirt.org/16602
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I8f314d70b8f32cfff58f9776bcc2182a748a9b67
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Lee Yarwood <lyarwood at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Peter V. Saveliev <peet at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list