Francesco Romani has posted comments on this change.
Change subject: tests: add simple balloon functional testcase ......................................................................
Patch Set 15:
(1 comment)
http://gerrit.ovirt.org/#/c/12820/15/tests/functional/vmQoSTests.py File tests/functional/vmQoSTests.py:
Line 65: Line 66: with RunningVm(self.vdsm, customization) as vm: Line 67: self._waitForStartup(vm, VM_MINIMAL_UPTIME) Line 68: with self._balloonPolicy(policy): Line 69: time.sleep(12) # MOM policy engine wake up every 10s
I wouldn't gate this patch, but we really need to expand the VirtTests fram
That's right. I'll work on a followup patch to address this. Line 70: status, msg, stats = self.vdsm.getVmStats(vm) Line 71: self.assertEqual(status, SUCCESS, msg) Line 72: self.assertEqual(stats['balloonInfo']['balloon_cur'], 0) Line 73: