ShaoHe Feng has posted comments on this change. Change subject: add simple VM creation functional test ......................................................................
Patch Set 3: Looks good to me, but someone else must approve (1 inline comment) .................................................... File tests/functional/vmTests.py Line 59: Line 60: try: Line 61: assertVdsOK(self.s.create(conf)) Line 62: Line 63: timeout = 65 # seconds how do you get value(65) of the timeout？ Line 64: # wait for VM to come up until timeout Line 65: start = time.time() Line 66: vmUp = False Line 67: guestUp = False -- 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: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Zhou Zheng Sheng <[email protected]> Gerrit-Reviewer: Deepak C Shetty <[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: 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
