Skip to content

Commit

Permalink
Fixes constraint generation for pypi providers (apache#15470)
Browse files Browse the repository at this point in the history
* Fixes constraint generation for pypi providers

The constraints generated from PyPI version of providers, missed
core requirements of Airflow, therefore the constraints were not
consistent with setup.py core requirements.

Fixes: apache#15463
(cherry picked from commit 5da74f6)
  • Loading branch information
potiuk committed May 6, 2021
1 parent 8571cfd commit fd3eddd
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion scripts/in_container/_in_container_utils.sh
Original file line number Diff line number Diff line change
Expand Up @@ -318,6 +318,7 @@ function install_local_airflow_with_eager_upgrade() {


function install_all_providers_from_pypi_with_eager_upgrade() {
NO_PROVIDERS_EXTRAS=$(python -c 'import setup; print(",".join(setup.CORE_EXTRAS_REQUIREMENTS))')
ALL_PROVIDERS_PACKAGES=$(python -c 'import setup; print(setup.get_all_provider_packages())')
local packages_to_install=()
local provider_package
Expand All @@ -340,7 +341,7 @@ function install_all_providers_from_pypi_with_eager_upgrade() {
# Installing it with Airflow makes sure that the version of package that matches current
# Airflow requirements will be used.
# shellcheck disable=SC2086
pip install -e . "${packages_to_install[@]}" ${EAGER_UPGRADE_ADDITIONAL_REQUIREMENTS} \
pip install -e ".[${NO_PROVIDERS_EXTRAS}]" "${packages_to_install[@]}" ${EAGER_UPGRADE_ADDITIONAL_REQUIREMENTS} \
--upgrade --upgrade-strategy eager

}
Expand Down

0 comments on commit fd3eddd

Please sign in to comment.