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 check for SA to client if not found in cache #178

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jsilverio22
Copy link

@jsilverio22 jsilverio22 commented Mar 3, 2023

Issue #, if available:
#174

Description of changes:
Trying to solve a race condition in aws-identity pod where serviceaccount and pod get created at the same time. It uses cache to check SA which might not be ready when the pod was created. In order to solve this if a SA is not found in the cache but is expected do not allow the request through so it gets retried later.

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

Signed-off-by: Joshua Silverio <joshua@acorn.io>
@jsilverio22 jsilverio22 marked this pull request as ready for review March 6, 2023 22:13
@jsilverio22 jsilverio22 requested a review from a team as a code owner March 6, 2023 22:13
@jessebye
Copy link

@jsilverio22 We're encountering this issue in our clusters and wondering if this fix could get merged. Have you heard from any maintainers on this?

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.

None yet

2 participants