Skip to content
This repository has been archived by the owner on Jun 29, 2022. It is now read-only.

MetalLB: BGP labels don't update on node replacement #1126

Closed
johananl opened this issue Oct 26, 2020 · 1 comment · Fixed by #1155
Closed

MetalLB: BGP labels don't update on node replacement #1126

johananl opened this issue Oct 26, 2020 · 1 comment · Fixed by #1155
Labels
area/components Items related to components bug Something isn't working platform/packet Packet-related size/s Issues which likely require up to a couple of work hours

Comments

@johananl
Copy link
Member

When replacing a node which runs MetalLB, the k8s node labels used for peer autodiscovery aren't being updated. The reason for this is apparently the fact that we replace the server underneath the node but keep the Node object. When the kubelet on the new machine re-registers with the API, the values in the --node-labels kubelet flag aren't used again.

Once #548 is merged we can get rid of the --node-labels workaround and instead have the Packet CCM apply (and hopefully update) annotations with BGP information.

@johananl johananl added bug Something isn't working area/components Items related to components platform/packet Packet-related size/s Issues which likely require up to a couple of work hours labels Oct 26, 2020
@invidian
Copy link
Member

When replacing a node which runs MetalLB, the k8s node labels used for peer autodiscovery aren't being updated. The reason for this is apparently the fact that we replace the server underneath the node but keep the Node object. When the kubelet on the new machine re-registers with the API, the values in the --node-labels kubelet flag aren't used again.

Hmm, nodes should de-register itself via systemd unit as part of host kubelet shutting down. Maybe this part is flaky somehow?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/components Items related to components bug Something isn't working platform/packet Packet-related size/s Issues which likely require up to a couple of work hours
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants