Skip to content
This repository was archived by the owner on Mar 23, 2026. It is now read-only.

Upgrade pinned Python dependencies#13682

Merged
alexrashed merged 3 commits intomainfrom
upgrade-dependencies
Feb 3, 2026
Merged

Upgrade pinned Python dependencies#13682
alexrashed merged 3 commits intomainfrom
upgrade-dependencies

Conversation

@localstack-bot
Copy link
Copy Markdown
Contributor

@localstack-bot localstack-bot commented Feb 3, 2026

This PR upgrades all the pinned Python dependencies.

@localstack-bot localstack-bot added area: dependencies Pull requests that update a dependency file docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes semver: patch Non-breaking changes which can be included in patch releases labels Feb 3, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 3, 2026 •
edited
Loading

S3 Image Test Results (AMD64 / ARM64)

    2 files      2 suites   8m 9s ⏱️
  552 tests   500 ✅  52 💤 0 ❌
1 104 runs  1 000 ✅ 104 💤 0 ❌

Results for commit 7766973.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 3, 2026 •
edited
Loading

Test Results - Preflight, Unit

23 088 tests  ±0   21 229 ✅ ±0   6m 20s ⏱️ +11s
     1 suites ±0    1 859 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 7766973. ± Comparison against base commit a8c43f2.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 3, 2026 •
edited
Loading

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   2m 50s ⏱️ -12s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit 7766973. ± Comparison against base commit a8c43f2.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 3, 2026 •
edited
Loading

Test Results (amd64) - Integration, Bootstrap

    5 files  ±0      5 suites  ±0   2h 36m 11s ⏱️ -1s
5 600 tests +1  5 039 ✅  - 2  561 💤 +3  0 ❌ ±0 
5 606 runs  +1  5 039 ✅  - 2  567 💤 +3  0 ❌ ±0 

Results for commit 7766973. ± Comparison against base commit a8c43f2.

This pull request skips 2 tests. 
tests.aws.services.sns.test_sns.TestSNSSubscriptionSQS ‑ test_publish_batch_exceptions
tests.aws.services.sns.test_sns.TestSNSSubscriptionSQSFifo ‑ test_validations_for_fifo

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 3, 2026 •
edited
Loading

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 58m 56s ⏱️ +20s
5 179 tests +1  4 788 ✅  - 2  391 💤 +3  0 ❌ ±0 
5 181 runs  +1  4 788 ✅  - 2  393 💤 +3  0 ❌ ±0 

Results for commit 7766973. ± Comparison against base commit a8c43f2.

This pull request skips 2 tests. 
tests.aws.services.sns.test_sns.TestSNSSubscriptionSQS ‑ test_publish_batch_exceptions
tests.aws.services.sns.test_sns.TestSNSSubscriptionSQSFifo ‑ test_validations_for_fifo

♻️ This comment has been updated with latest results.

@alexrashed
Copy link
Copy Markdown
Member

alexrashed commented Feb 3, 2026

@bentsku There are a few tests failing in this upgrade PR:

FAILED tests/aws/services/apigateway/test_apigateway_lambda.py::test_aws_proxy_response_payload_format_validation
FAILED tests/aws/services/sns/test_sns.py::TestSNSSubscriptionSQS::test_publish_batch_exceptions - Failed: DID NOT RAISE <class 'botocore.exceptions.ClientError'>
FAILED tests/aws/services/sns/test_sns.py::TestSNSSubscriptionSQSFifo::test_validations_for_fifo - Failed: DID NOT RAISE <class 'botocore.exceptions.ClientError'>

Since you are owning both API GW and SNS, and I think some of the issues might be related to the rolo release, I assigned the PR to you. Hope that's okay, let me know if I can help!

@alexrashed alexrashed added this to the 4.14 milestone Feb 3, 2026
@alexrashed alexrashed mentioned this pull request Feb 3, 2026
Closed
@bentsku
Copy link
Copy Markdown
Contributor

bentsku commented Feb 3, 2026 •
edited
Loading

@alexrashed I've pushed a pin for rolo and localstack-twisted so that we can merge this PR quickly, I'll need to do a new release of rolo to fix the multi-value headers handling. I'll open a separate PR to upgrade them 👍 thanks for the ping! (this upgrade is not required by anything else and was just a chore to upgrade Twisted, so nothing is waiting on it 👍)

@bentsku
Copy link
Copy Markdown
Contributor

bentsku commented Feb 3, 2026 •
edited
Loading

It seems like the following:

FAILED tests/aws/services/sns/test_sns.py::TestSNSSubscriptionSQS::test_publish_batch_exceptions - Failed: DID NOT RAISE <class 'botocore.exceptions.ClientError'>
FAILED tests/aws/services/sns/test_sns.py::TestSNSSubscriptionSQSFifo::test_validations_for_fifo - Failed: DID NOT RAISE <class 'botocore.exceptions.ClientError'>

Are actually Moto regressions? But as we're going to switch SNS v2 very soon, we probably can skip them safely? It will introduce a small regression in latest but shouldn't last. \cc @baermat

@bentsku bentsku requested a review from baermat as a code owner February 3, 2026 09:57
@viren-nadkarni viren-nadkarni mentioned this pull request Feb 3, 2026
Merged
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 3, 2026

Test Results - Alternative Providers

206 tests   - 1 260   178 ✅  - 720   2m 13s ⏱️ - 31m 3s
  1 suites  -     4    28 💤  - 540 
  1 files    -     4     0 ❌ ±  0 

Results for commit 7766973. ± Comparison against base commit a8c43f2.

This pull request removes 1261 and adds 1 tests. Note that renamed tests count towards both. 
tests.aws.services.cloudformation.api.test_changesets ‑ test_autoexpand_capability_requirement
tests.aws.services.cloudformation.api.test_changesets ‑ test_create_and_then_remove_non_supported_resource_change_set
tests.aws.services.cloudformation.api.test_changesets ‑ test_create_and_then_remove_supported_resource_change_set
tests.aws.services.cloudformation.api.test_changesets ‑ test_create_and_then_update_refreshes_template_metadata
tests.aws.services.cloudformation.api.test_changesets ‑ test_create_change_set_create_existing
tests.aws.services.cloudformation.api.test_changesets ‑ test_create_change_set_invalid_params
tests.aws.services.cloudformation.api.test_changesets ‑ test_create_change_set_missing_stackname
tests.aws.services.cloudformation.api.test_changesets ‑ test_create_change_set_no_changes
tests.aws.services.cloudformation.api.test_changesets ‑ test_create_change_set_update_nonexisting
tests.aws.services.cloudformation.api.test_changesets ‑ test_create_change_set_update_without_parameters
…

tests.aws.services.sns.test_sns.TestSNSPublishCrud ‑ test_publish_no_confirm_subscription

bentsku
bentsku approved these changes Feb 3, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@bentsku bentsku left a comment

Choose a reason for hiding this comment

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

LGTM!

Small update, we've pinned rolo and localstack-twisted until a new version of rolo is released with a fix for the Multivalue headers regression.

Also, 2 tests in SNS are skipped due to a revert to a previous change done regarding boolean casting of values, because the SNS v2 provider is going to be merged soon.

The failing test is a flake due to timing issue, something that will need to be looked into 👀

@alexrashed alexrashed merged commit 706ea55 into main Feb 3, 2026
71 of 72 checks passed
@alexrashed alexrashed deleted the upgrade-dependencies branch February 3, 2026 12:05
@bentsku bentsku mentioned this pull request Feb 4, 2026
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@alexrashed alexrashed Awaiting requested review from alexrashed alexrashed is a code owner

@silv-io silv-io Awaiting requested review from silv-io

@k-a-il k-a-il Awaiting requested review from k-a-il k-a-il is a code owner

@baermat baermat Awaiting requested review from baermat

1 more reviewer

@bentsku bentsku bentsku approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@bentsku bentsku

Labels

area: dependencies Pull requests that update a dependency file docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes semver: patch Non-breaking changes which can be included in patch releases

Projects

None yet

Milestone

4.14

Development

Successfully merging this pull request may close these issues.

3 participants

@localstack-bot @alexrashed @bentsku