Stop corrupting resolver cache.

Signed-off-by: Ben Luddy <[email protected]>

Author: benluddy

pkg/controller/registry/resolver/resolver.go

@@ -159,13 +159,26 @@ func (r *SatResolver) SolveOperators(namespaces []string, csvs []*v1alpha1.Clust
 			errs = append(errs, err)
 			continue
 		}
+
+		// copy consumed fields to avoid directly mutating cachep
+		op = &cache.Entry{
+			Name:       op.Name,
+			Replaces:   op.Replaces,
+			BundlePath: op.BundlePath,
+			SourceInfo: &cache.OperatorSourceInfo{
+				Catalog:     op.SourceInfo.Catalog,
+				StartingCSV: op.SourceInfo.StartingCSV,
+			},
+			Properties: op.Properties,
+			Bundle:     op.Bundle,
+		}
 		if len(installableOperator.Replaces) > 0 {
-			op.Replaces = installableOperator.Replaces // TODO: Don't mutate object from cache!
+			op.Replaces = installableOperator.Replaces
 		}
 
 		// lookup if this installable came from a starting CSV
 		if _, ok := startingCSVs[csvName]; ok {
-			op.SourceInfo.StartingCSV = csvName // TODO: Don't mutate object from cache!
+			op.SourceInfo.StartingCSV = csvName
 		}
 
 		operators[csvName] = op