Michal Skrivanek has posted comments on this change.
Change subject: tests: validate cdrom path sent to/received from vdsm
......................................................................
Patch Set 1:
(2 comments)
....................................................
File tests/functional/virtTests.py
Line 271: deviceDef['hotplugDisk']),
timeout=10)
Line 272: self.retryAssert(partial(self.vdsm.hotunplugDisk,
Line 273: deviceDef['hotplugDisk']),
timeout=10)
Line 274:
Line 275: @permutations([['self'], ['specParams'], ['both']])
if this is creating the VM 3 times…isn't it enough to have 1 VM and do changeCD?
Line 276: def testVmWithCdrom(self, pathLocation):
Line 277: fd, path = tempfile.mkstemp()
Line 278: os.chmod(path, 0o644)
Line 279:
Line 289: else:
Line 290: # we can use the same path to avoid having to generate
another
Line 291: # tmpfile
Line 292: cdrom['path'] = path
Line 293: cdrom['specParams']['path'] = path
how can we tell which one was used then? We can just make the second up (the
one we don't expect to be taken into account)
Line 294:
Line 295: customization = {'vmId':
'77777777-ffff-3333-bbbb-222222222222',
Line 296: 'vmName': 'testVm', 'devices': [cdrom]}
Line 297:
--
To view, visit http://gerrit.ovirt.org/22194
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I620c481056f7ee3b4d3270ba1f4bc45cbb132ebb
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpole...@redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegu...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Martin Polednik <mpole...@redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeen...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
