Omer Frenkel has posted comments on this change. Change subject: core: throttle running of VMs (#843058) ......................................................................
Patch Set 1: Looks good to me, but someone else must approve looks like a very nice solution, 2 things: 1. please verify this on as many environments as you can (one host, multi host, multi vms (with os installed)...) just to make sure there are no side effects for this. 2. the change in vdsManager should be in a separate patch as its not related to this patch, and this patch contains important changes so better as clean as possible. -- To view, visit http://gerrit.ovirt.org/7204 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I076ede6cba919bc61f7546d7b29ef436eb6d3375 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
