Update airflowHost in webhook translator to point to osm-webserver 11/15211/2
authorgarciadeblas <gerardo.garciadeblas@telefonica.com>
Tue, 3 Jun 2025 11:12:17 +0000 (13:12 +0200)
committergarciadeblas <gerardo.garciadeblas@telefonica.com>
Tue, 3 Jun 2025 13:01:01 +0000 (15:01 +0200)
This is a workaround put in place while Airflow helm chart is not
dealing properly with the value `fullnameOverride`, which is required
now that Airflow is a subchart of OSM Helm chart.

The issue in Airflow is reported in this link:
https://github.com/apache/airflow/issues/22622

Change-Id: I8e8a068a1386121c2d2196fdaed550fbd693d9d5
Signed-off-by: garciadeblas <gerardo.garciadeblas@telefonica.com>
installers/helm/osm/templates/webhook_translator/webhook-translator-configmap.yaml
installers/helm/osm/values.yaml

index a266ea7..5c61913 100644 (file)
@@ -22,6 +22,6 @@ metadata:
   labels:
     {{- include "osm.labels" . | nindent 4 }}
 data:
-  AIRFLOW_HOST: "airflow-webserver"
-  AIRFLOW_PORT: "8080"
-{{- end }}
\ No newline at end of file
+  AIRFLOW_HOST: {{ .Values.webhookTranslator.config.airflowHost | default "airflow-webserver" }}
+  AIRFLOW_PORT: {{ default "8080" .Values.webhookTranslator.config.airflowPort | quote }}
+{{- end }}
index daf060a..2190a51 100644 (file)
@@ -662,3 +662,6 @@ webhookTranslator:
   #   type: NodePort
   #   nodePort: 9998
   #   port: 9998
+  config:
+    airflowHost: "osm-webserver"
+    # airflowPort: "8080"