feat: External ConfigMap Support for Cross-Operator Integration.

Author: dmaniloff

api/v1alpha1/llamastackdistribution_types.go

@@ -263,6 +263,31 @@ spec:
                     x-kubernetes-validations:
                     - message: Only one of name or image can be specified
                       rule: '!(has(self.name) && has(self.image))'
+                  envFromExternalConfigMaps:
+                    description: EnvFromExternalConfigMaps defines external ConfigMaps
+                      to inject as environment variables
+                    items:
+                      description: ExternalConfigMapSpec defines external ConfigMaps
+                        to inject as environment variables
+                      properties:
+                        mapping:
+                          additionalProperties:
+                            type: string
+                          description: |-
+                            Mapping defines how ConfigMap keys map to environment variable names
+                            Key is the ConfigMap key, Value is the environment variable name
+                          type: object
+                        name:
+                          description: Name is the name of the ConfigMap
+                          type: string
+                        namespace:
+                          description: Namespace is the namespace of the ConfigMap
+                          type: string
+                      required:
+                      - name
+                      - namespace
+                      type: object
+                    type: array
                   podOverrides:
                     description: PodOverrides allows advanced pod-level customization.
                     properties:

api/v1alpha1/zz_generated.deepcopy.go

@@ -0,0 +1,41 @@
+apiVersion: llamastack.io/v1alpha1
+kind: LlamaStackDistribution
+metadata:
+  name: llamastackdistribution-with-external-configmaps
+spec:
+  replicas: 1
+  server:
+    containerSpec:
+      env:
+        - name: INFERENCE_MODEL
+          value: 'llama3.2:1b'
+        - name: OLLAMA_URL
+          value: 'http://ollama-server-service.ollama-dist.svc.cluster.local:11434'
+      name: llama-stack
+    distribution:
+      name: starter
+    storage:
+      size: "20Gi"
+      mountPath: "/home/lls/.lls"
+    # External ConfigMaps configuration - enables referencing Konflux-built images
+    # from the redhat-ods-applications namespace ConfigMaps
+    envFromExternalConfigMaps:
+      - name: trustyai-service-operator-config
+        namespace: redhat-ods-applications
+        mapping:
+          ragas-provider-image: RAGAS_PROVIDER_IMAGE
+          garak-provider-image: GARAK_PROVIDER_IMAGE
+---
+# Example of how the ConfigMap would look like in redhat-ods-applications namespace
+# This would be automatically created/managed by the trustyai-service-operator
+apiVersion: v1
+kind: ConfigMap
+metadata:
+  name: trustyai-service-operator-config
+  namespace: redhat-ods-applications
+data:
+  ragas-provider-image: "quay.io/trustyai/llama-stack-provider-ragas:v1.2.3-konflux-build-123"
+  garak-provider-image: "quay.io/trustyai/llama-stack-provider-trustyai-garak:v1.2.3-konflux-build-456"
+  # Other trustyai operator configuration values...
+  trustyaiServiceImage: "quay.io/trustyai/trustyai-service:latest"
+  trustyaiOperatorImage: "quay.io/trustyai/trustyai-service-operator:latest"

config/crd/bases/llamastack.io_llamastackdistributions.yaml

@@ -25,6 +25,8 @@ import (
 
 	llamav1alpha1 "github.com/llamastack/llama-stack-k8s-operator/api/v1alpha1"
 	corev1 "k8s.io/api/core/v1"
+	k8serrors "k8s.io/apimachinery/pkg/api/errors"
+	"k8s.io/apimachinery/pkg/types"
 	"k8s.io/apimachinery/pkg/util/intstr"
 	"k8s.io/utils/ptr"
 	"sigs.k8s.io/controller-runtime/pkg/log"
@@ -186,6 +188,13 @@ func configureContainerEnvironment(ctx context.Context, r *LlamaStackDistributio
 		}
 	}
 
+	// Add environment variables from external ConfigMaps
+	if err := addExternalConfigMapEnvVars(ctx, r, instance, container); err != nil {
+		// Log error but don't fail deployment - allows graceful degradation
+		logger := log.FromContext(ctx)
+		logger.Error(err, "Failed to add external ConfigMap environment variables")
+	}
+
 	// Finally, add the user provided env vars
 	container.Env = append(container.Env, instance.Spec.Server.ContainerSpec.Env...)
 }
@@ -662,3 +671,54 @@ func (r *LlamaStackDistributionReconciler) resolveImage(distribution llamav1alph
 		return "", errors.New("failed to validate distribution: either distribution.name or distribution.image must be set")
 	}
 }
+
+// addExternalConfigMapEnvVars adds environment variables from external ConfigMaps
+func addExternalConfigMapEnvVars(ctx context.Context, r *LlamaStackDistributionReconciler, instance *llamav1alpha1.LlamaStackDistribution, container *corev1.Container) error {
+	if len(instance.Spec.Server.EnvFromExternalConfigMaps) == 0 {
+		return nil
+	}
+
+	logger := log.FromContext(ctx)
+
+	for _, extConfigMap := range instance.Spec.Server.EnvFromExternalConfigMaps {
+		// Fetch the external ConfigMap
+		configMap := &corev1.ConfigMap{}
+		err := r.Get(ctx, types.NamespacedName{
+			Name:      extConfigMap.Name,
+			Namespace: extConfigMap.Namespace,
+		}, configMap)
+		
+		if err != nil {
+			if k8serrors.IsNotFound(err) {
+				logger.Info("External ConfigMap not found, skipping",
+					"configMapName", extConfigMap.Name,
+					"configMapNamespace", extConfigMap.Namespace)
+				continue
+			}
+			return fmt.Errorf("failed to get external ConfigMap %s/%s: %w", 
+				extConfigMap.Namespace, extConfigMap.Name, err)
+		}
+
+		// Process the mapping and add environment variables
+		for configMapKey, envVarName := range extConfigMap.Mapping {
+			if value, exists := configMap.Data[configMapKey]; exists {
+				container.Env = append(container.Env, corev1.EnvVar{
+					Name:  envVarName,
+					Value: value,
+				})
+				logger.V(1).Info("Added environment variable from external ConfigMap",
+					"configMapName", extConfigMap.Name,
+					"configMapNamespace", extConfigMap.Namespace,
+					"configMapKey", configMapKey,
+					"envVarName", envVarName)
+			} else {
+				logger.Info("ConfigMap key not found, skipping",
+					"configMapName", extConfigMap.Name,
+					"configMapNamespace", extConfigMap.Namespace,
+					"configMapKey", configMapKey)
+			}
+		}
+	}
+
+	return nil
+}