Currently SRIOV detection logic is reporting error if it fails to detect SRIOV device

Deepthi Dharwar · Deepthi Dharwar · commit a4b59a5d7c1d · 2020-03-02T19:31:37.000+05:30
on the system. This patch aims to fix the same.

Signed-off-by: Deepthi Dharwar &lt;ddharwar@redhat.com&gt;
diff --git a/test/e2e_node/topology_manager_test.go b/test/e2e_node/topology_manager_test.go
@@ -170,9 +170,8 @@ func findNUMANodeWithoutSRIOVDevicesFromSysfs(numaNodes int) (int, bool) {
 	}
 
 	if len(pciPerNuma) == 0 {
-		// if we got this far we already passed a rough check that SRIOV devices
-		// are available in the box, so something is seriously wrong
-		framework.Failf("failed to find any VF devices from %v", pciDevs)
+		framework.Logf("failed to find any VF device from %v", pciDevs)
+		return -1, false
 	}
 
 	for nodeNum := 0; nodeNum < numaNodes; nodeNum++ {

Original file line number	Diff line number	Diff line change
`@@ -170,9 +170,8 @@ func findNUMANodeWithoutSRIOVDevicesFromSysfs(numaNodes int) (int, bool) {`
`170`	`170`	`}`
`171`	`171`
`172`	`172`	`if len(pciPerNuma) == 0 {`
`173`		`- // if we got this far we already passed a rough check that SRIOV devices`
`174`		`- // are available in the box, so something is seriously wrong`
`175`		`- framework.Failf("failed to find any VF devices from %v", pciDevs)`
	`173`	`+ framework.Logf("failed to find any VF device from %v", pciDevs)`
	`174`	`+ return -1, false`
`176`	`175`	`}`
`177`	`176`
`178`	`177`	`for nodeNum := 0; nodeNum < numaNodes; nodeNum++ {`