-
-
Notifications
You must be signed in to change notification settings - Fork 183
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support up-to-date botocore
and urllib3 2.0
#1037
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1037 +/- ##
==========================================
- Coverage 86.27% 86.23% -0.04%
==========================================
Files 58 58
Lines 5740 5740
==========================================
- Hits 4952 4950 -2
- Misses 788 790 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
adce0a0
to
a0496c2
Compare
383ba1d
to
eebf721
Compare
ae4143c
to
ea5d1fc
Compare
d9117cd
to
7459d16
Compare
I think this will help me with another issue. I'm trying to get a service definition for AWS bedrock supported across both aiobotocore and aioboto3. That requires a botocore bump to atleast 1.31.57. |
That should be possible once this PR is merged and a release is published. |
6904c30
to
a7b2f7c
Compare
a7b2f7c
to
c30e596
Compare
c30e596
to
724dd28
Compare
botocore
and urllib3 2.0
Until this PR is reviewed, I intend to monitor |
Hi team, looks like we have a lot of folks blocked waiting for this change. Any maintainers available to review this? cc @thehesiod @terrycain @jettify Thanks! |
Pinning to a patch version of |
Agreed.
Unfortunately, this is not possible due to the tight integration between botocore and aiobotocore. |
4e0ce67
to
f62c930
Compare
looking! sorry for delay, have been super busy |
my apologies for delay! kids/work/personal stuff has been honestly overwhelming this last month. Deploying shortly |
We hear you! 😄 |
FYI merging 2.7.0 + 2.7.1 in #1034 as 2.7.0 was never released |
Sorry, my bad! |
Sorry for posting on a closed PR. Is there any reason why botocore was pinned to
instead of
The problem I am having is that So I am stuck with |
You should be able to gain the benefits of this PR, if you work with |
Yes thanks, you are correct. The install works, albeit with a quite old version of s3fs (so I am waiting for s3fs to support aiobotocore 2.7.0).
|
Same here. Already merged in fsspec/s3fs#809 |
And.... THANKS. This one will solve a lof of our problems in Apache Airflow ! We had to figure out a very weird scheme there to cope with pinning. |
Description of Change
This PR intends to improve general compatibility of
aiobotocore
within the Python ecosystem by relaxing the dependency specification ofbotocore
, as well asboto3
andawscli
.Thanks to boto/botocore#3034, this PR also implicitly provides
support for urllib3 2.0 for Python 3.10+
. Installingurllib3>=2.0.0
causes unit tests to fail due to an outdated development dependency. This PR bumpsdocker
to address the issue.Assumptions
Upstream diff does not contain any changes that require adjustments to the
aiobotocore
codebase: Besides JSON data and docstring updates, there are only minor changes tobotocore
Python code and unit tests, none of which affectaiobotocore
. This allows for the proposed widening of thebotocore
dependency specification.Checklist for All Submissions
Checklist when updating botocore and/or aiohttp versions