Skip to content

wait for control plane node to become ready after joining#12794

Merged
k8s-ci-robot merged 1 commit intokubernetes-sigs:masterfrom
VannTen:fix/join_new_control_plane
Jan 28, 2026
Merged

wait for control plane node to become ready after joining#12794
k8s-ci-robot merged 1 commit intokubernetes-sigs:masterfrom
VannTen:fix/join_new_control_plane

Conversation

@VannTen
Copy link
Copy Markdown
Contributor

@VannTen VannTen commented Dec 16, 2025

What type of PR is this?
/kind bug

What this PR does / why we need it:
When joining a control plane node and "upgrading" the cluster setup (for
example, to update etcd addresses after adding a new etcd) in the same
playbook run, the node can take a bit of time to become ready after
joining.
This triggers a kubeadm preflight check (ControlPlaneNodesReady) in
kubeadm upgrade, which is run directly after the join tasks.

Add a configurable wait for the control plane node to become Ready to
fix this race condition.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:
Found that while migrating my clusters with #12610

Does this PR introduce a user-facing change?:

Add a wait after control plane joining to prevent a kubeadm preflight error when upgrading the control plane configuration at the same time.

@k8s-ci-robot k8s-ci-robot added kind/bug Categorizes issue or PR as related to a bug. release-note-none Denotes a PR that doesn't merit a release note. labels Dec 16, 2025
@k8s-ci-robot k8s-ci-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Dec 16, 2025
@VannTen
Copy link
Copy Markdown
Contributor Author

VannTen commented Dec 16, 2025

/cherrypick release-2.27
/cherrypick release-2.28
/cherrypick release-2.29

@k8s-infra-cherrypick-robot
Copy link
Copy Markdown

@VannTen: once the present PR merges, I will cherry-pick it on top of release-2.27, release-2.28, release-2.29 in new PRs and assign them to you.

Details

In response to this:

/cherrypick release-2.27
/cherrypick release-2.28
/cherrypick release-2.29

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@k8s-ci-robot k8s-ci-robot added release-note Denotes a PR that will be considered when it comes time to generate release notes. and removed release-note-none Denotes a PR that doesn't merit a release note. labels Dec 16, 2025
@VannTen
Copy link
Copy Markdown
Contributor Author

VannTen commented Dec 16, 2025

/retest

@VannTen VannTen force-pushed the fix/join_new_control_plane branch from 8f37342 to bc2f466 Compare December 16, 2025 10:58
When joining a control plane node and "upgrading" the cluster setup (for
example, to update etcd addresses after adding a new etcd) in the same
playbook run, the node can take a bit of time to become ready after
joining.
This triggers a kubeadm preflight check (ControlPlaneNodesReady) in
kubeadm upgrade, which is run directly after the join tasks.

Add a configurable wait for the control plane node to become Ready to
fix this race condition.
@VannTen VannTen force-pushed the fix/join_new_control_plane branch from bc2f466 to b6df8ad Compare December 16, 2025 11:01
@VannTen
Copy link
Copy Markdown
Contributor Author

VannTen commented Jan 27, 2026

/cc @tico88612

Copy link
Copy Markdown
Member

@tico88612 tico88612 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jan 28, 2026
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: tico88612, VannTen

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot merged commit fcecaf6 into kubernetes-sigs:master Jan 28, 2026
47 checks passed
@k8s-infra-cherrypick-robot
Copy link
Copy Markdown

@VannTen: new pull request created: #12922

Details

In response to this:

/cherrypick release-2.27
/cherrypick release-2.28
/cherrypick release-2.29

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@k8s-infra-cherrypick-robot
Copy link
Copy Markdown

@VannTen: new pull request created: #12923

Details

In response to this:

/cherrypick release-2.27
/cherrypick release-2.28
/cherrypick release-2.29

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@k8s-infra-cherrypick-robot
Copy link
Copy Markdown

@VannTen: new pull request created: #12924

Details

In response to this:

/cherrypick release-2.27
/cherrypick release-2.28
/cherrypick release-2.29

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

jaeyung1001 pushed a commit to jaeyung1001/kubespray that referenced this pull request Apr 11, 2026
…-sigs#12794)

When joining a control plane node and "upgrading" the cluster setup (for
example, to update etcd addresses after adding a new etcd) in the same
playbook run, the node can take a bit of time to become ready after
joining.
This triggers a kubeadm preflight check (ControlPlaneNodesReady) in
kubeadm upgrade, which is run directly after the join tasks.

Add a configurable wait for the control plane node to become Ready to
fix this race condition.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/bug Categorizes issue or PR as related to a bug. lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants