-
Notifications
You must be signed in to change notification settings - Fork 811
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
Update kind dev tooling to 1.12 #896
Conversation
63d5aa0
to
ed88749
Compare
Build Succeeded 👏 Build Id: 34d2cb37-2d85-46d5-995e-abc8d3d58d9a The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
Build Succeeded 👏 Build Id: 3d16210d-d1f4-4953-9de3-ae4f839d4407 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
ed88749
to
bb72205
Compare
Build Succeeded 👏 Build Id: 96f0e785-2388-45e3-8ce2-d71695180b5b The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
I'll take this for a spin, but @Kuqd did you also want to look, since KinD was your baby? |
bb72205
to
a6ff53b
Compare
Build Failed 😱 Build Id: 6e421a66-fbc8-4cf1-a1b5-a2160b16ae54 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Failed 😱 Build Id: 75fe2dbf-32ea-413c-8ec4-f1e16b10e1fe To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Succeeded 👏 Build Id: 7d5bdb0c-64fe-44dd-a385-294c8fc781da The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
a6ff53b
to
87522c2
Compare
Build Succeeded 👏 Build Id: 9714356e-fc1e-4858-b596-15931cfa6fe3 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
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, I have another fix in my branch for the allocator to run as nodeport instead of loadbalancer, possibly in 1.12.9 loadbalancer service type is supported.
Kind does not appear to have support for type loadbalancer (see kubernetes-sigs/kind#411) so it'd be great to get your change upstreamed. |
87522c2
to
20a269f
Compare
Build Succeeded 👏 Build Id: 4868f812-d450-4c8c-a16d-b4bb8d991328 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
Build Succeeded 👏 Build Id: 9fdd5d86-d699-42ef-bdb3-e022d24870e3 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
Part of #717.
A couple of notes:
go get
install instructions didn't work for me, so instead of trying to keep our instructions up to date I just replaced it with a link to the Kind docs (it turns out they switched to go modules, which was why).