Skip to content
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

docs: Explain updating k8s components. #5959

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mrichar1
Copy link
Contributor

Pull Request

What?

Explain that editing cluster config may also require a k8s-upgrade and component restart step,
with link to 'upgrading kubernetes' docs.

Updated these docs to explain that version-number can be preserved.

Signed-off-by: Matthew Richardson [email protected]

Why? (reasoning)

It is not currently clear that editing the cluster part of the machine config also requires these extra steps before the components will be running with the new config.

Acceptance

  • you generated documentation (make docs)

Explained that editing cluster config may also require a k8s-upgrade and component restart step,
with link to 'upgrading kubernetes' docs.

Updated these docs to explain that version-number can be preserved.

Signed-off-by: Matthew Richardson <[email protected]>
@@ -48,6 +48,8 @@ The list of config changes allowed to be applied immediately in Talos {{< releas
* `.machine.kernel`
* `.machine.registries` (CRI containerd plugin will not pick up the registry authentication settings without a reboot)

Changes to `.cluster` configuration for Kubernetes components will not take effect until the Kubernetes cluster is [upgraded]({{< relref "../../kubernetes-guides/upgrading-kubernetes" >}}) and the components are restarted - either manually or via a reboot.
Copy link
Member

Choose a reason for hiding this comment

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

I think we might need more careful wording here.

There are two types of components configured in .cluster section:

  • control plane components running as static pods (API server, controller manager and scheduler) - changes to these components are applied immediately on the fly
  • other settings which in fact update bootstrap manifests - manifests themselves, CNI, kube-proxy configuration - these are not applied automatically, only missing resources are created, but existing ones are not updated (and never removed) - these require a Kubernetes upgrade to be applied

Copy link

github-actions bot commented Jul 4, 2024

This PR is stale because it has been open 45 days with no activity.

@github-actions github-actions bot added the Stale label Jul 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants