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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
