draft

Signed-off-by: Andrea Panattoni <[email protected]>

test/conformance/tests/test_sriov_operator.go

@@ -1709,14 +1709,17 @@ func findMainSriovDevice(executorPod *corev1.Pod, sriovDevices []*sriovv1.Interf
 		stdout, _, err = pod.ExecCommand(clients, executorPod, "ip", "link", "show", device.Name)
 		Expect(err).ToNot(HaveOccurred())
 		Expect(len(stdout)).Should(Not(Equal(0)), "Unable to query link state")
+
 		if strings.Contains(stdout, "state DOWN") {
+			fmt.Printf("Debug: DOWN %+v\n", device.Name)
 			continue // The interface is not active
 		}
 		if strings.Contains(stdout, "master ovs-system") {
 			fmt.Println("Chosen ", device.Name, " as it is used by ovs")
 			return device
 		}
 	}
+
 	return nil
 }
 

test/conformance/tests/test_switchdev.go

@@ -2,6 +2,7 @@ package tests
 
 import (
 	"context"
+	"fmt"
 	"time"
 
 	sriovv1 "github.com/k8snetworkplumbingwg/sriov-network-operator/api/v1"
@@ -40,12 +41,15 @@ var _ = Describe("[sriov] Switchdev", Ordered, func() {
 		testNode, interfaces, err := sriovInfos.FindSriovDevicesAndNode()
 		Expect(err).ToNot(HaveOccurred())
 
-		By("Testing on node " + testNode)
+		By(fmt.Sprintf("Testing on node %s, %d devices found", testNode, len(interfaces)))
 
 		mainDevice := findMainSriovDevice(getConfigDaemonPod(testNode), interfaces)
-		Expect(mainDevice).ToNot(BeNil())
-
-		By("Skipping " + mainDevice.Name + " as it is the primary NIC")
+		if mainDevice == nil {
+			mainDevice = &sriovv1.InterfaceExt{}
+			By("No SR-IOV device is the primary NIC of the node")
+		} else {
+			By("Skipping " + mainDevice.Name + " as it is the primary NIC")
+		}
 
 		for _, intf := range interfaces {
 			if !doesInterfaceSupportSwitchdev(intf) {