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

Migrate IAM user implementation#13806

Merged
pinzon merged 2 commits into
iam/moto-migrationfrom
daniel/unc-228
Feb 20, 2026
Merged

Migrate IAM user implementation#13806
pinzon merged 2 commits into
iam/moto-migrationfrom
daniel/unc-228

Conversation

@dfangl

@dfangl dfangl commented Feb 20, 2026

Copy link
Copy Markdown
Member

Motivation

Next up is the migration of IAM users (including logic related to policies).

Changes

Migrated IAM user methods (excluding group membership and access keys) from moto to LocalStack.

Tests

Tests in test_iam_user.py have been unskipped (except those for groups and access keys).

Related

Closes UNC-228

@dfangl dfangl added this to the 4.14 milestone Feb 20, 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: skip Pull request does not have to be mentioned in the release notes labels Feb 20, 2026
@github-actions

Copy link
Copy Markdown

Test Results - Preflight, Unit

23 123 tests  ±0   21 252 ✅ ±0   6m 7s ⏱️ +2s
     1 suites ±0    1 871 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit edc88e3. ± Comparison against base commit 3e9b86f.

@github-actions

Copy link
Copy Markdown

Test Results (amd64) - Acceptance

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

Results for commit edc88e3. ± Comparison against base commit 3e9b86f.

@github-actions

Copy link
Copy Markdown

Test Results (amd64) - Integration, Bootstrap

    5 files  ±0      5 suites  ±0   2h 37m 22s ⏱️ - 3m 42s
6 026 tests  - 1  5 406 ✅  - 9  587 💤  - 15  33 ❌ +23 
6 032 runs   - 1  5 406 ✅  - 9  593 💤  - 15  33 ❌ +23 

For more details on these failures, see this check.

Results for commit edc88e3. ± Comparison against base commit 3e9b86f.

@github-actions

Copy link
Copy Markdown

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   2h 3m 30s ⏱️ - 2m 17s
5 621 tests  - 1  5 159 ✅  - 8  431 💤  - 15  31 ❌ +22 
5 623 runs   - 1  5 159 ✅  - 8  433 💤  - 15  31 ❌ +22 

For more details on these failures, see this check.

Results for commit edc88e3. ± Comparison against base commit 3e9b86f.

@pinzon pinzon left a comment

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.

LGTM. 👍

This was referenced Feb 20, 2026
@pinzon pinzon merged commit 00594bb into iam/moto-migration Feb 20, 2026
36 of 43 checks passed
@pinzon pinzon deleted the daniel/unc-228 branch February 20, 2026 22:54
dfangl added a commit that referenced this pull request Feb 26, 2026
dfangl added a commit that referenced this pull request Feb 26, 2026
dfangl added a commit that referenced this pull request Mar 4, 2026
dfangl added a commit that referenced this pull request Mar 6, 2026
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: 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.

2 participants