bug: inherited defaults not respected in cache BuilderWithOptions

When using BuilderWithOptions the inherited (manager/cluster) options
are lost. If they were provided, they are stronger than the cache package defaults.
(Overrides default dynamic mapper in my case)

Signed-off-by: Alex Kalenyuk <[email protected]>

Author: akalenyu

pkg/cache/cache.go

@@ -199,6 +199,7 @@ func BuilderWithOptions(options Options) NewCacheFunc {
 		if err != nil {
 			return nil, err
 		}
+		options = defaultToInheritedOpts(options, inherited)
 		options, err = defaultOpts(config, options)
 		if err != nil {
 			return nil, err
@@ -424,6 +425,21 @@ func defaultOpts(config *rest.Config, opts Options) (Options, error) {
 	return opts, nil
 }
 
+func defaultToInheritedOpts(opts, inherited Options) Options {
+	if opts.Scheme == nil {
+		opts.Scheme = inherited.Scheme
+	}
+
+	if opts.Mapper == nil {
+		opts.Mapper = inherited.Mapper
+	}
+
+	if opts.Resync == nil {
+		opts.Resync = inherited.Resync
+	}
+	return opts
+}
+
 func convertToByGVK[T any](byObject map[client.Object]T, def T, scheme *runtime.Scheme) (map[schema.GroupVersionKind]T, error) {
 	byGVK := map[schema.GroupVersionKind]T{}
 	for object, value := range byObject {