Moti Asayag has posted comments on this change. Change subject: core: throttle running of VMs (#843058) ......................................................................
Patch Set 1: (3 inline comments) few typos in commit message. .................................................... Commit Message Line 7: core: throttle running of VMs (#843058) Line 8: Line 9: https://bugzilla.redhat.com/show_bug.cgi?id=843058 Line 10: Line 11: Bulk running of VMs reguallary fail short in running all VMs. The reason is s/reguallary/regularly Line 12: The VdsSelector counts and reserve memory using Line 13: a) host commited memory: # of VMs * each VM static mem Line 14: b) pending VM count memory : # of VMs to run * thier memory count - a shared state variable Line 15: Line 13: a) host commited memory: # of VMs * each VM static mem Line 14: b) pending VM count memory : # of VMs to run * thier memory count - a shared state variable Line 15: Line 16: pending VM count is increased by the end of RUnVm and decreased by VdsUpdateRunTimeInfo Line 17: i.e two independent proccesses. s/proccesses/processes Line 18: Line 19: As long as the VdsUpdateRunTimeInfo don't run, theres an overflow in the calculation Line 20: of the free memory to run VM which causes a host to be falsely not selected to run the VM. Line 21: Line 27: Line 28: setup: Host with 8Gb mem, 3.0 pool, cluster, 10Vms Line 29: 2 Host with 4Gb mem, 3.1 pool, cluster 5VMs Line 30: Line 31: muliple run 10 VMs - prior to the fix the 9th and 10th can locate a host to run on s/muliple/multiple Line 32: after the fix all 10 VMs complete the run. In the 9th VM run the selector waits for the update to decrease memory Line 33: and continue after less than a second. Line 34: Line 35: migrate VMs back and forth of 5Vms with no issues. -- 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 <rgo...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Moti Asayag <masa...@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com> Gerrit-Reviewer: Roy Golan <rgo...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
