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
Windows support via kubeadm #1036
Windows support via kubeadm #1036
Conversation
ad99bdb
to
9cdef8a
Compare
is MachinePool VMSS support non-goal/future work? |
bd8b581
to
41be801
Compare
I took a look and VMSS was pretty straight forward to add support. |
45f5dc4
to
7ff1115
Compare
69fa7f5
to
27bc104
Compare
/retest |
c41fc67
to
e6de58d
Compare
/retest |
e6de58d
to
3f51c93
Compare
/test pull-cluster-api-provider-azure-verify |
/test pull-cluster-api-provider-azure-e2e-windows |
/test pull-cluster-api-provider-azure-e2e |
1 similar comment
/test pull-cluster-api-provider-azure-e2e |
The e2e job failed for different set of tests each time /test pull-cluster-api-provider-azure-e2e |
787f1fe
to
fe5aaba
Compare
created #1095 for further investigation |
fe5aaba
to
507da8b
Compare
/test pull-cluster-api-provider-azure-e2e-windows |
looks like there was a flake grabbing the kubectl version
I don't believe that is the correct version that is needed since our tests run 1.19.4. I will follow up with an issue/pr to fix that. /test pull-cluster-api-provider-azure-e2e-windows |
This PR looks great. I'd love to get it pulled in sooner than later, but I think we should hold off until after the holiday to make sure everyone gets a final review. Awesome work, @jsturtevant! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
thanks for this! 🎉
Nice :) /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: CecileRobertMichon The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #153
Implements https://github.com/kubernetes-sigs/cluster-api/blob/master/docs/proposals/20200804-windows-support.md
Special notes for your reviewer:
Uses an image from kubernetes-sigs/image-builder#382. This PR is close to merging and then this can go in.
Similar to #1035
Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.
TODOs:
Release note: