vincbeck commented on code in PR #39946: URL: https://github.com/apache/airflow/pull/39946#discussion_r1625028890
########## airflow/providers/amazon/provider.yaml: ########## @@ -92,17 +92,17 @@ dependencies: # We should update minimum version of boto3 and here regularly to avoid `pip` backtracking with the number # of candidates to consider. Make sure to configure boto3 version here as well as in all the tools below # in the `devel-dependencies` section to be the same minimum version. - - boto3>=1.33.0 - - botocore>=1.33.0 + - boto3>=1.34.0 + - botocore>=1.34.0 - inflection>=0.5.1 # Allow a wider range of watchtower versions for flexibility among users - - watchtower>=2.0.1,<4 + - watchtower>=3.0.0,<4 Review Comment: > @o-nikolas @vincbeck @ferruzzi -> I recall that there was a reason you wanted to have watchtower 2, but what I found out during those tests is that **some** tests were failing when it was watchtower 2, so if we go back to watchtower 2, I might need your help on deciding what to do with those and proposing a fix (we could deliberately skip thos tests if watchtower 2 is used or - better implementing a fix to work for both. Sounds good, I cannot recall the reason why we needed watchtower 2. Let's try with 3 and see it from there -- 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]
