| commit | bbd1af06394e910f44f7685fca821a7b68673fe0 | [log] [tgz] |
|---|---|---|
| author | garciadeblas <gerardo.garciadeblas@telefonica.com> | Tue Jun 03 13:12:17 2025 +0200 |
| committer | garciadeblas <gerardo.garciadeblas@telefonica.com> | Tue Jun 03 15:01:01 2025 +0200 |
| tree | dd6909b8aa1b29b0d4ae7e4bfc2eb2aa2c49a815 | |
| parent | a338bd77a150649b1933471384de1d8fafab7988 [diff] |
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 <gerardo.garciadeblas@telefonica.com>
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 a266ea7..5c61913 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 @@ 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 daf060a..2190a51 100644 --- a/installers/helm/osm/values.yaml +++ b/installers/helm/osm/values.yaml
@@ -662,3 +662,6 @@ # type: NodePort # nodePort: 9998 # port: 9998 + config: + airflowHost: "osm-webserver" + # airflowPort: "8080"