Skip to content

Introduce autorolestargetname to allow for custom target IAM roles#15

Open
nmasur wants to merge 1 commit intoassetnote:masterfrom
nmasur:master
Open

Introduce autorolestargetname to allow for custom target IAM roles#15
nmasur wants to merge 1 commit intoassetnote:masterfrom
nmasur:master

Conversation

@nmasur
Copy link

@nmasur nmasur commented Nov 16, 2023

Instead of hardcoding GhostBusterTargetAccountRole, allow for an optional role name. This is helpful for Control Tower users, since all accounts include the aws-controltower-ReadOnlyExecutionRole role for the Audit account.

Fixes #14 .

I haven't spent much time on this pull request. I just came up with something and introduced it as an example.

Instead of hardcoding GhostBusterTargetAccountRole, allow for an
optional role name. This is helpful for Control Tower users, since all
accounts include the aws-controltower-ReadOnlyExecutionRole.
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.

Hardcoded GhostbusterTargetAccountRole makes it difficult for Control Tower users

1 participant