Change in vdsm[master]: Rename migration_timeout to migration_downtime_delay

danken at redhat.com danken at redhat.com
Wed Jan 8 16:48:29 UTC 2014


Dan Kenigsberg has submitted this change and it was merged.

Change subject: Rename migration_timeout to migration_downtime_delay
......................................................................


Rename migration_timeout to migration_downtime_delay

Previously the migration_timeout value was supposed to be used for
multiple things. Now the only meaning left is the calculation for
the delay between the downtime steps.
To reflect this, the value has been renamed and the code updated to make
this meaning more obvious.

Now it is 75 seconds per GiB of RAM the minimum time still is 150 seconds.

Change-Id: I6297b2714c37c98df6af50ed9799df2ef9c596c1
Signed-off-by: Vinzenz Feenstra <vfeenstr at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/22421
Reviewed-by: Michal Skrivanek <michal.skrivanek at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M lib/vdsm/config.py.in
M vdsm/vm.py
2 files changed, 12 insertions(+), 18 deletions(-)

Approvals:
  Vinzenz Feenstra: Verified
  Dan Kenigsberg: Looks good to me, approved
  Michal Skrivanek: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6297b2714c37c98df6af50ed9799df2ef9c596c1
Gerrit-PatchSet: 14
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek 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


More information about the vdsm-patches mailing list