Skip to content

Add LocalStack relevant services to whitelist#113

Merged
viren-nadkarni merged 5 commits into
localstackfrom
whitelist
Feb 19, 2026
Merged

Add LocalStack relevant services to whitelist#113
viren-nadkarni merged 5 commits into
localstackfrom
whitelist

Conversation

@viren-nadkarni

@viren-nadkarni viren-nadkarni commented Feb 17, 2026

Copy link
Copy Markdown
Member

This PR whitelists services that are relevant to LocalStack. Attempting to use other services will raise an error. Alongside this, tests for these non-whitelisted services are also disabled.

This is being done for following reasons:

  • Tests in CI have been failing for a service we don't care about. We don't want to spend time investigating it because it's not being used by LocalStack
  • CI runs faster by ~33%

For testing, the development release 5.1.24.dev0 made from this branch was tested against LocalStack in following runs to make sure that there are no instances of ServiceNotWhitelisted exceptions:

  • AWS / Community Integration Tests against Pro # 801
  • AWS / Pro Integration Tests # 4149
  • AWS / Integration Tests # 14

See PNX-637

@viren-nadkarni viren-nadkarni self-assigned this Feb 17, 2026
@viren-nadkarni
viren-nadkarni force-pushed the whitelist branch 11 times, most recently from 82a51bb to 9c5ce7c Compare February 18, 2026 11:42
@viren-nadkarni viren-nadkarni changed the title Add relevant services to whitelist Add LocalStack relevant services to whitelist Feb 19, 2026
@viren-nadkarni
viren-nadkarni marked this pull request as ready for review February 19, 2026 06:58

@giograno giograno 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.

💯

@viren-nadkarni
viren-nadkarni merged commit 875938a into localstack Feb 19, 2026
74 checks passed
@viren-nadkarni
viren-nadkarni deleted the whitelist branch February 19, 2026 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants