Signed-off-by: Luyao Huang <lhuang(a)redhat.com&gt; --- utils/utils.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/utils/utils.py b/utils/utils.py index 954b2bf..a6e6965 100644 --- a/utils/utils.py +++ b/utils/utils.py @@ -906,9 +906,10 @@ def get_standard_deviation(cb1, cb2, opaque1, opaque2, number = 1000): """ D = 0 for i in range(number): - a = cb1(opaque1) + a1 = cb1(opaque1) b = cb2(opaque2) - D += (int(a) - int(b))**2 + a2 = cb1(opaque1) + D += ((int(a1) + int(a2))/2 - int(b))**2 return math.sqrt(D/number)

def param_to_tuple_nolength(paramlist):

Signed-off-by: Luyao Huang <lhuang(a)redhat.com&gt; --- cases/linux_domain.conf | 6 +++ repos/domain/cpu_status.py | 113 +++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 119 insertions(+) create mode 100644 repos/domain/cpu_status.py diff --git a/cases/linux_domain.conf b/cases/linux_domain.conf index 0a7d134..9f64226 100644 --- a/cases/linux_domain.conf +++ b/cases/linux_domain.conf @@ -269,6 +269,12 @@ virconn:connection_security_model guestname $defaultname +domain:virDomain_getCPUStats + guestname + $defaultname + conn + qemu:///system + domain:destroy guestname $defaultname diff --git a/repos/domain/cpu_status.py b/repos/domain/cpu_status.py new file mode 100644 index 0000000..6e511c0 --- /dev/null +++ b/repos/domain/cpu_status.py @@ -0,0 +1,113 @@ +#!/usr/bin/env python +import libvirt +from libvirt import libvirtError +from utils import utils + +required_params = ('guestname',) +optional_params = {'conn': 'qemu:///system'} + +ONLINE_CPU = '/sys/devices/system/cpu/online' +CGROUP_PERCPU = '/sys/fs/cgroup/cpuacct/machine.slice/machine-qemu\\x2d%s.scope/cpuacct.usage_percpu' +CGROUP_PERVCPU = '/sys/fs/cgroup/cpuacct/machine.slice/machine-qemu\\x2d%s.scope/vcpu%d/cpuacct.usage_percpu' +CGROUP_USAGE = '/sys/fs/cgroup/cpuacct/machine.slice/machine-qemu\\x2d%s.scope/cpuacct.usage' +CGROUP_STAT = '/sys/fs/cgroup/cpuacct/machine.slice/machine-qemu\\x2d%s.scope/cpuacct.stat' + +def getcputime(a): + return open(a[0]).read().split()[a[1]] + +def virtgetcputime(a): + return a[0].getCPUStats(0)[a[1]][a[2]] + +def getvcputime(a): + ret = 0 + for i in range(int(a[0])): + ret += int(open(CGROUP_PERVCPU % (a[1], i)).read().split()[a[2]]) + + return ret + +def virtgettotalcputime(a): + return a[0].getCPUStats(1)[0][a[1]] + +def virtgettotalcputime2(a): + return a[0].getCPUStats(1)[0][a[1]]/10000000 + +def cpu_status(params): + """ + test API for getCPUStats in class virDomain + """ + logger = params['logger'] + fail=0 + + cpu = utils.file_read(ONLINE_CPU) + logger.info("host online cpulist is %s" % cpu) + + cpu_tuple = utils.param_to_tuple_nolength(cpu) + if not cpu_tuple: + logger.info("error in function param_to_tuple_nolength") + return 1 + + try: + conn = libvirt.open(params['conn']) + + logger.info("get connection to libvirtd") + guest = params['guestname'] + vm = conn.lookupByName(guest) + vcpus = vm.info()[3] + for n in range(len(cpu_tuple)): + if not cpu_tuple[n]: + continue + + D = utils.get_standard_deviation(getcputime, virtgetcputime, \ + [CGROUP_PERCPU % guest, n], [vm,n,'cpu_time']) + logger.info("Standard Deviation for host cpu %d cputime is %d" % (n, D)) + + """ expectations 403423 is a average collected in a x86_64 low load machine""" + if D > 403423*5: + fail=1 + logger.info("FAIL: Standard Deviation is too big \ + (biger than %d) for host cpu %d" % (403423*5, n)) + + D = utils.get_standard_deviation(getvcputime, virtgetcputime, \ + [vcpus, guest, n], [vm,n,'vcpu_time']) + logger.info("Standard Deviation for host cpu %d vcputime is %d" % (n, D)) + + """ expectations 4034 is a average collected in a x86_64 low load machine""" + if D > 4034*5*vcpus: + fail=1 + logger.info("FAIL: Standard Deviation is too big \ + (biger than %d) for host cpu time %d" % (4034*5*vcpus, n)) + + D = utils.get_standard_deviation(getcputime, virtgettotalcputime, \ + [CGROUP_USAGE % guest, 0], [vm,'cpu_time']) + logger.info("Standard Deviation for host cpu total cputime is %d" % D) + + """ expectations 313451 is a average collected in a x86_64 low load machine""" + if D > 313451*5*len(cpu_tuple): + fail=1 + logger.info("FAIL: Standard Deviation is too big \ + (biger than %d) for host cpu time %d" % (313451*5*len(cpu_tuple), n)) + + D = utils.get_standard_deviation(getcputime, virtgettotalcputime2, \ + [CGROUP_STAT % guest, 3], [vm,'system_time']) + logger.info("Standard Deviation for host cpu total system time is %d" % D) + + """ expectations 10 is a average collected in a x86_64 low load machine""" + if D > 10*5: + fail=1 + logger.info("FAIL: Standard Deviation is too big \ + (biger than %d) for host system cpu time %d" % (10*5, n)) + + D = utils.get_standard_deviation(getcputime, virtgettotalcputime2, \ + [CGROUP_STAT % guest, 1], [vm,'user_time']) + logger.info("Standard Deviation for host cpu total user time is %d" % D) + + """ expectations 10 is a average collected in a x86_64 low load machine""" + if D > 10*5: + fail=1 + logger.info("FAIL: Standard Deviation is too big \ + (biger than %d) for host user cpu time %d" % (10*5, n)) + + except libvirtError, e: + logger.error("API error message: %s" % e.message) + fail=1 + return fail

On 06/02/2015 07:57 AM, hongming wrote: > ACK and pushed > Don't forget to remove trailing whitespace next time. > Thanks > > Hongming > On 05/18/2015 09:28 AM, Luyao Huang wrote: >> Signed-off-by: Luyao Huang <lhuang(a)redhat.com&gt; >> --- >> cases/linux_domain.conf | 6 +++ >> repos/domain/cpu_status.py | 113 >> +++++++++++++++++++++++++++++++++++++++++++++ >> 2 files changed, 119 insertions(+) >> create mode 100644 repos/domain/cpu_status.py >> >> diff --git a/cases/linux_domain.conf b/cases/linux_domain.conf >> index 0a7d134..9f64226 100644 >> --- a/cases/linux_domain.conf >> +++ b/cases/linux_domain.conf >> @@ -269,6 +269,12 @@ virconn:connection_security_model >> guestname >> $defaultname >> +domain:virDomain_getCPUStats The case name is wrong , I have fixed it directly commit 5c433b7c934cd8ac3ad783939c5906b7cbc129a8 Author: Hongming Zhang <honzhang(a)redhat.com&gt; Date: Mon Aug 24 11:25:04 2015 +0800 Fix a wrong case name in linux_domain.conf Modify the virDomain_getCPUStats as cpu_status modified: cases/linux_domain.conf

>> + guestname >> + $defaultname >> + conn >> + qemu:///system >> + >> domain:destroy >> guestname >> $defaultname >> diff --git a/repos/domain/cpu_status.py b/repos/domain/cpu_status.py >> new file mode 100644 >> index 0000000..6e511c0 >> --- /dev/null >> +++ b/repos/domain/cpu_status.py >> @@ -0,0 +1,113 @@ >> +#!/usr/bin/env python >> +import libvirt >> +from libvirt import libvirtError >> +from utils import utils >> + >> +required_params = ('guestname',) >> +optional_params = {'conn': 'qemu:///system'} >> + >> +ONLINE_CPU = '/sys/devices/system/cpu/online' >> +CGROUP_PERCPU = >> '/sys/fs/cgroup/cpuacct/machine.slice/machine-qemu\\x2d%s.scope/cpuacct.usage_percpu' >> +CGROUP_PERVCPU = >> '/sys/fs/cgroup/cpuacct/machine.slice/machine-qemu\\x2d%s.scope/vcpu%d/cpuacct.usage_percpu' >> +CGROUP_USAGE = >> '/sys/fs/cgroup/cpuacct/machine.slice/machine-qemu\\x2d%s.scope/cpuacct.usage' >> +CGROUP_STAT = >> '/sys/fs/cgroup/cpuacct/machine.slice/machine-qemu\\x2d%s.scope/cpuacct.stat' >> + >> +def getcputime(a): >> + return open(a[0]).read().split()[a[1]] >> + >> +def virtgetcputime(a): >> + return a[0].getCPUStats(0)[a[1]][a[2]] >> + >> +def getvcputime(a): >> + ret = 0 >> + for i in range(int(a[0])): >> + ret += int(open(CGROUP_PERVCPU % (a[1], >> i)).read().split()[a[2]]) >> + >> + return ret >> + >> +def virtgettotalcputime(a): >> + return a[0].getCPUStats(1)[0][a[1]] >> + >> +def virtgettotalcputime2(a): >> + return a[0].getCPUStats(1)[0][a[1]]/10000000 >> + >> +def cpu_status(params): >> + """ >> + test API for getCPUStats in class virDomain >> + """ >> + logger = params['logger'] >> + fail=0 >> + >> + cpu = utils.file_read(ONLINE_CPU) >> + logger.info("host online cpulist is %s" % cpu) >> + >> + cpu_tuple = utils.param_to_tuple_nolength(cpu) >> + if not cpu_tuple: >> + logger.info("error in function param_to_tuple_nolength") >> + return 1 >> + >> + try: >> + conn = libvirt.open(params['conn']) >> + >> + logger.info("get connection to libvirtd") >> + guest = params['guestname'] >> + vm = conn.lookupByName(guest) >> + vcpus = vm.info()[3] >> + for n in range(len(cpu_tuple)): >> + if not cpu_tuple[n]: >> + continue >> + >> + D = utils.get_standard_deviation(getcputime, >> virtgetcputime, \ >> + [CGROUP_PERCPU % guest, n], [vm,n,'cpu_time']) >> + logger.info("Standard Deviation for host cpu %d cputime >> is %d" % (n, D)) >> + >> + """ expectations 403423 is a average collected in a >> x86_64 low load machine""" >> + if D > 403423*5: >> + fail=1 >> + logger.info("FAIL: Standard Deviation is too big \ >> + (biger than %d) for host cpu %d" % >> (403423*5, n)) >> + >> + D = utils.get_standard_deviation(getvcputime, >> virtgetcputime, \ >> + [vcpus, guest, n], [vm,n,'vcpu_time']) >> + logger.info("Standard Deviation for host cpu %d >> vcputime is %d" % (n, D)) >> + >> + """ expectations 4034 is a average collected in a >> x86_64 low load machine""" >> + if D > 4034*5*vcpus: >> + fail=1 >> + logger.info("FAIL: Standard Deviation is too big \ >> + (biger than %d) for host cpu time %d" >> % (4034*5*vcpus, n)) >> + >> + D = utils.get_standard_deviation(getcputime, >> virtgettotalcputime, \ >> + [CGROUP_USAGE % guest, 0], [vm,'cpu_time']) >> + logger.info("Standard Deviation for host cpu total cputime >> is %d" % D) >> + >> + """ expectations 313451 is a average collected in a x86_64 >> low load machine""" >> + if D > 313451*5*len(cpu_tuple): >> + fail=1 >> + logger.info("FAIL: Standard Deviation is too big \ >> + (biger than %d) for host cpu time %d" % >> (313451*5*len(cpu_tuple), n)) >> + >> + D = utils.get_standard_deviation(getcputime, >> virtgettotalcputime2, \ >> + [CGROUP_STAT % guest, 3], [vm,'system_time']) >> + logger.info("Standard Deviation for host cpu total system >> time is %d" % D) >> + >> + """ expectations 10 is a average collected in a x86_64 low >> load machine""" >> + if D > 10*5: >> + fail=1 >> + logger.info("FAIL: Standard Deviation is too big \ >> + (biger than %d) for host system cpu time >> %d" % (10*5, n)) >> + >> + D = utils.get_standard_deviation(getcputime, >> virtgettotalcputime2, \ >> + [CGROUP_STAT % guest, 1], [vm,'user_time']) >> + logger.info("Standard Deviation for host cpu total user >> time is %d" % D) >> + >> + """ expectations 10 is a average collected in a x86_64 low >> load machine""" >> + if D > 10*5: >> + fail=1 >> + logger.info("FAIL: Standard Deviation is too big \ >> + (biger than %d) for host user cpu time %d" >> % (10*5, n)) >> + >> + except libvirtError, e: >> + logger.error("API error message: %s" % e.message) >> + fail=1 >> + return fail > > -- > libvir-list mailing list > libvir-list(a)redhat.com > https://www.redhat.com/mailman/listinfo/libvir-list