refactor(vision): use one shared flags configmap

Author: a-klos

ADR-vision-embedding-rollout.md

@@ -17,7 +17,7 @@ Roll out vision support behind explicit feature flags that default to `false`:
 - `VISION_EMBEDDING_ENABLED=false`
 - `VISION_IMAGE_RETRIEVER_ENABLED=false`
 
-Flags are exposed via Helm values and config maps for backend, admin-backend, and extractor.
+Flags are exposed via Helm values and a shared config map consumed by backend, admin-backend, and extractor.
 
 ## Consequences
 

infrastructure/rag/templates/_admin_backend_and_extractor_helpers.tpl

@@ -140,18 +140,10 @@
 {{- printf "%s-source-uploader-configmap" .Release.Name | trunc 63 | trimSuffix "-" -}}
 {{- end -}}
 
-{{- define "configmap.adminVisionName" -}}
-{{- printf "%s-admin-vision-configmap" .Release.Name | trunc 63 | trimSuffix "-" -}}
-{{- end -}}
-
 {{- define "configmap.extractorSitemapName" -}}
 {{- printf "%s-extractor-sitemap-configmap" .Release.Name | trunc 63 | trimSuffix "-" -}}
 {{- end -}}
 
-{{- define "configmap.extractorVisionName" -}}
-{{- printf "%s-extractor-vision-configmap" .Release.Name | trunc 63 | trimSuffix "-" -}}
-{{- end -}}
-
 # image
 {{- define "adminBackend.fullImageName" -}}
 {{- $tag := default .Chart.AppVersion .Values.adminBackend.image.tag -}}

infrastructure/rag/templates/_backend_helpers.tpl

@@ -103,8 +103,8 @@ basic-auth
 {{- printf "%s-retriever-configmap" .Release.Name | trunc 63 | trimSuffix "-" -}}
 {{- end -}}
 
-{{- define "configmap.backendVisionName" -}}
-{{- printf "%s-backend-vision-configmap" .Release.Name | trunc 63 | trimSuffix "-" -}}
+{{- define "configmap.visionName" -}}
+{{- printf "%s-vision-configmap" .Release.Name | trunc 63 | trimSuffix "-" -}}
 {{- end -}}
 
 {{- define "configmap.langfuseName" -}}

infrastructure/rag/templates/admin-backend/configmap.yaml

@@ -42,12 +42,3 @@ data:
   {{- range $key, $value := .Values.adminBackend.envs.sourceUploader }}
   {{ $key }}: {{ $value | quote }}
   {{- end }}
----
-apiVersion: v1
-kind: ConfigMap
-metadata:
-  name: {{ template "configmap.adminVisionName" . }}
-data:
-  {{- range $key, $value := .Values.adminBackend.envs.vision }}
-  {{ $key }}: {{ $value | quote }}
-  {{- end }}

infrastructure/rag/templates/admin-backend/deployment.yaml

@@ -118,7 +118,7 @@ spec:
           - configMapRef:
               name: {{ template "configmap.sourceUploaderName" . }}
           - configMapRef:
-              name: {{ template "configmap.adminVisionName" . }}
+              name: {{ template "configmap.visionName" . }}
           - configMapRef:
               name: {{ template "configmap.retryDecoratorName" . }}
           - configMapRef:

infrastructure/rag/templates/backend/configmap.yaml

@@ -19,9 +19,9 @@ data:
 apiVersion: v1
 kind: ConfigMap
 metadata:
-  name: {{ template "configmap.backendVisionName" . }}
+  name: {{ template "configmap.visionName" . }}
 data:
-  {{- range $key, $value := .Values.backend.envs.vision }}
+  {{- range $key, $value := .Values.shared.envs.vision }}
   {{ $key }}: {{ $value | quote }}
   {{- end }}
 ---

infrastructure/rag/templates/backend/deployment.yaml

@@ -110,7 +110,7 @@ spec:
           - configMapRef:
               name: {{ template "configmap.retrieverName" . }}
           - configMapRef:
-              name: {{ template "configmap.backendVisionName" . }}
+              name: {{ template "configmap.visionName" . }}
           - configMapRef:
               name: {{ template "configmap.rerankerName" . }}
           - configMapRef:

infrastructure/rag/templates/extractor/configmap.yaml

@@ -6,12 +6,3 @@ data:
   {{- range $key, $value := .Values.extractor.envs.sitemap }}
   {{ $key }}: {{ $value | quote }}
   {{- end }}
----
-apiVersion: v1
-kind: ConfigMap
-metadata:
-  name: {{ template "configmap.extractorVisionName" . }}
-data:
-  {{- range $key, $value := .Values.extractor.envs.vision }}
-  {{ $key }}: {{ $value | quote }}
-  {{- end }}

infrastructure/rag/templates/extractor/deployment.yaml

@@ -113,7 +113,7 @@ spec:
           - configMapRef:
               name: {{ template "configmap.extractorSitemapName" . }}
           - configMapRef:
-              name: {{ template "configmap.extractorVisionName" . }}
+              name: {{ template "configmap.visionName" . }}
           - secretRef:
               name: {{ template "secret.s3RefName" . }}
         {{- $hfCacheDir := include "extractor.huggingfaceCacheDir" . }}

infrastructure/rag/values.yaml

@@ -196,10 +196,6 @@ backend:
       RETRIEVER_TABLE_K_DOCUMENTS: 10
       RETRIEVER_IMAGE_THRESHOLD: 0.7
       RETRIEVER_IMAGE_K_DOCUMENTS: 10
-    vision:
-      VISION_IMAGE_LANE_ENABLED: false
-      VISION_EMBEDDING_ENABLED: false
-      VISION_IMAGE_RETRIEVER_ENABLED: false
     errorMessages:
       ERROR_MESSAGES_NO_DOCUMENTS_MESSAGE: "I'm sorry, my responses are limited. You must ask the right questions."
       ERROR_MESSAGES_NO_OR_EMPTY_COLLECTION: "No documents were provided for searching."
@@ -394,10 +390,6 @@ adminBackend:
     sourceUploader:
       # Large sitemap ingestions (per-page summaries) can take > 1 hour.
       SOURCE_UPLOADER_TIMEOUT: 3600
-    vision:
-      VISION_IMAGE_LANE_ENABLED: false
-      VISION_EMBEDDING_ENABLED: false
-      VISION_IMAGE_RETRIEVER_ENABLED: false
 
 extractor:
   replicaCount: 1
@@ -456,10 +448,6 @@ extractor:
       # Options: "docusaurus" (default), "astro", "generic"
       # Note: https://docs.stackit.cloud is built with Astro/Starlight -> use "astro".
       SITEMAP_PARSER: docusaurus
-    vision:
-      VISION_IMAGE_LANE_ENABLED: false
-      VISION_EMBEDDING_ENABLED: false
-      VISION_IMAGE_RETRIEVER_ENABLED: false
 
 adminFrontend:
   name: admin-frontend
@@ -551,6 +539,10 @@ shared:
     s3:
       S3_ENDPOINT: http://rag-minio:9000
       S3_BUCKET: documents
+    vision:
+      VISION_IMAGE_LANE_ENABLED: false
+      VISION_EMBEDDING_ENABLED: false
+      VISION_IMAGE_RETRIEVER_ENABLED: false
     retryDecorator:
       RETRY_DECORATOR_MAX_RETRIES: "5"
       RETRY_DECORATOR_RETRY_BASE_DELAY: "0.5"