Skip to content

LDM Migration #12852

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

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

LDM Migration #12852

wants to merge 14 commits into from

Conversation

MEPalma
Copy link
Contributor

@MEPalma MEPalma commented Jul 10, 2025

Motivation

Carries out the migration of the LDM.

Changes

Removed LDM modules and introduced needed integrations to support the migration.

@MEPalma MEPalma added this to the Playground milestone Jul 10, 2025
@MEPalma MEPalma self-assigned this Jul 10, 2025
@MEPalma MEPalma added the semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases label Jul 10, 2025
Copy link

github-actions bot commented Jul 10, 2025

Test Results - Preflight, Unit

21 850 tests   - 12   20 193 ✅  - 12   6m 25s ⏱️ +10s
     1 suites ± 0    1 657 💤 ± 0 
     1 files   ± 0        0 ❌ ± 0 

Results for commit ccc4430. ± Comparison against base commit 6bc6c40.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jul 10, 2025

Test Results (amd64) - Acceptance

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

Results for commit ccc4430. ± Comparison against base commit 6bc6c40.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jul 10, 2025

Test Results (amd64) - Integration, Bootstrap

    5 files  ±0      5 suites  ±0   2h 20m 50s ⏱️ +53s
5 288 tests ±0  4 358 ✅ ±0  930 💤 ±0  0 ❌ ±0 
5 294 runs  ±0  4 358 ✅ ±0  936 💤 ±0  0 ❌ ±0 

Results for commit ccc4430. ± Comparison against base commit 6bc6c40.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jul 10, 2025

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 45m 2s ⏱️ -6s
4 929 tests ±0  4 152 ✅ ±0  777 💤 ±0  0 ❌ ±0 
4 931 runs  ±0  4 152 ✅ ±0  779 💤 ±0  0 ❌ ±0 

Results for commit ccc4430. ± Comparison against base commit 6bc6c40.

♻️ This comment has been updated with latest results.

@MEPalma MEPalma marked this pull request as ready for review July 14, 2025 12:35
@MEPalma MEPalma force-pushed the MEP-LDM-provisioner branch from 90030a5 to 6bc6c40 Compare July 16, 2025 07:25
Base automatically changed from MEP-LDM-provisioner to main July 17, 2025 11:59
@localstack-bot
Copy link
Contributor

Currently, only patch changes are allowed on main. Your PR labels (semver: minor) indicate that it cannot be merged into the main at this time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

2 participants