fix: correctly map resources with differrent ns in MapperSameID

The namespace should be taken from the output resource definition, not from the input metadata. Signed-off-by: Artem Chernyshev <[email protected]>
cosi-project · Dec 5, 2024 · 6f0a50c · 6f0a50c
1 parent f179603
commit 6f0a50c
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/pkg/controller/generic/qtransform/options.go b/pkg/controller/generic/qtransform/options.go
@@ -30,10 +30,10 @@ type MapperFuncGeneric[I generic.ResourceWithRD] func(context.Context, *zap.Logg
 func MapperSameID[I generic.ResourceWithRD, O generic.ResourceWithRD]() MapperFuncGeneric[I] {
 	var zeroOutput O
 
-	outputType := zeroOutput.ResourceDefinition().Type
+	outputRD := zeroOutput.ResourceDefinition()
 
 	return func(_ context.Context, _ *zap.Logger, _ controller.QRuntime, v I) ([]resource.Pointer, error) {
-		return []resource.Pointer{resource.NewMetadata(v.Metadata().Namespace(), outputType, v.Metadata().ID(), resource.VersionUndefined)}, nil
+		return []resource.Pointer{resource.NewMetadata(outputRD.DefaultNamespace, outputRD.Type, v.Metadata().ID(), resource.VersionUndefined)}, nil
 	}
 }