Use options for NewLibraryLocator

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

internal/discover/graphics.go

@@ -49,10 +49,10 @@ func NewDRMNodesDiscoverer(logger logger.Interface, devices image.VisibleDevices
 
 // NewGraphicsMountsDiscoverer creates a discoverer for the mounts required by graphics tools such as vulkan.
 func NewGraphicsMountsDiscoverer(logger logger.Interface, driverRoot string, nvidiaCTKPath string) (Discover, error) {
-	locator, err := lookup.NewLibraryLocator(logger, driverRoot)
+	locator := lookup.NewLibraryLocator(
-	if err != nil {
+		lookup.WithLogger(logger),
-		return nil, fmt.Errorf("failed to construct library locator: %v", err)
+		lookup.WithRoot(driverRoot),
-	}
+	)
 	libraries := NewMounts(
 		logger,
 		locator,

internal/lookup/cuda/cuda.go

@@ -58,13 +58,10 @@ func New(opts ...Options) lookup.Locator {
 		c.driverRoot = "/"
 	}
 
-	// TODO: Do we want to set the Count to 1 here?
+	c.Locator = lookup.NewLibraryLocator(
-	l, _ := lookup.NewLibraryLocator(
+		lookup.WithLogger(c.logger),
-		c.logger,
+		lookup.WithRoot(c.driverRoot),
-		c.driverRoot,
 	)
-
-	c.Locator = l
 	return c
 }
 

internal/lookup/library.go

@@ -30,12 +30,9 @@ type ldcacheLocator struct {
 
 var _ Locator = (*ldcacheLocator)(nil)
 
-// NewLibraryLocator creates a library locator using the specified logger.
+// NewLibraryLocator creates a library locator using the specified options.
-func NewLibraryLocator(logger logger.Interface, root string) (Locator, error) {
+func NewLibraryLocator(opts ...Option) Locator {
-	// We construct a symlink locator for expected library locations.
+	opts = append(opts,
-	symlinkLocator := NewSymlinkLocator(
-		WithLogger(logger),
-		WithRoot(root),
 		WithSearchPaths([]string{
 			"/",
 			"/usr/lib64",
@@ -50,24 +47,28 @@ func NewLibraryLocator(logger logger.Interface, root string) (Locator, error) {
 			"/lib/aarch64-linux-gnu/nvidia/current",
 		}...),
 	)
+	// We construct a symlink locator for expected library locations.
+	symlinkLocator := NewSymlinkLocator(opts...)
 
 	l := First(
 		symlinkLocator,
-		newLdcacheLocator(logger, root),
+		newLdcacheLocator(opts...),
 	)
-	return l, nil
+	return l
 }
 
-func newLdcacheLocator(logger logger.Interface, root string) Locator {
+func newLdcacheLocator(opts ...Option) Locator {
-	cache, err := ldcache.New(logger, root)
+	b := newBuilder(opts...)
+
+	cache, err := ldcache.New(b.logger, b.root)
 	if err != nil {
 		// If we failed to open the LDCache, we default to a symlink locator.
-		logger.Warningf("Failed to load ldcache: %v", err)
+		b.logger.Warningf("Failed to load ldcache: %v", err)
 		return nil
 	}
 
 	return &ldcacheLocator{
-		logger: logger,
+		logger: b.logger,
 		cache:  cache,
 	}
 }

internal/lookup/library_test.go

@@ -43,7 +43,10 @@ func TestLDCacheLocator(t *testing.T) {
 	require.NoError(t, os.Symlink(versionLib, sonameLink))
 	require.NoError(t, os.Symlink(sonameLink, soLink))
 
-	lut := newLdcacheLocator(logger, testDir)
+	lut := newLdcacheLocator(
+		WithLogger(logger),
+		WithRoot(testDir),
+	)
 
 	testCases := []struct {
 		description   string
@@ -94,7 +97,6 @@ func TestLDCacheLocator(t *testing.T) {
 			require.EqualValues(t, tc.expected, cleanedCandidates)
 		})
 	}
-
 }
 
 func TestLibraryLocator(t *testing.T) {
@@ -125,8 +127,10 @@ func TestLibraryLocator(t *testing.T) {
 	require.NoError(t, os.Symlink(libTarget1, source1))
 	require.NoError(t, os.Symlink(source1, source2))
 
-	lut, err := NewLibraryLocator(logger, testDir)
+	lut := NewLibraryLocator(
-	require.NoError(t, err)
+		WithLogger(logger),
+		WithRoot(testDir),
+	)
 
 	testCases := []struct {
 		description   string