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

Add option to allow ':' in ARNs#13296

Merged
baermat merged 2 commits into
mainfrom
iot/fix-group-names
Oct 23, 2025
Merged

Add option to allow ':' in ARNs#13296
baermat merged 2 commits into
mainfrom
iot/fix-group-names

Conversation

@baermat

@baermat baermat commented Oct 22, 2025

Copy link
Copy Markdown
Member

Motivation

Certain services (like IoT) allow colons in their resource names, and therefore in their arns. This PR adds an option for this

Changes

  • add option to _resource_arn to allow colons

@baermat baermat added 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 labels Oct 22, 2025
@github-actions

github-actions Bot commented Oct 22, 2025 •
edited
Loading

Copy link
Copy Markdown

Test Results - Preflight, Unit

22 367 tests  +1   20 617 ✅ +1   16m 21s ⏱️ +48s
     1 suites ±0    1 750 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 412623c. ± Comparison against base commit 06df84d.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Oct 22, 2025 •
edited
Loading

Copy link
Copy Markdown

Test Results (amd64) - Acceptance

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

Results for commit 412623c. ± Comparison against base commit 06df84d.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Oct 22, 2025 •
edited
Loading

Copy link
Copy Markdown

Test Results (amd64) - Integration, Bootstrap

    5 files  ±0      5 suites  ±0   2h 38m 36s ⏱️ + 1m 2s
5 252 tests ±0  4 737 ✅ ±0  515 💤 ±0  0 ❌ ±0 
5 258 runs  ±0  4 737 ✅ ±0  521 💤 ±0  0 ❌ ±0 

Results for commit 412623c. ± Comparison against base commit 06df84d.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Oct 22, 2025 •
edited
Loading

Copy link
Copy Markdown

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   2h 0m 2s ⏱️ -6s
4 878 tests ±0  4 523 ✅ ±0  355 💤 ±0  0 ❌ ±0 
4 880 runs  ±0  4 523 ✅ ±0  357 💤 ±0  0 ❌ ±0 

Results for commit 412623c. ± Comparison against base commit 06df84d.

♻️ This comment has been updated with latest results.

@baermat baermat marked this pull request as ready for review October 22, 2025 13:22
viren-nadkarni
viren-nadkarni approved these changes Oct 22, 2025
View reviewed changes

@viren-nadkarni viren-nadkarni 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.

I strongly suggest adding a unit-test that covers this new argument. Looks good otherwise!

@baermat baermat merged commit 2467f0d into main Oct 23, 2025
42 checks passed
@baermat baermat deleted the iot/fix-group-names branch October 23, 2025 07:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@viren-nadkarni viren-nadkarni viren-nadkarni approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

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

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@baermat @viren-nadkarni