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

Add a configurable delay before writing a dynamic secret #477

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

adrianmoisey
Copy link
Contributor

Fixes #271

Some services (such as AWS IAM) are eventually consistent and require some time between generating the secret, and using the secret.

Without this delay our services can't access AWS for a short while immediately after the secret rotation happens.

I'm not sure if my tests are sane. They could introduce some flakiness.

Fixes hashicorp#271

Some services (such as AWS IAM) are eventually consistent and require
some time between generating the secret, and using the secret.

Without this delay our services can't access AWS for a short while
immediately after the secret rotation happens.
@adrianmoisey adrianmoisey requested a review from a team as a code owner November 22, 2023 12:20
@adrianmoisey
Copy link
Contributor Author

Hey @benashz
Would it be possible to get this change (or something similar) looked at?

This is a blocker for us from using VSO.

@adrianmoisey
Copy link
Contributor Author

Is it possible for this change to be merged?

I'm happy to do the work to get it updated, but I'm not certain it will actually get prioritised and merged.

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.

Artificial delay for eventually consistent secrets
1 participant