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

upgrade aws-sam-translator#13490

Merged
alexrashed merged 2 commits into
mainfrom
upgrade-dependencies
Dec 10, 2025
Merged

upgrade aws-sam-translator#13490
alexrashed merged 2 commits into
mainfrom
upgrade-dependencies

Conversation

@bentsku

@bentsku bentsku commented Dec 9, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Motivation

Following #13483, it seemed we downgraded aws-sam-translator back to 1.103.0, but as @cloutierMat found out, this has rolled back support for Lambda Managed Instances: https://github.com/aws/serverless-application-model/releases/tag/v1.104.0

This was pinned in Moto due to Python 3.14 issues, which do not concern us yet: getmoto/moto#9521

So this PR is force pinning it at least the latest version as of today.

Changes

  • pin was-sam-translator to at least 1.105.0 and regenerate the requirements

Tests

Related

@bentsku bentsku self-assigned this Dec 9, 2025
@bentsku bentsku added semver: patch Non-breaking changes which can be included in patch releases docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes labels Dec 9, 2025
@github-actions

github-actions Bot commented Dec 9, 2025 •
edited
Loading

Copy link
Copy Markdown

S3 Image Test Results (AMD64 / ARM64)

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

Results for commit 743c4bc.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Dec 9, 2025 •
edited
Loading

Copy link
Copy Markdown

Test Results - Preflight, Unit

23 000 tests  ±0   21 158 ✅ ±0   6m 28s ⏱️ +6s
     1 suites ±0    1 842 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 743c4bc. ± Comparison against base commit 1224e92.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Dec 9, 2025 •
edited
Loading

Copy link
Copy Markdown

Test Results (amd64) - Acceptance

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

Results for commit 743c4bc. ± Comparison against base commit 1224e92.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Dec 9, 2025

Copy link
Copy Markdown

Test Results - Alternative Providers

588 tests   331 ✅  17m 15s ⏱️
  1 suites  257 💤
  1 files      0 ❌

Results for commit eab341f.

@github-actions

github-actions Bot commented Dec 9, 2025 •
edited
Loading

Copy link
Copy Markdown

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   2h 32m 45s ⏱️
5 510 tests 4 954 ✅ 556 💤 0 ❌
5 516 runs  4 954 ✅ 562 💤 0 ❌

Results for commit 743c4bc.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Dec 9, 2025

Copy link
Copy Markdown

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 57m 8s ⏱️ +20s
5 130 tests ±0  4 734 ✅ ±0  396 💤 ±0  0 ❌ ±0 
5 132 runs  ±0  4 734 ✅ ±0  398 💤 ±0  0 ❌ ±0 

Results for commit 743c4bc. ± Comparison against base commit 1224e92.

@bentsku bentsku marked this pull request as ready for review December 9, 2025 23:38
@bentsku bentsku added the dependencies Pull requests that update a dependency file label Dec 9, 2025
alexrashed
alexrashed approved these changes Dec 10, 2025
View reviewed changes

@alexrashed alexrashed left a comment •
edited
Loading

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks a lot for jumping on this! 💯 The min dependency seems like a good workaround, and it seems that the transitive conflict is not an issue. 🚀 🦸🏽

@alexrashed alexrashed merged commit fe9daa5 into main Dec 10, 2025
92 of 101 checks passed
@alexrashed alexrashed deleted the upgrade-dependencies branch December 10, 2025 08:22
@viren-nadkarni viren-nadkarni mentioned this pull request Jan 6, 2026
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@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

@sannya-singal sannya-singal Awaiting requested review from sannya-singal

1 more reviewer

@alexrashed alexrashed alexrashed approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@bentsku bentsku

Labels

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

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bentsku @alexrashed