[GitHub] [airflow] jedcunningham commented on a change in pull request #17397: Update to Celery 5

[GitBox <gi...@apache.org>]

jedcunningham commented on a change in pull request #17397:
URL: https://github.com/apache/airflow/pull/17397#discussion_r685515621



##########
File path: Dockerfile.ci
##########
@@ -278,7 +278,7 @@ ENV AIRFLOW_REPO=${AIRFLOW_REPO}\
 # * lazy-object-proxy<1.5.0: required by astroid
 # * pyjwt<2.0.0: flask-jwt-extended requires it
 # * dill<0.3.3 required by apache-beam
-ARG EAGER_UPGRADE_ADDITIONAL_REQUIREMENTS="lazy-object-proxy<1.5.0 pyjwt<2.0.0 dill<0.3.3 certifi<2021.0.0"
+ARG EAGER_UPGRADE_ADDITIONAL_REQUIREMENTS="lazy-object-proxy<1.5.0 pyjwt<2.0.0 dill<0.3.3 certifi<2021.0.0 botocore>=1.20.107,<1.21.0"

Review comment:
       Hmm, okay I'll keep playing with it. This still wasn't enough to get it working and removing the lower limit made it even worse.
   
   Btw, I think something is broken for these eager update PRs - it only tries to do the eager update when there is a single commit. If I add a second fixup commit, it installs with constraints and pip complains about the conflict. That's why I've been squashing my commits in this PR, but I haven't really dug into it yet.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org