Skip to content

Commit

Permalink
EKS Auto Enroll: wait for agent to be installed (#48734)
Browse files Browse the repository at this point in the history
After doing this flow a couple of times, it seems that 6 seconds is not
enough.

It should not take longer than 20 seconds.
  • Loading branch information
marcoandredinis authored Nov 11, 2024
1 parent 2833437 commit 11d0ffe
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion lib/integrations/awsoidc/eks_enroll_clusters.go
Original file line number Diff line number Diff line change
Expand Up @@ -565,12 +565,13 @@ func checkAgentAlreadyInstalled(ctx context.Context, actionConfig *action.Config
var err error
// We setup a little backoff loop because sometimes access entry auth needs a bit more time to propagate and take
// effect, so we could get errors when trying to access cluster right after giving us permissions to do so.
// From real scenarios, we've seen this taking as long as 20 seconds.
for attempt := 1; attempt <= 3; attempt++ {
listCmd := action.NewList(actionConfig)
releases, err = listCmd.Run()
if err != nil {
select {
case <-time.After(time.Duration(attempt) * time.Second):
case <-time.After(time.Duration(attempt*5) * time.Second):
case <-ctx.Done():
return false, trace.NewAggregate(err, ctx.Err())
}
Expand Down

0 comments on commit 11d0ffe

Please sign in to comment.