Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix account bootstrap on OU move #539

Merged
merged 1 commit into from
Oct 7, 2022

Conversation

sbkok
Copy link
Collaborator

@sbkok sbkok commented Oct 4, 2022

Why?

When you move an account from the root to an organization unit (OU), an event is emitted that should trigger the Account Bootstrap state machine.

However, the task role that was configured did not have the necessary permissions to invoke the state machine.

What?

A new role is created that will enable the event to trigger the Account Bootstrap state machine.

While investigating the issue, it was also noticed that the original task role had permissions that were no longer required. Those stale permissions are removed too.


By submitting this pull request, I confirm that you can use, modify, copy, and
redistribute this contribution, under the terms of your choice.

**Why?**

When you move an account from the root to an OU, an event is emitted that should
trigger the Account Bootstrap state machine.

However, the task role that was configured did not have the necessary
permissions to invoke the state machine.

**What?**

A new role is created that will enable the event to trigger the Account
Bootstrap state machine.

While investigating the issue, it was also noticed that the original task role
had permissions that were no longer required. Those stale permissions are
removed too.
@sbkok sbkok added the bug Something isn't working label Oct 4, 2022
@sbkok sbkok added this to the v3.2.0 milestone Oct 4, 2022
@sbkok sbkok merged commit 9e1c0a2 into awslabs:master Oct 7, 2022
@sbkok sbkok deleted the fix/account-bootstrapping-on-ou-move branch October 7, 2022 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants