Change in vdsm[master]: Fix the CPU quota MOM policy computations

rgolan at redhat.com rgolan at redhat.com
Tue Apr 7 14:27:43 UTC 2015


Roy Golan has posted comments on this change.

Change subject: Fix the CPU quota MOM policy computations
......................................................................


Patch Set 8:

(1 comment)

https://gerrit.ovirt.org/#/c/39411/8/tests/momPolicyTests.py
File tests/momPolicyTests.py:

Line 143:                          p.set_policy("04-cputune", cpu_policy_string))
Line 144: 
Line 145:         p.evaluate(host, [vm])
Line 146: 
Line 147:         self.assertEqual(vm.controls["vcpu_quota"], 1100)
> But it is. Multiply it by the number of logical CPUs and you get 10% of the
the cpu count is 1. I don't see where did you set 10 logical cpus


-- 
To view, visit https://gerrit.ovirt.org/39411
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia1f014d1b4058ef447a962cbda3336115e8610dc
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list