Skip to content
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

Upgrade pinned Python dependencies #10853

Merged
merged 4 commits into from
May 23, 2024
Merged

Upgrade pinned Python dependencies #10853

merged 4 commits into from
May 23, 2024

Conversation

localstack-bot
Copy link
Collaborator

This PR upgrades all the pinned Python dependencies.

@localstack-bot localstack-bot added area: dependencies Pull requests that update a dependency file semver: patch Non-breaking changes which can be included in patch releases labels May 21, 2024
Copy link

github-actions bot commented May 21, 2024

S3 Image Test Results (AMD64 / ARM64)

  2 files    2 suites   3m 8s ⏱️
399 tests 347 ✅  52 💤 0 ❌
798 runs  694 ✅ 104 💤 0 ❌

Results for commit b0a04dc.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented May 21, 2024

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 37m 44s ⏱️ - 1m 49s
2 985 tests ±0  2 675 ✅ ±0  310 💤 ±0  0 ❌ ±0 
2 987 runs  ±0  2 675 ✅ ±0  312 💤 ±0  0 ❌ ±0 

Results for commit b0a04dc. ± Comparison against base commit 0c66a74.

♻️ This comment has been updated with latest results.

Copy link
Member

@alexrashed alexrashed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately, it seems like one of the upgrades is breaking the pipeline...

@bblommers
Copy link
Contributor

@alexrashed The latest requests breaks everyone's pipelines, it seems:
psf/requests#6707

@silv-io
Copy link
Member

silv-io commented May 21, 2024

We'll wait here for the next release of docker. This will solve two issues:

  1. We will be able to remove the pin for requests, with the newest fix coming in Fix for requests 2.32 docker/docker-py#3257
  2. We will be able to remove the pin for docker itself since this bigger issue will finally give us a release containing this fix: fix: tag regex should allow ports docker/docker-py#3196

@alexrashed
Copy link
Member

docker-py 7.1.0 has been released which should fix the issue with requests.
I removed the pins and re-generated the locks, let's see if it works now :)

Copy link
Member

@alexrashed alexrashed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pipeline is 💚, so the fixes seem to integrate well. :)

@alexrashed alexrashed merged commit 054ac89 into master May 23, 2024
37 checks passed
@alexrashed alexrashed deleted the upgrade-dependencies branch May 23, 2024 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: dependencies Pull requests that update a dependency file semver: patch Non-breaking changes which can be included in patch releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants