-
Notifications
You must be signed in to change notification settings - Fork 26
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
CPEM requires Kubernetes node name to match Equinix Metal device name #533
Comments
You shouldn't be setting providerID, that's something CPEM sets for you. Why it's not setting it here though, that's the real question. Hmm. We had this part working in the work we did before kubecon, do you still have access to that config? Probably something we had to disable on the talos side. |
It should be noted that it's also not clearing a taint I suspect it's responsible for: |
I have another open issue related to the /healthz check: #519 |
Lively conversation happing in #support channel on Talos / Sidero slack: https://taloscommunity.slack.com/archives/CMARMBC4E/p1712793108556169 Seems it might be related to the deviceByName function fallback wanting the kubernetes node names to match the Equinix devices names exactly. Possibly? https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/blob/main/metal/devices.go#L165-L167 |
Going to try setting the |
This fixes kubernetes-sigs/cloud-provider-equinix-metal#533 // deviceByName returns an instance whose hostname matches the kubernetes node.Name Defined here : https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/blob/main/metal/devices.go#L165C1-L166C1 The reason it fixes it is the logic in CPEM deviceByName requires the equinix metal device name match the kubernetes node name in order for eip_controlplane_reconciliation to complete.
I found a work around, but it was a bit difficult to find. I might be a one-off, but it might make sense to take some steps to raise visibility so others don't get stuck on this in the future:
|
I'm not sure where to set providerID. I don't remember setting it in the past. Any suggestions?
CPEM daemonset
cloud-sa.json
CPEM logs
The text was updated successfully, but these errors were encountered: