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

The "Launching Kubernetes on Windows Clusters" documentation should be revamped #1074

Open
kourosh7 opened this issue Jan 19, 2024 · 5 comments · May be fixed by #1306
Open

The "Launching Kubernetes on Windows Clusters" documentation should be revamped #1074

kourosh7 opened this issue Jan 19, 2024 · 5 comments · May be fixed by #1306

Comments

@kourosh7
Copy link
Member

kourosh7 commented Jan 19, 2024

Summary

The Launching Kubernetes on Windows Clusters documentation should be revamped. It seems to have outdated info that was leftover from provisioning RKE1 Windows clusters which is now EOL.

Here are some items that I noticed are outdated/incorrect after provisioning a supported WIndows RKE2 cluster through Rancher:

  1. The first section on the page mentions RKE but it should be RKE2 since RKE1 Windows Clusters are EOL.
  2. The first section says, You can only add Windows nodes to a cluster if Windows support is enabled when the cluster is created. Windows support cannot be enabled for existing clusters. However, this is no longer the case on Windows RKE2 clusters.
  3. The first section says, Kubernetes 1.15+ is required. This should be updated to a supported K8s version
  4. The next session is titled: OS and Docker Requirements but Docker is not used in RKE2
  5. The Networking Requirements section says Rancher only supports Windows using Flannel as the network provider. But it is actually calico in RKE2
  6. The Tutorial: How to Create a Cluster with Windows Support says When you create or edit your cluster from the Rancher UI, you will see a Customize Node Run Command that you can run on each server to add it to your cluster. That has changed to the Registration Command
  7. In section 2. Create the Cluster on Existing Nodes, steps 5,6,7,8 are incorrect and should mention a supported K8s version, Calico instead of Flannel, and there is no longer an extra option to enable Windows Support or choose a Flannel Backend
  8. Section 3. Add Nodes to the Cluster should be updated for all three subsections (Add Linux Master Node, Add Linux Worker Node, Add Windows Worker Node). Once the cluster is created from the Cluster Configuration page, you will then have a Registration section which will allow you to select Node Roles in the Step 1 section. Then the Step 2 section will have the Registration Command you run on the nodes. The registration command for the Windows node will become available when the cluster has working Linux nodes with etcd, control plane, and worker roles active.
@btat
Copy link
Contributor

btat commented Feb 1, 2024

Possibly related to #997

@sunilarjun
Copy link
Contributor

Hello @kourosh7! I had some questions regarding the page refresh with respect to RKE2:

  1. In the Kubernetes Version section where it states information on v1.15+/v1.21 K8s requirements, are these still needed here or can they be replaced with the support matrix for RKE2 for relevant Kubernetes version info?
  2. Are these descriptions about the two network options relevant here or are they more RKE1 material that can be removed?

Any guidance much appreciated, thank you!

@kourosh7
Copy link
Member Author

kourosh7 commented Jun 3, 2024

Hi @sunilarjun, sorry for the late reply.

  1. Not needed, I think it would be best to point to the support matrix
  2. Also not needed, those options link to flannel docs, and for rke2 we use calico

@kourosh7
Copy link
Member Author

kourosh7 commented Jun 3, 2024

For reference, you can see this RKE2 custom cluster created in Rancher using the flannel CNI does not give an option to register a Windows node:
image
image

Whereas, in this RKE2 custom cluster created in Rancher where calico is the CNI, you can clearly see an option to register a Windows node:
image

@sunilarjun
Copy link
Contributor

@kourosh7 Thank you for the response! I've adjusted my PR accordingly with the clarifications.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants