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

Treat CustomKubeletIdentityMissingPermissionError as retryable #3286

Merged
merged 4 commits into from
Sep 13, 2023

Conversation

theunrepentantgeek
Copy link
Member

What this PR does / why we need it:

Treat the error CustomKubeletIdentityMissingPermissionError as a retryable error, to avoid a race condition where slow creation of a UserAssignedIdentity might cause creation of a ManagedCluster to fail with a permanent error.

Closes #3267

Special notes for your reviewer:

How does this PR make you feel:
gif

@codecov-commenter
Copy link

codecov-commenter commented Sep 12, 2023

Codecov Report

Merging #3286 (0354441) into main (fcb5a90) will increase coverage by 0.03%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #3286      +/-   ##
==========================================
+ Coverage   54.48%   54.52%   +0.03%     
==========================================
  Files        1482     1482              
  Lines      633430   633443      +13     
==========================================
+ Hits       345141   345363     +222     
+ Misses     231667   231455     -212     
- Partials    56622    56625       +3     
Files Changed Coverage Δ
...rvice/customizations/managed_cluster_extensions.go 54.43% <0.00%> (-10.73%) ⬇️

... and 27 files with indirect coverage changes

@theunrepentantgeek theunrepentantgeek merged commit 7bc5135 into main Sep 13, 2023
@theunrepentantgeek theunrepentantgeek deleted the feature/managed-cluster-retry branch September 13, 2023 22:24
nibooy pushed a commit to nibooy/azure-service-operator that referenced this pull request Sep 22, 2023
…#3286)

* Treat CustomKubeletIdentityMissingPermissionError as retryable
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
Development

Successfully merging this pull request may close these issues.

Have ManagedClusters reconcile again when hitting error code CustomKubeletIdentityMissingPermissionError
3 participants