Change in vdsm[master]: Adding utility methods and conf for CPU limit MOM integration
msivak at redhat.com
msivak at redhat.com
Fri Jun 13 08:18:49 UTC 2014
Martin Sivák has posted comments on this change.
Change subject: Adding utility methods and conf for CPU limit MOM integration
......................................................................
Patch Set 57:
(3 comments)
http://gerrit.ovirt.org/#/c/27258/57/tests/vmTests.py
File tests/vmTests.py:
Line 53:
Line 54: def listAllNetworks(self, *args):
Line 55: return []
Line 56:
Line 57: def lookupByUUIDString(self, id):
> is this ever used?
It seems to be used only outside tests. I removed it.
Line 58: pass
Line 59:
Line 60:
Line 61: class FakeDomain:
http://gerrit.ovirt.org/#/c/27258/57/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 91: # The Metadata URI for VM tunables
Line 92: METADATA_VM_TUNE_URI = 'http://ovirt.org/vm/tune/1.0'
Line 93:
Line 94: # A libvirt constant for undefined cpu quota
Line 95: NO_CPU_QUOTA = 0
> Does anybody need these values out of the vm module? If not, mark them as p
The FakeDomain uses it in the mock method.
Line 96:
Line 97: # A libvirt constant for undefined cpu period
Line 98: NO_CPU_PERIOD = 0
Line 99:
Line 98: NO_CPU_PERIOD = 0
Line 99:
Line 100: # CPU tune sampling window
Line 101: # minimum supported value is 2
Line 102: CPU_TUNE_SAMPLING_WINDOW = 2
> this constant can easily hide within the function that uses it.
Done
Line 103:
Line 104:
Line 105: def isVdsmImage(drive):
Line 106: """
--
To view, visit http://gerrit.ovirt.org/27258
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic502d9a4a976cd76bb6042bbb51f6cd281199631
Gerrit-PatchSet: 57
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Kobi Ianko <kobi at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: David Caro <dcaroest at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Kobi Ianko <kobi at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak 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