Change in mom[master]: CpuTune - use previous value if quota or period is None
Adam Litke has submitted this change and it was merged.
Change subject: CpuTune - use previous value if quota or period is None
......................................................................
CpuTune - use previous value if quota or period is None
The policy may not set the value of quota or period if it detected no
change. the controller than isn't performing the call to setVmCpuTune.
The fix will make sure to call setVmCpuTune whenever one of the params
changed and it isn't null.
Bug-Url: https://bugzilla.redhat.com/1143992
Change-Id: I6ee063062b3596dde0e16d78900688d5dfb059bd
Signed-off-by: Roy Golan <rgolan(a)redhat.com>
Reviewed-on: http://gerrit.ovirt.org/35329
Reviewed-by: Martin Sivák <msivak(a)redhat.com>
Reviewed-by: Doron Fediuck <dfediuck(a)redhat.com>
Reviewed-by: Adam Litke <alitke(a)redhat.com>
---
M mom/Controllers/CpuTune.py
1 file changed, 9 insertions(+), 3 deletions(-)
Approvals:
Adam Litke: Looks good to me, approved
Martin Sivák: Looks good to me, but someone else must approve
Doron Fediuck: Looks good to me, but someone else must approve
Roy Golan: Verified
--
To view, visit http://gerrit.ovirt.org/35329
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6ee063062b3596dde0e16d78900688d5dfb059bd
Gerrit-PatchSet: 5
Gerrit-Project: mom
Gerrit-Branch: master
Gerrit-Owner: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org