fix: support aws regions#12
Conversation
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
There was a problem hiding this comment.
Pull request overview
Adds support for configuring AWS regions for Cloud Custodian execution, allowing operators to narrow regional scans instead of always using --region all.
Changes:
- Introduces
aws_regionsplugin configuration, parsed into a deduplicated[]stringinParsedConfig. - Threads configured AWS regions through execution requests and appends
--region <value>flags for AWS checks (defaulting toallwhen unset). - Extends config parsing unit test coverage for the new
aws_regionsfield.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| main.go | Adds aws_regions config parsing, plumbs it into execution requests, and updates custodian CLI arg construction for AWS checks. |
| main_test.go | Updates config parse test to assert region list parsing/deduplication behavior. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.