From 89b220ed73853c1d26d7fb167ea186a09241cf0a Mon Sep 17 00:00:00 2001 From: Colton Padden Date: Wed, 8 Dec 2021 10:40:27 -0500 Subject: [PATCH] Instructions and default app configuration should use generic Airflow URLs Updated the instructions in the README, and the example entries in the app-config.yml file to use `https://your.airflow.instance.com` instead of `localhost:8080`. Signed-off-by: Colton Padden --- app-config.yaml | 4 ++-- plugins/apache-airflow/README.md | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/app-config.yaml b/app-config.yaml index 72de2fe34c..6cd05f44e8 100644 --- a/app-config.yaml +++ b/app-config.yaml @@ -98,7 +98,7 @@ proxy: Authorization: ${ILERT_AUTH_HEADER} '/airflow': - target: http://localhost:8080/api/v1 + target: https://your.airflow.instance.com/api/v1 headers: Authorization: ${AIRFLOW_BASIC_AUTH_HEADER} @@ -444,4 +444,4 @@ azureDevOps: organization: my-company apacheAirflow: - baseUrl: http://localhost:8080 + baseUrl: https://your.airflow.instance.com diff --git a/plugins/apache-airflow/README.md b/plugins/apache-airflow/README.md index f3163ecd93..4bec028298 100644 --- a/plugins/apache-airflow/README.md +++ b/plugins/apache-airflow/README.md @@ -49,7 +49,7 @@ configuration: ```yaml proxy: '/airflow': - target: http://localhost:8080/api/v1 + target: https://your.airflow.instance.com/api/v1 headers: Authorization: ${AIRFLOW_BASIC_AUTH_HEADER} ```