Merge pull request #2279 from katiewasnothere/user/kabaldau/fix_parse…

…_devices Fix parse pod devices to not include invalid devices
microsoft · Oct 14, 2024 · 677a76a · 677a76a
2 parents e78ef44 + ffe8282
commit 677a76a
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/internal/oci/uvm.go b/internal/oci/uvm.go
@@ -231,12 +231,12 @@ func parseDevices(ctx context.Context, specWindows *specs.Windows) []uvm.VPCIDev
 	if specWindows == nil || specWindows.Devices == nil {
 		return nil
 	}
-	extraDevices := make([]uvm.VPCIDeviceID, len(specWindows.Devices))
-	for i, d := range specWindows.Devices {
+	extraDevices := []uvm.VPCIDeviceID{}
+	for _, d := range specWindows.Devices {
 		pciID, index := devices.GetDeviceInfoFromPath(d.ID)
 		if uvm.IsValidDeviceType(d.IDType) {
 			key := uvm.NewVPCIDeviceID(pciID, index)
-			extraDevices[i] = key
+			extraDevices = append(extraDevices, key)
 		} else {
 			log.G(ctx).WithFields(logrus.Fields{
 				"device": d,