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 <rgo...@redhat.com>
Reviewed-on: http://gerrit.ovirt.org/35329
Reviewed-by: Martin Sivák <msi...@redhat.com>
Reviewed-by: Doron Fediuck <dfedi...@redhat.com>
Reviewed-by: Adam Litke <ali...@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 <rgo...@redhat.com>
Gerrit-Reviewer: Adam Litke <ali...@redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfedi...@redhat.com>
Gerrit-Reviewer: Martin Sivák <msi...@redhat.com>
Gerrit-Reviewer: Roy Golan <rgo...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to