Martin Polednik has posted comments on this change. Change subject: hostdev: add test for SR-IOV VF ......................................................................
Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/43094/1/tests/hostdevTests.py File tests/hostdevTests.py: Line 410: def testCreateSRIOVVF(self): Line 411: dev_spec = {'type': 'hostdev', 'device': _SRIOV_VF, Line 412: 'specParams': Line 413: {'macAddr': 'ff:ff:ff:ff:ff:ff', 'vlanId': 3}} Line 414: device = hostdevice.HostDevice(self.conf, self.log, **dev_spec) > nit: Thanks for the note, interesting! Now, numvfs itself is tested in hostdev module tests, so I'm not sure that it is somewhat required - but since I don't want to touch FS consider it done. -- To view, visit https://gerrit.ovirt.org/43094 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1bf2e07f492508757be2691417c706219ded54af Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Martin Polednik <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Ido Barkan <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
