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

Skip kube-api-access Volume Comparison #550

Merged
merged 2 commits into from
Apr 28, 2021
Merged

Skip kube-api-access Volume Comparison #550

merged 2 commits into from
Apr 28, 2021

Conversation

prryb
Copy link
Collaborator

@prryb prryb commented Apr 28, 2021

Hotfix for the incompatibility between the Operator and Kubernetes 1.21.

Kubernetes in 1.21 ships with BoundServiceAccountTokenVolume enabled, which causes the service account admission controller to add a new volume: kube-api-access-<random-suffix>. That is causing the operator to loop over comparing the actual and expected volumes. Now that new volume will not be compared.

Kubernetes reference: https://kubernetes.io/docs/reference/access-authn-authz/service-accounts-admin/#bound-service-account-token-volume

This hotfixes the incompatibility between the Operator and Kubernetes
1.21.

Kubernetes 1.21 started adding a new volume named
"kube-api-access-<random-suffix>" and that broke the comparison
function and resulted in an infinite loop.
@prryb prryb added the bug Something isn't working label Apr 28, 2021
@KorusMateusz KorusMateusz self-requested a review April 28, 2021 12:03
@prryb prryb merged commit 223119b into jenkinsci:master Apr 28, 2021
@prryb prryb deleted the hotfix-k8s-1.21 branch April 28, 2021 12:23
@prryb prryb added this to the 0.6.0 milestone May 4, 2021
jsanchez034 pushed a commit to animoto/kubernetes-operator that referenced this pull request May 20, 2024
* Skip kube-api-access volume comparison

This hotfixes the incompatibility between the Operator and Kubernetes
1.21.

Kubernetes 1.21 started adding a new volume named
"kube-api-access-<random-suffix>" and that broke the comparison
function and resulted in an infinite loop.
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
None yet
Development

Successfully merging this pull request may close these issues.

3 participants