(rukpak) extend bundle renderer to accept config opts

Introduce BundleConfig that contains InstallConfig and DeploymentConfig.

Author: anik120

internal/operator-controller/rukpak/convert/helm.go

@@ -7,6 +7,7 @@ import (
 
 	"helm.sh/helm/v3/pkg/chart"
 
+	"github.com/operator-framework/operator-controller/internal/operator-controller/rukpak/bundle"
 	"github.com/operator-framework/operator-controller/internal/operator-controller/rukpak/bundle/source"
 	"github.com/operator-framework/operator-controller/internal/operator-controller/rukpak/render"
 )
@@ -18,11 +19,27 @@ type BundleToHelmChartConverter struct {
 }
 
 func (r *BundleToHelmChartConverter) ToHelmChart(bundle source.BundleSource, installNamespace string, watchNamespace string) (*chart.Chart, error) {
+	config := make(map[string]interface{})
+
+	if watchNamespace != "" {
+		config["watchNamespace"] = watchNamespace
+	}
+
 	rv1, err := bundle.GetBundle()
 	if err != nil {
 		return nil, err
 	}
 
+	if _, present := config["watchNamespace"]; !present {
+		config["watchNamespace"] = r.deriveDefaultWatchNamespace(&rv1, installNamespace)
+	}
+
+	// Derive target namespaces from watchNamespace
+	targetNamespaces, err := r.deriveTargetNamespaces(config)
+	if err != nil {
+		return nil, fmt.Errorf("error deriving target namespaces: %w", err)
+	}
+
 	if len(rv1.CSV.Spec.APIServiceDefinitions.Owned) > 0 {
 		return nil, fmt.Errorf("unsupported bundle: apiServiceDefintions are not supported")
 	}
@@ -41,7 +58,7 @@ func (r *BundleToHelmChartConverter) ToHelmChart(bundle source.BundleSource, ins
 
 	objs, err := r.BundleRenderer.Render(
 		rv1, installNamespace,
-		render.WithTargetNamespaces(watchNamespace),
+		render.WithTargetNamespaces(targetNamespaces...),
 		render.WithCertificateProvider(r.CertificateProvider),
 	)
 
@@ -77,3 +94,55 @@ func (r *BundleToHelmChartConverter) ToHelmChart(bundle source.BundleSource, ins
 
 	return chrt, nil
 }
+
+// deriveDefaultWatchNamespace determines the default watch namespace based on bundle's supported install modes
+func (r *BundleToHelmChartConverter) deriveDefaultWatchNamespace(rv1 *bundle.RegistryV1, installNamespace string) string {
+	// Check what install modes the bundle supports
+	supportedModes := make(map[string]bool)
+	for _, installMode := range rv1.CSV.Spec.InstallModes {
+		if installMode.Supported {
+			supportedModes[string(installMode.Type)] = true
+		}
+	}
+
+	// Priority order for defaults:
+	// 1. If supports AllNamespaces -> "" (AllNamespaces)
+	// 2. If supports OwnNamespace -> installNamespace (OwnNamespace)
+	// 3. If supports SingleNamespace -> installNamespace (treat as OwnNamespace since we can't know the target)
+	// 4. Fallback -> "" (AllNamespaces)
+
+	if supportedModes["AllNamespaces"] {
+		return "" // Empty string means AllNamespaces
+	}
+	if supportedModes["OwnNamespace"] {
+		return installNamespace // OwnNamespace
+	}
+	if supportedModes["SingleNamespace"] {
+		return installNamespace // Treat as OwnNamespace since we don't know the target namespace
+	}
+
+	// Fallback to AllNamespaces
+	return ""
+}
+
+// deriveTargetNamespaces converts config.watchNamespace to the target namespaces for the renderer
+func (r *BundleToHelmChartConverter) deriveTargetNamespaces(cfg map[string]interface{}) ([]string, error) {
+	watchNsRaw, exists := cfg["watchNamespace"]
+	if !exists {
+		// No configuration provided, default to AllNamespaces
+		return []string{""}, nil
+	}
+
+	watchNs, ok := watchNsRaw.(string)
+	if !ok {
+		return nil, fmt.Errorf("watchNamespace must be a string, got %T", watchNsRaw)
+	}
+
+	if watchNs == "" {
+		// Empty string means AllNamespaces
+		return []string{""}, nil
+	} else {
+		// Non-empty string means SingleNamespace (or OwnNamespace if it equals installNamespace)
+		return []string{watchNs}, nil
+	}
+}