Skip to content

Conversation

@alexrashed
Copy link
Member

@alexrashed alexrashed commented Sep 16, 2025

Motivation

With localstack/moto#78, we introduced a new workflow in localstack/moto which should automatically - on a weekly schedule - sync-rebase and release our fork of moto (pushing moto-ext to pypi).
In order to implement this continuous integration of changes in moto here in LocalStack, we now should now make sure that updates to moto(-ext) get included regularly.
This is why this PR removes the pin to a specific version in the requirements of localstack-core and bumps the version of moto-ext to the latest release (the first one which has been created fully automatically).

This bump will cause some issues (since there was quite a distance), but afterwards we are hoping that the more frequent integration decreases the pain there. Basically this means getting a bit lower on the X-axis on this simple diagram from Martin Fowler (posted 14 years ago in this blog post 😅) will hopefully get us down on the pain axis:
image

Changes

  • Unpin moto-ext in the pyproject.toml
  • Update the dependency pins for moto-ext
  • TODO: Add fixes to upgrade to the latest version here.

TODO

  • Make sure all pipelines here and in our downstream project are 💚

Related

Related to PNX-127

@alexrashed alexrashed added semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases docs: skip Pull request does not require documentation changes labels Sep 16, 2025
@github-actions
Copy link

github-actions bot commented Sep 16, 2025

S3 Image Test Results (AMD64 / ARM64)

    2 files  ±0    2 suites  ±0   8m 0s ⏱️ -13s
  532 tests ±0  480 ✅ ±0   52 💤 ±0  0 ❌ ±0 
1 064 runs  ±0  960 ✅ ±0  104 💤 ±0  0 ❌ ±0 

Results for commit 314a961. ± Comparison against base commit c8f43c9.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Sep 16, 2025

Test Results - Preflight, Unit

22 160 tests  ±0   20 422 ✅ ±0   6m 32s ⏱️ -1s
     1 suites ±0    1 738 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 314a961. ± Comparison against base commit c8f43c9.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Sep 16, 2025

Test Results (amd64) - Acceptance

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

Results for commit 314a961. ± Comparison against base commit c8f43c9.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Sep 16, 2025

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   2h 40m 44s ⏱️
5 046 tests 4 555 ✅ 491 💤 0 ❌
5 052 runs  4 555 ✅ 497 💤 0 ❌

Results for commit 314a961.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Sep 16, 2025

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   2h 1m 31s ⏱️ + 3m 25s
4 672 tests ±0  4 341 ✅ ±0  331 💤 ±0  0 ❌ ±0 
4 674 runs  ±0  4 341 ✅ ±0  333 💤 ±0  0 ❌ ±0 

Results for commit 314a961. ± Comparison against base commit c8f43c9.

♻️ This comment has been updated with latest results.

@localstack localstack deleted a comment from localstack-bot Sep 17, 2025
@alexrashed alexrashed marked this pull request as ready for review September 17, 2025 17:07
@alexrashed alexrashed merged commit 329a337 into main Sep 18, 2025
57 of 59 checks passed
@alexrashed alexrashed deleted the unpin-moto branch September 18, 2025 06:57
@alexrashed alexrashed added the notes: skip Pull request does not have to be mentioned in the release notes label Sep 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants