-
Notifications
You must be signed in to change notification settings - Fork 164
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
AKS - Error while updating taint on the cluster #816
Comments
Thanks @andresvasile.
Not aware of it, thanks for lettings us know.
Strange that they wouldn't allow a daemonset to set the labels though, I believe it's a fairly common practice. When running the FYI, the |
Sure, here is the full log:
|
@ctalledo for now I'm using this particular command to bypass the validation:
but I'm not really sure what concerns or implications could bring it.. |
Thanks @andresvasile for the work-around above to install Sysbox on AKS. I guess we could improve the sysbox-deploy-k8s daemonset to detect it's installing on AKS and try to label the node, but if it's unsuccessful (e.g., because AKS restricts this), then just move on instead of failing. IIRC, sysbox-deploy-k8s uses those labels to keep track of it's progress and it installs Sysbox on the K8s cluster, as well as to prevent sysbox containers from getting deployed on a node until Sysbox is actually installed and running on it. |
Hello guys I'm trying to install Sysbox in my AKS cluster and I got this error:
Error from server: admission webhook "aks-node-validating-webhook.azmk8s.io" denied the request: (UID: 9a9f6a3d-c7c3-4dcc-b73b-a22885904ebe) Label update/add request "sysbox-runtime:installing" refused. User is attempting to update/add a label configured on node pool. Please change node pool label configuration through Azure..
I saw different issues that mention that AKS modified their workaround, and it is no longer feasible to update the taints.
I would like to know if you are aware of this or if it's a different workaround to manage Sysbox in AKS
cc: @ctalledo
Thanks.
The text was updated successfully, but these errors were encountered: