Dan Kenigsberg has posted comments on this change.

Change subject: add simple VM creation functional test
......................................................................


Patch Set 9: (1 inline comment)

....................................................
File tests/functional/xmlrpcTests.py
Line 34: 
Line 35: class XMLRPCTest(TestCaseBase):
Line 36:     UPSTATES = frozenset(('Up', 'Powering up', 'Running'))
Line 37: 
Line 38:     @testValidation.ValidateRunningAsRoot
I actually like to run this as non-root, which can be done if ssl=False. Could 
you think of a gentler skip?
Line 39:     def setUp(self):
Line 40:         self.s = vdscli.connect()
Line 41: 
Line 42:     def testGetCaps(self):


--
To view, visit http://gerrit.ovirt.org/7396
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Icb0d86ce20a547ef809d5407fe12d6ade474c4d2
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Zhou Zheng Sheng <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Deepak C Shetty <[email protected]>
Gerrit-Reviewer: Gal Hammer <[email protected]>
Gerrit-Reviewer: Mark Wu <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Royce Lv <[email protected]>
Gerrit-Reviewer: ShaoHe Feng <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Zhou Zheng Sheng <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to