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

Migrate IAM/STS from moto to LocalStack#13720

Closed
dfangl wants to merge 39 commits into
mainfrom
iam/moto-migration
Closed

Migrate IAM/STS from moto to LocalStack#13720
dfangl wants to merge 39 commits into
mainfrom
iam/moto-migration

Conversation

@dfangl

@dfangl dfangl commented Feb 9, 2026

Copy link
Copy Markdown
Member

Motivation

This is a feature PR for the entire migration of IAM and STS services from moto to the core LocalStack codebase.

(Relevant) contribution should be made via separate PRs

Changes

  • ...

Tests

Tests will be migrated and AWS validated from moto

Related

IAM/STS internalization project

@dfangl dfangl added this to the 4.14 milestone Feb 9, 2026
@dfangl dfangl 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 notes: needed Pull request should be mentioned in the release notes labels Feb 9, 2026
@github-actions

github-actions Bot commented Feb 9, 2026

Copy link
Copy Markdown

Test Results - Preflight, Unit

23 104 tests  ±0   21 217 ✅ ±0   6m 13s ⏱️ -5s
     1 suites ±0    1 887 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 708a15e. ± Comparison against base commit 9642b8e.

♻️ This comment has been updated with latest results.

@dfangl dfangl force-pushed the iam/moto-migration branch from d08f4b8 to bfc7836 Compare February 10, 2026 17:47
@github-actions

github-actions Bot commented Feb 10, 2026

Copy link
Copy Markdown

LocalStack Community integration with Pro

    2 files  ±  0      2 suites  ±0   2h 9m 52s ⏱️ + 5m 4s
5 662 tests +307  5 039 ✅ +55  372 💤 +1  251 ❌ +251 
5 664 runs  +307  5 039 ✅ +55  374 💤 +1  251 ❌ +251 

For more details on these failures, see this check.

Results for commit 708a15e. ± Comparison against base commit 9642b8e.

This pull request removes 10 and adds 317 tests. Note that renamed tests count towards both.
tests.aws.services.iam.test_iam.TestIAMIntegrations ‑ test_create_describe_role
tests.aws.services.iam.test_iam.TestIAMIntegrations ‑ test_list_roles_with_permission_boundary
tests.aws.services.iam.test_iam.TestIAMIntegrations ‑ test_role_attach_policy
tests.aws.services.iam.test_iam.TestIAMIntegrations ‑ test_update_assume_role_policy
tests.aws.services.iam.test_iam.TestRoles ‑ test_role_with_tags
tests.aws.services.sts.test_sts.TestSTSIntegrations ‑ test_get_caller_identity_role_access_key[False]
tests.aws.services.sts.test_sts.TestSTSIntegrations ‑ test_get_caller_identity_role_access_key[True]
tests.aws.services.sts.test_sts.TestSTSIntegrations ‑ test_get_caller_identity_root
tests.aws.services.sts.test_sts.TestSTSIntegrations ‑ test_get_caller_identity_user_access_key[False]
tests.aws.services.sts.test_sts.TestSTSIntegrations ‑ test_get_caller_identity_user_access_key[True]
tests.aws.services.iam.test_iam_account_aliases.TestIAMAccountAliases ‑ test_account_alias_lifecycle
tests.aws.services.iam.test_iam_account_authorization_details.TestGetAccountAuthorizationDetails ‑ test_get_account_authorization_details_filter_aws_managed_policy
tests.aws.services.iam.test_iam_account_authorization_details.TestGetAccountAuthorizationDetails ‑ test_get_account_authorization_details_filter_group
tests.aws.services.iam.test_iam_account_authorization_details.TestGetAccountAuthorizationDetails ‑ test_get_account_authorization_details_filter_local_managed_policy
tests.aws.services.iam.test_iam_account_authorization_details.TestGetAccountAuthorizationDetails ‑ test_get_account_authorization_details_filter_role
tests.aws.services.iam.test_iam_account_authorization_details.TestGetAccountAuthorizationDetails ‑ test_get_account_authorization_details_filter_user
tests.aws.services.iam.test_iam_account_authorization_details.TestGetAccountAuthorizationDetails ‑ test_get_account_authorization_details_no_filter
tests.aws.services.iam.test_iam_account_authorization_details.TestGetAccountAuthorizationDetails ‑ test_get_account_authorization_details_with_inline_policies
tests.aws.services.iam.test_iam_account_authorization_details.TestGetAccountAuthorizationDetails ‑ test_get_account_authorization_details_with_tags
tests.aws.services.iam.test_iam_account_password_policy.TestAccountPasswordPolicy ‑ test_delete_account_password_policy
…

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Feb 10, 2026

Copy link
Copy Markdown

Test Results (amd64) - Acceptance

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

Results for commit 708a15e. ± Comparison against base commit 9642b8e.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Feb 10, 2026

Copy link
Copy Markdown

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   2h 40m 24s ⏱️
6 067 tests 5 538 ✅ 528 💤 1 ❌
6 073 runs  5 538 ✅ 534 💤 1 ❌

For more details on these failures, see this check.

Results for commit 708a15e.

♻️ This comment has been updated with latest results.

@dfangl dfangl force-pushed the iam/moto-migration branch 3 times, most recently from 7211a8d to aabae16 Compare February 17, 2026 13:30
@github-actions

github-actions Bot commented Feb 18, 2026

Copy link
Copy Markdown

S3 Image Test Results (AMD64 / ARM64)

    2 files    2 suites   8m 7s ⏱️
  573 tests 466 ✅  56 💤  51 ❌
1 146 runs  932 ✅ 112 💤 102 ❌

For more details on these failures, see this check.

Results for commit 9655ffd.

♻️ This comment has been updated with latest results.

@dfangl dfangl force-pushed the iam/moto-migration branch from 49e13e2 to cbcd0f4 Compare February 19, 2026 18:50
@dfangl dfangl modified the milestones: 4.14, 2026.03 Feb 24, 2026
@localstack-bot

Copy link
Copy Markdown
Contributor

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

@dfangl dfangl force-pushed the iam/moto-migration branch 4 times, most recently from 449d4e1 to 2770dfd Compare March 4, 2026 13:48
dfangl and others added 27 commits March 6, 2026 12:03
…tion detail tests (#13785)

Co-authored-by: Simon Walker <simon.walker@localstack.cloud>
Co-authored-by: Daniel Fangl <daniel.fangl@localstack.cloud>
Co-authored-by: Daniel Fangl <daniel.fangl@localstack.cloud>
Co-authored-by: Daniel Fangl <daniel.fangl@localstack.cloud>
Co-authored-by: Daniel Fangl <daniel.fangl@localstack.cloud>
Co-authored-by: Daniel Fangl <daniel.fangl@localstack.cloud>
@dfangl dfangl force-pushed the iam/moto-migration branch from 9655ffd to 708a15e Compare March 6, 2026 11:03
@dfangl dfangl closed this Mar 9, 2026
@alexrashed alexrashed deleted the iam/moto-migration branch March 13, 2026 07:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

docs: skip Pull request does not require documentation changes notes: needed Pull request should 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