fix: remove obsolete boto3 extra from aiobotocore dependency#64330
fix: remove obsolete boto3 extra from aiobotocore dependency#64330nidhi1603 wants to merge 11 commits intoapache:mainfrom
Conversation
aiobotocore 3.3.0 removed the boto3 extra (aio-libs/aiobotocore#1424). Since boto3 is already a required dependency of the Amazon provider package, the extra is unnecessary. Fixes apache#64283
|
Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contributors' Guide (https://github.com/apache/airflow/blob/main/contributing-docs/README.rst)
|
|
@nidhi1603 - what were some of the tests that you ran here? |
I verified that boto3 remains a direct dependency in the Amazon provider's pyproject.toml (listed under [project] dependencies), so removing the [boto3] extra from aiobotocore doesn't change what gets installed. |
|
I'd suggest checking out the documentation for running Airflow locally with breeze. @vincbeck provided a scenario that makes sense to test. |
I set up Breeze locally and validated this change in the Airflow Breeze environment. Environment:
I also verified inside the Breeze container that the relevant packages resolve correctly after removing the obsolete
I then ran the Amazon provider deferrable test sweep with: pytest providers/amazon/tests -k deferrable -q | tee /tmp/amazon_deferrable_test.txtResult:
So the deferrable AWS operator/sensor paths are working correctly on my side after this dependency change. |
Remove obsolete
[boto3]extra fromaiobotocoredependency in Amazon provider'spyproject.toml.aiobotocore3.3.0 removed theboto3extra (aio-libs/aiobotocore#1424).Since
boto3is already a required dependency of the Amazon provider package, the extra is unnecessary.closes: #64283
Was generative AI tooling used to co-author this PR?
{pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.