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

[v15] fix: prevent deleting AWS OIDC integration used by EAS #40851

Merged
merged 3 commits into from Apr 24, 2024

Conversation

nklaassen
Copy link
Contributor

Backport #40630 to branch/v15

changelog: prevent deleting AWS OIDC integration used by External Audit Storage.

@nklaassen
Copy link
Contributor Author

nklaassen commented Apr 23, 2024

I still need to add the condition to check that auth servers on an older minor/patch version are not holding the lock #40630 (comment)

edit: done in 4a60e0d

@nklaassen
Copy link
Contributor Author

I will update the e ref to include https://github.com/gravitational/teleport.e/pull/4018 before merging this

The External Audit Storage feature for Teleport Cloud depends on an AWS
OIDC integration to exist in order to get credentials for the customer
AWS account.
It's currently possible for a customer to delete the AWS OIDC
integration referenced by the EAS configuration, leading to the EAS
integration breaking and the Auth service crashing the next time it's
started up.

This commit prevents deleting an AWS OIDC integration currently
referenced by and EAS integration, and prevents creating an EAS
integration referencing an unknown AWS OIDC integration.
@nklaassen nklaassen force-pushed the bot/backport-40630-branch/v15 branch from 4a60e0d to 23fcd11 Compare April 24, 2024 22:53
@nklaassen nklaassen added this pull request to the merge queue Apr 24, 2024
Merged via the queue into branch/v15 with commit a8a2017 Apr 24, 2024
35 checks passed
@nklaassen nklaassen deleted the bot/backport-40630-branch/v15 branch April 24, 2024 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants