From 3fa0d077af1336134b6d2e713f9a968ef65d98f4 Mon Sep 17 00:00:00 2001 From: Jarek Potiuk Date: Sun, 29 May 2022 00:18:41 +0200 Subject: [PATCH] Remove certifi limitations from eager upgrade limits The certifi limitation was introduced to keep snowflake happy while performing eager upgrade because it added limits on certifi. However seems like it is not limitation any more in latest versions of snowflake python connector, so we can safely remove it from here. The only remaining limit is dill but this one still holds. --- Dockerfile | 3 +-- Dockerfile.ci | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/Dockerfile b/Dockerfile index ebd7f835086b8..cab46244ae954 100644 --- a/Dockerfile +++ b/Dockerfile @@ -1329,9 +1329,8 @@ ARG INSTALL_PACKAGES_FROM_CONTEXT="false" ARG AIRFLOW_IS_IN_CONTEXT="false" # Those are additional constraints that are needed for some extras but we do not want to # Force them on the main Airflow package. -# * certifi<2021.0.0 required to keep snowflake happy # * dill<0.3.3 required by apache-beam -ARG EAGER_UPGRADE_ADDITIONAL_REQUIREMENTS="dill<0.3.3 certifi<2021.0.0" +ARG EAGER_UPGRADE_ADDITIONAL_REQUIREMENTS="dill<0.3.3" ENV ADDITIONAL_PYTHON_DEPS=${ADDITIONAL_PYTHON_DEPS} \ INSTALL_PACKAGES_FROM_CONTEXT=${INSTALL_PACKAGES_FROM_CONTEXT} \ diff --git a/Dockerfile.ci b/Dockerfile.ci index 41aaf579ec749..c089544f244b3 100644 --- a/Dockerfile.ci +++ b/Dockerfile.ci @@ -1244,9 +1244,8 @@ RUN echo "Airflow version: ${AIRFLOW_VERSION}" # Those are additional constraints that are needed for some extras but we do not want to # force them on the main Airflow package. Those limitations are: -# * certifi<2021.0.0: required by snowflake provider # * dill<0.3.3 required by apache-beam -ARG EAGER_UPGRADE_ADDITIONAL_REQUIREMENTS="dill<0.3.3 certifi<2021.0.0" +ARG EAGER_UPGRADE_ADDITIONAL_REQUIREMENTS="dill<0.3.3" ARG UPGRADE_TO_NEWER_DEPENDENCIES="false" ENV EAGER_UPGRADE_ADDITIONAL_REQUIREMENTS=${EAGER_UPGRADE_ADDITIONAL_REQUIREMENTS} \ UPGRADE_TO_NEWER_DEPENDENCIES=${UPGRADE_TO_NEWER_DEPENDENCIES}