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.

Sign up for GitHub

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

Jump to bottom

Build kubeadm for Windows Server #364

Closed
Techwolfy opened this issue Jul 31, 2017 · 3 comments · Fixed by kubernetes/kubernetes#53553
Closed

Build kubeadm for Windows Server #364

Techwolfy opened this issue Jul 31, 2017 · 3 comments · Fixed by kubernetes/kubernetes#53553
Labels
area/releasing priority/backlog Higher priority than priority/awaiting-more-evidence.
Milestone
v1.9

Comments

@Techwolfy
Copy link

Is this a BUG REPORT or FEATURE REQUEST?

FEATURE REQUEST

Versions

kubeadm version (use kubeadm version): kubeadm version: &version.Info{Major:"1", Minor:"7", GitVersion:"v1.7.2", GitCommit:"922a86cfcd65915a9b2f69f3f193b8907d741d9c", GitTreeState:"clean", BuildDate:"2017-07-21T08:08:00Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}

Environment:

  • Kubernetes version (use kubectl version): Client Version: version.Info{Major:"1", Minor:"7", GitVersion:"v1.7.2", GitCommit:"922a86cfcd65915a9b2f69f3f193b8907d741d9c", GitTreeState:"clean", BuildDate:"2017-07-21T08:23:22Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}
  • Cloud provider or hardware configuration: Hyper-V and bare metal
  • OS (e.g. from /etc/os-release): Ubuntu 16.04.2 LTS, Windows Server 2016
  • Kernel (e.g. uname -a): 4.4.0-87-generic
  • Others: N/A

What happened?

The documentation indicates that kubeadm is the recommended tool to use for creating a new Kubernetes cluster, and it seems to be working fine on Linux despite the beta warnings. However, there isn't currently a Windows build for the tool, so I can't connect Windows nodes to the cluster. It seems like this should be possible if I write a script to run the join steps of https://kubernetes.io/docs/admin/kubeadm/, but it would be a lot easier (and far more user-friendly) if I could use kubeadm join on Windows as well.

Ideally kubeadm will become entirely cross-platform in the future, but as Windows control planes aren't supported yet that will likely take a lot more work.
@luxas
Copy link
Member

luxas commented Aug 8, 2017

cc @kubernetes/sig-release-feature-requests
Technically there is nothing stopping us from building kubeadm on Windows I think, and I'm 99.99% sure the join flow works on Windows. Only hard part is running kubelet with the right args and deploying a CNI network plugin that works both on Windows and Linux.

@luxas luxas added this to the v1.8 milestone Aug 8, 2017
@luxas luxas added area/releasing kind/enhancement priority/backlog Higher priority than priority/awaiting-more-evidence. labels Aug 8, 2017
@luxas
Copy link
Member

luxas commented Aug 23, 2017

cc @kubernetes/sig-windows-feature-requests

@luxas
Copy link
Member

luxas commented Aug 24, 2017

Talked to @michmike about this, and we're not gonna do this for v1.8
Instead we're gonna look into deeper integration possibilities for v1.9

@luxas luxas modified the milestones: v1.9, v1.8 Aug 24, 2017
@bsteciuk bsteciuk mentioned this issue Oct 6, 2017
Merged
vdemeester pushed a commit to vdemeester/kubernetes that referenced this issue Nov 1, 2017
Merge pull request kubernetes#53553 from bsteciuk/kubeadm-windows
574492a 
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Kubeadm - Added initial support for Windows worker nodes to join cluster using kubeadm

**What this PR does / why we need it**:
This PR adds initial support for adding a Windows worker node to a Kubernetes cluster with kubeadm.  Also adds Windows build of kubeadm to node build targets. 

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes kubernetes/kubeadm#364 

**Special notes for your reviewer**:

Depends on kubernetes#53730 

**Release note**:

```release-note
kubeadm: Add support for adding a Windows node
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/releasing priority/backlog Higher priority than priority/awaiting-more-evidence.
Projects
None yet
Milestone
v1.9
2 participants
@Techwolfy @luxas