Skip to content

Add LocalStack relevant services to whitelist#113

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

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

Conversation

@viren-nadkarni
Copy link
Copy Markdown
Member

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

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
Copy link
Copy Markdown
Member

@giograno giograno left a comment

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