Update internal/operator-controller/rukpak/render/render.go

perdasilva · camilamacedo86 · web-flow · commit 7355dbfc0e18 · 2025-10-23T11:18:46.000-07:00
Co-authored-by: Camila Macedo &lt;7708031+camilamacedo86@users.noreply.github.com&gt;
diff --git a/internal/operator-controller/rukpak/render/render.go b/internal/operator-controller/rukpak/render/render.go
@@ -159,7 +159,7 @@ func validateTargetNamespaces(rv1 *bundle.RegistryV1, installNamespace string, t
 		// message than just the generic (at least one target namespace must be specified) which would occur
 		// in case only the MultiNamespace install mode is supported by the bundle.
 		// If AllNamespaces mode is supported, the default will be [""] -> watch all namespaces
-		// If only OwnNamespace is supported, the default will be [install-namespace] -> only watch the install/own namespace
+		// If AllNamespaces is not supported, no default is applied and configuration must provide the target namespace(s)
 		if supportedInstallModes.Has(v1alpha1.InstallModeTypeMultiNamespace) {
 			return errors.New("at least one target namespace must be specified")
 		}

Original file line number	Diff line number	Diff line change
`@@ -159,7 +159,7 @@ func validateTargetNamespaces(rv1 *bundle.RegistryV1, installNamespace string, t`
`159`	`159`	`// message than just the generic (at least one target namespace must be specified) which would occur`
`160`	`160`	`// in case only the MultiNamespace install mode is supported by the bundle.`
`161`	`161`	`// If AllNamespaces mode is supported, the default will be [""] -> watch all namespaces`
`162`		`- // If only OwnNamespace is supported, the default will be [install-namespace] -> only watch the install/own namespace`
	`162`	`+ // If AllNamespaces is not supported, no default is applied and configuration must provide the target namespace(s)`
`163`	`163`	`if supportedInstallModes.Has(v1alpha1.InstallModeTypeMultiNamespace) {`
`164`	`164`	`return errors.New("at least one target namespace must be specified")`
`165`	`165`	`}`