From bbd1af06394e910f44f7685fca821a7b68673fe0 Mon Sep 17 00:00:00 2001 From: garciadeblas Date: Tue, 3 Jun 2025 13:12:17 +0200 Subject: [PATCH] Update airflowHost in webhook translator to point to osm-webserver 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 --- .../webhook_translator/webhook-translator-configmap.yaml | 6 +++--- installers/helm/osm/values.yaml | 3 +++ 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/installers/helm/osm/templates/webhook_translator/webhook-translator-configmap.yaml b/installers/helm/osm/templates/webhook_translator/webhook-translator-configmap.yaml index a266ea7d..5c619138 100644 --- a/installers/helm/osm/templates/webhook_translator/webhook-translator-configmap.yaml +++ b/installers/helm/osm/templates/webhook_translator/webhook-translator-configmap.yaml @@ -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 }} diff --git a/installers/helm/osm/values.yaml b/installers/helm/osm/values.yaml index daf060ad..2190a51d 100644 --- a/installers/helm/osm/values.yaml +++ b/installers/helm/osm/values.yaml @@ -662,3 +662,6 @@ webhookTranslator: # type: NodePort # nodePort: 9998 # port: 9998 + config: + airflowHost: "osm-webserver" + # airflowPort: "8080" -- 2.25.1