Change in vdsm[ovirt-3.3]: Introduce a maximum time limit a migration may take

ybronhei at redhat.com ybronhei at redhat.com
Wed Feb 26 11:51:13 UTC 2014


Yaniv Bronhaim has submitted this change and it was merged.

Change subject: Introduce a maximum time limit a migration may take
......................................................................


Introduce a maximum time limit a migration may take

There are cases where migrations can take too much time. To set a finite
end to this we're now introducing a new configuration value
migration_max_time_per_gib_mem which defined the maximum time
the migration may take per GiB memory. The default value of 64 seconds
is calculated by 50% of current default value for the maximum bandwidth.
(1 GiB needs 32 seconds on the maximum bandwidth)

When migration_max_time_per_gib_mem is set to `0` the functionality
is disabled.

This time tracking is now handled in the MigrationMonitorThread on the
source host side of the migration.

Change-Id: Ifd2f76b9334fcb7d2db24c081cccae15e8fd0b0c
Bug-Url: https://bugzilla.redhat.com/1069731
Signed-off-by: Vinzenz Feenstra <vfeenstr at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/21708
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-by: Tomáš Došek <tdosek at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/25022
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M lib/vdsm/config.py.in
M vdsm/vm.py
2 files changed, 34 insertions(+), 9 deletions(-)

Approvals:
  Yaniv Bronhaim: Looks good to me, approved
  Vinzenz Feenstra: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifd2f76b9334fcb7d2db24c081cccae15e8fd0b0c
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Tomáš Došek <tdosek at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list