On Thu, 2012-10-18 at 10:00 -0400, Chris Evich wrote:
> On 09/18/2012 03:19 AM, [email protected] wrote:
> > From: Satheesh Rajendran<[email protected]>
> >
> > This is version 2 patchset for testing the virsh nodecpustats command
> > and addresses the below github issue
> >
> > https://github.com/autotest/autotest/issues/524
> >
> > Satheesh Rajendran (4):
> >    client.base_utils.py: to add functions to get uptime and cpu stats
> >    tests.virt.virttest.virsh.py: to add virsh_nodecpustats function
> >    tests.virt.shared.cfg.subtests.cfg.sample: to add support for 
> > virsh_nodecpustats test
> >    tests.virt.libvirt.tests.virsh_nodecpustats.py: addition of new test case
> >
> >   client/base_utils.py                               |   32 +++
> >   .../tests/virt/libvirt/tests/virsh_nodecpustats.py |  214 
> > ++++++++++++++++++++
> >   client/tests/virt/shared/cfg/subtests.cfg.sample   |   22 ++
> >   client/tests/virt/virttest/virsh.py                |   12 +
> >   4 files changed, 280 insertions(+), 0 deletions(-)
> >   create mode 100644 client/tests/virt/libvirt/tests/virsh_nodecpustats.py
> >
> 
> Hi Satheesh,
> 
> Sorry it took so long to get to this.  I've rebased it for you (over the 
> repo. split) and only had to make a few minor changes:
> 
> * Renamed param. "itr" to "inner_test_iterations" - to make it more 
> clear what it's for when mixed with the billions of other parameters.
> 
> * Fixed some minor whitespace issues- (double-blanks after functions and 
> spaces on either side of assignment operators). 
> autotest/utils/run_pylint was updated recently to catch these things.
> 
> I've not pushed anything up yet because there's one minor administrative 
> issue we need to deal with (patch-sets which span multiple repos). 
> Lucas has some ideas, and once we figure it out, I'll take appropriate 
> action.
> 
Sure, Thanks.
> Thanks for the test addition, and thanks for your patients.
> 
Regards,
-Satheesh.


_______________________________________________
Autotest-kernel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/autotest-kernel

Reply via email to