Skip to content

Commit

Permalink
fix: use 'control-plane' Kubernetes node label
Browse files Browse the repository at this point in the history
Talos was labeling control plane nodes with both `control-plane` and
`master` labels since Talos 0.8, so it should be safe to do that change.

Talos 1.2.0 is supposed to drop legacy `master` label, so we need to
make sure CACCPT is ready.

Signed-off-by: Andrey Smirnov <[email protected]>
  • Loading branch information
smira committed Aug 2, 2022
1 parent ac90f86 commit 6fdde72
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion controllers/taloscontrolplane_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -552,7 +552,7 @@ func (r *TalosControlPlaneReconciler) updateStatus(ctx context.Context, tcp *con
}

nodeSelector := labels.NewSelector()
req, err := labels.NewRequirement(constants.LabelNodeRoleMaster, selection.Exists, []string{})
req, err := labels.NewRequirement(constants.LabelNodeRoleControlPlane, selection.Exists, []string{})
if err != nil {
return err
}
Expand Down

0 comments on commit 6fdde72

Please sign in to comment.