Skip to content

Commit

Permalink
Fix branch used for constraints in k8s env creation (apache#46318)
Browse files Browse the repository at this point in the history
This was using `constraints-{AIRFLOW_BRANCH}`, which could be e.g.
`constraints-v3-0-test` outside of main, where it should be
`constraints-3-0`, which is already defined in
`DEFAULT_AIRFLOW_CONSTRAINTS_BRANCH`.
  • Loading branch information
jedcunningham authored and amoghrajesh committed Feb 3, 2025
1 parent e309222 commit 6faea49
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions dev/breeze/src/airflow_breeze/utils/kubernetes_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
from typing import Any, NamedTuple
from urllib import request

from airflow_breeze.branch_defaults import AIRFLOW_BRANCH
from airflow_breeze.branch_defaults import DEFAULT_AIRFLOW_CONSTRAINTS_BRANCH
from airflow_breeze.global_constants import (
ALLOWED_ARCHITECTURES,
ALLOWED_PYTHON_MAJOR_MINOR_VERSIONS,
Expand Down Expand Up @@ -336,7 +336,7 @@ def _install_packages_in_k8s_virtualenv():
"--constraint",
"https://raw.githubusercontent.com/"
f"{APACHE_AIRFLOW_GITHUB_REPOSITORY}/"
f"constraints-{AIRFLOW_BRANCH}/constraints-{python_major_minor_version}.txt",
f"{DEFAULT_AIRFLOW_CONSTRAINTS_BRANCH}/constraints-{python_major_minor_version}.txt",
],
)
install_packages_result = run_command(
Expand Down

0 comments on commit 6faea49

Please sign in to comment.