fix: Use driver.Root instead of config option in modifier Factory

Signed-off-by: Evan Lezar <elezar@nvidia.com>

Author: elezar

internal/modifier/csv_test.go

@@ -117,8 +117,6 @@ func TestNewCSVModifier(t *testing.T) {
 				devRoot = filepath.Join(lookupRoot, tc.devRootfs)
 			}
 			driver := root.New(root.WithDriverRoot(driverRoot), root.WithDevRoot(devRoot))
-			// TODO: We should point to the factory root here.
-			tc.cfg.NVIDIAContainerCLIConfig.Root = driverRoot
 			// Override the CSV file search path for this root.
 			tc.cfg.NVIDIAContainerRuntimeConfig.Modes.CSV.MountSpecPath = filepath.Join(driverRoot, "/etc/nvidia-container-runtime/host-files-for-container.d")
 			tc.cfg.NVIDIACTKConfig.Path = "/usr/bin/nvidia-cdi-hook"

internal/modifier/factory.go

@@ -62,7 +62,6 @@ func createFactory(opts ...Option) *Factory {
 	for _, opt := range opts {
 		opt(f)
 	}
-
 	if f.editsFactory == nil {
 		f.editsFactory = edits.NewFactory(edits.WithLogger(f.logger))
 	}
@@ -71,6 +70,9 @@ func createFactory(opts ...Option) *Factory {
 }
 
 func (f *Factory) validate() error {
+	if f.driver == nil {
+		return fmt.Errorf("a driver must be specified")
+	}
 	switch string(f.runtimeMode) {
 	case "":
 		return fmt.Errorf("a mode must be specified")