This is an automated email from the ASF dual-hosted git repository.
rahulvats pushed a commit to branch v3-1-test
in repository https://gitbox.apache.org/repos/asf/airflow.git
commit 8cb7e13dd8f3786a2c2648f3117fe69c851cb7c0
Author: github-actions[bot]
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Tue Mar 10 10:44:29 2026 +0100
[v3-1-test] Fix undefined variable in install_from_external_spec error
message (#63233) (#63237)
The error message in install_from_external_spec() referenced
${INSTALLATION_METHOD} which does not exist — the correct variable
is ${AIRFLOW_INSTALLATION_METHOD}. With set -u active, hitting this
error path would crash with an "unbound variable" error instead of
printing the intended user-friendly message.
The typo was introduced in a1717a652b and carried forward into the
inlined copies in both Dockerfiles.
(cherry picked from commit 2ab6f9490dfbc205d1176d3d5dc1204d206f5397)
Co-authored-by: Xiaodong DENG <[email protected]>
---
Dockerfile | 2 +-
Dockerfile.ci | 2 +-
scripts/docker/install_airflow_when_building_images.sh | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/Dockerfile b/Dockerfile
index 4ae1247f4ad..4f052181f35 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -1202,7 +1202,7 @@ function install_from_external_spec() {
installation_command_flags="apache-airflow[${AIRFLOW_EXTRAS}]${AIRFLOW_VERSION_SPECIFICATION}"
else
echo
- echo "${COLOR_RED}The '${INSTALLATION_METHOD}' installation method is
not supported${COLOR_RESET}"
+ echo "${COLOR_RED}The '${AIRFLOW_INSTALLATION_METHOD}' installation
method is not supported${COLOR_RESET}"
echo
echo "${COLOR_YELLOW}Supported methods are ('.',
'apache-airflow')${COLOR_RESET}"
echo
diff --git a/Dockerfile.ci b/Dockerfile.ci
index c9e3c2ede38..d871ad2bcc3 100644
--- a/Dockerfile.ci
+++ b/Dockerfile.ci
@@ -937,7 +937,7 @@ function install_from_external_spec() {
installation_command_flags="apache-airflow[${AIRFLOW_EXTRAS}]${AIRFLOW_VERSION_SPECIFICATION}"
else
echo
- echo "${COLOR_RED}The '${INSTALLATION_METHOD}' installation method is
not supported${COLOR_RESET}"
+ echo "${COLOR_RED}The '${AIRFLOW_INSTALLATION_METHOD}' installation
method is not supported${COLOR_RESET}"
echo
echo "${COLOR_YELLOW}Supported methods are ('.',
'apache-airflow')${COLOR_RESET}"
echo
diff --git a/scripts/docker/install_airflow_when_building_images.sh
b/scripts/docker/install_airflow_when_building_images.sh
index 40f5b7cba15..bec13ca2a5d 100644
--- a/scripts/docker/install_airflow_when_building_images.sh
+++ b/scripts/docker/install_airflow_when_building_images.sh
@@ -147,7 +147,7 @@ function install_from_external_spec() {
installation_command_flags="apache-airflow[${AIRFLOW_EXTRAS}]${AIRFLOW_VERSION_SPECIFICATION}"
else
echo
- echo "${COLOR_RED}The '${INSTALLATION_METHOD}' installation method is
not supported${COLOR_RESET}"
+ echo "${COLOR_RED}The '${AIRFLOW_INSTALLATION_METHOD}' installation
method is not supported${COLOR_RESET}"
echo
echo "${COLOR_YELLOW}Supported methods are ('.',
'apache-airflow')${COLOR_RESET}"
echo
