Only set VF GUID and MAC for when VFs are created

Changing other VF driver type or other parameter will no longer trigger setting GUID and MAC
k8snetworkplumbingwg · Feb 8, 2022 · cf74fae · cf74fae
1 parent cc777c4
commit cf74fae
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/pkg/utils/utils.go b/pkg/utils/utils.go
@@ -287,12 +287,15 @@ func configSriovDevice(iface *sriovnetworkv1.Interface, ifaceStatus *sriovnetwor
 					break
 				}
 			}
-			if strings.EqualFold(iface.LinkType, "IB") {
-				if err = setVfGuid(addr, pfLink); err != nil {
+			if iface.NumVfs != ifaceStatus.NumVfs {
+				// only set VF GUID and MAC for when VFs are created
+				if strings.EqualFold(iface.LinkType, "IB") {
+					if err = setVfGuid(addr, pfLink); err != nil {
+						return err
+					}
+				} else if err = setVfAdminMac(addr, pfLink); err != nil {
 					return err
 				}
-			} else if err = setVfAdminMac(addr, pfLink); err != nil {
-				return err
 			}
 			if err = unbindDriverIfNeeded(addr, isRdma); err != nil {
 				return err