Fix duplicate specs for jit-cdi mode

Signed-off-by: Evan Lezar <[email protected]>

Author: elezar

internal/modifier/cdi.go

@@ -212,7 +212,7 @@ type cdiModeIdentifiers struct {
 func cdiModeIdentfiersFromDevices(devices ...string) *cdiModeIdentifiers {
 	perModeIdentifiers := make(map[string][]string)
 	perModeDeviceClass := map[string]string{"auto": automaticDeviceClass}
-	uniqueModes := []string{"auto"}
+	var uniqueModes []string
 	seen := make(map[string]bool)
 	for _, device := range devices {
 		mode, id := getModeIdentifier(device)

internal/modifier/cdi_test.go

@@ -181,7 +181,7 @@ func Test_cdiModeIdentfiersFromDevices(t *testing.T) {
 			description: "empty device list",
 			devices:     []string{},
 			expected: &cdiModeIdentifiers{
-				modes:             []string{"auto"},
+				modes:             nil,
 				idsByMode:         map[string][]string{},
 				deviceClassByMode: map[string]string{"auto": "gpu"},
 			},
@@ -208,7 +208,7 @@ func Test_cdiModeIdentfiersFromDevices(t *testing.T) {
 			description: "device with explicit mode",
 			devices:     []string{"mode=gds,id=foo"},
 			expected: &cdiModeIdentifiers{
-				modes:             []string{"auto", "gds"},
+				modes:             []string{"gds"},
 				idsByMode:         map[string][]string{"gds": {"foo"}},
 				deviceClassByMode: map[string]string{"auto": "gpu"},
 			},
@@ -231,15 +231,15 @@ func Test_cdiModeIdentfiersFromDevices(t *testing.T) {
 			devices:     []string{"mode=nvswitch"},
 			expected: &cdiModeIdentifiers{
 				modes:             []string{"nvswitch"},
-				idsByMode:         map[string][]string{"nvswitch": {}},
+				idsByMode:         map[string][]string{},
 				deviceClassByMode: map[string]string{"auto": "gpu"},
 			},
 		},
 		{
 			description: "duplicate modes",
 			devices:     []string{"mode=gds,id=x", "mode=gds,id=y", "mode=gds"},
 			expected: &cdiModeIdentifiers{
-				modes:             []string{"auto", "gds"},
+				modes:             []string{"gds"},
 				idsByMode:         map[string][]string{"gds": {"x", "y"}},
 				deviceClassByMode: map[string]string{"auto": "gpu"},
 			},