Skip to content

Conversation

@simonrw
Copy link
Contributor

@simonrw simonrw commented Sep 18, 2025

Motivation

In #13037 we updated to Python 3.13 (🎉) however we missed updating a couple of dev scripts. These overwrite the venv site-packages path in a running pro container/pod and as such need to use the correct path.

Changes

Swap 3.11 for 3.13

Testing

Run python -m localstack.dev.run with the pro image with a code change and it should be picked up. Without this PR the code change is not picked up.

@simonrw simonrw added this to the 4.9 milestone Sep 18, 2025
@simonrw simonrw added semver: patch Non-breaking changes which can be included in patch releases review: merge when ready Signals to the reviewer that a PR can be merged if accepted docs: skip Pull request does not require documentation changes labels Sep 18, 2025
@github-actions
Copy link

Test Results - Preflight, Unit

22 172 tests  ±0   20 434 ✅ ±0   16m 31s ⏱️ +15s
     1 suites ±0    1 738 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 1f467f2c. ± Comparison against base commit 74a088f.

@github-actions
Copy link

Test Results (amd64) - Acceptance

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

Results for commit 1f467f2c. ± Comparison against base commit 74a088f.

@simonrw simonrw marked this pull request as ready for review September 18, 2025 16:58
@simonrw simonrw requested a review from dfangl September 18, 2025 16:58
@github-actions
Copy link

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   2h 35m 31s ⏱️
5 037 tests 4 556 ✅ 481 💤 0 ❌
5 043 runs  4 556 ✅ 487 💤 0 ❌

Results for commit 1f467f2c.

@github-actions
Copy link

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 59m 37s ⏱️ + 1m 52s
4 663 tests ±0  4 342 ✅ ±0  321 💤 ±0  0 ❌ ±0 
4 665 runs  ±0  4 342 ✅ ±0  323 💤 ±0  0 ❌ ±0 

Results for commit 1f467f2c. ± Comparison against base commit 74a088f.

@simonrw
Copy link
Contributor Author

simonrw commented Sep 18, 2025

Closing as a duplicate of #13163

@simonrw simonrw closed this Sep 18, 2025
@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 review: merge when ready Signals to the reviewer that a PR can be merged if accepted 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.

3 participants