-
Notifications
You must be signed in to change notification settings - Fork 810
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
Add autopilot instructions to doc as Alpha #3004
Conversation
Build Failed 😱 Build Id: affe623d-371f-4ffe-86a0-1294ebbb0e1e To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
0ea3e62
to
349e907
Compare
Build Failed 😱 Build Id: 3283f84f-b88a-4f7a-8847-c65110a5cdc0 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Is it my link structure? |
Build Failed 😱 Build Id: 5a9c93a7-c031-4496-b2a0-451d076c4ee8 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
@shannonxtreme I think it's an improper ref?
|
a3daef4
to
9104bd5
Compare
Build Failed 😱 Build Id: 2f738820-2186-4413-9548-0e5a5f827535 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Failed 😱 Build Id: 36e32383-5650-4155-b950-4c64a5ab6e40 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
9104bd5
to
d7bd68d
Compare
Build Failed 😱 Build Id: b64b1a08-4f4e-48d3-8fcb-a42da70c75ea To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
d7bd68d
to
6bdd1b3
Compare
Build Succeeded 👏 Build Id: 2ea894c8-a729-4fdd-996f-bca975a2a61f 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:
|
6bdd1b3
to
af2cc9b
Compare
Build Failed 😱 Build Id: 4ac89402-701d-437a-9838-6f82639c65a9 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
9371344
to
a5a9280
Compare
Build Failed 😱 Build Id: 87ce1801-9f2f-4414-b3ee-c8c1303cf5a9 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Failed 😱 Build Id: 19bc547e-0ad1-4b5f-8b05-50c244100990 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
98a7a81
to
8317a85
Compare
Build Failed 😱 Build Id: d67baf84-4c69-40d9-96e4-2861ae3254e1 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
This LGTM - if you can address @markmandel's comments, I'll give it another pass and merge. |
[GKE release channel](https://cloud.google.com/kubernetes-engine/docs/release-notes#current_versions) | ||
for the cluster. Can be one of `rapid`, `regular`, or `stable`. The default | ||
is `regular`. The release channel must use a | ||
[supported Kubernetes version]({{<ref "/docs/Installation#usage-requirements">}}). |
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.
Should we note here that even though the channel has a default, other versions are available with --cluster-version
?
The annoying thing is I can't find a page that has the equivalent data to go/gke-versions. You can infer it from channel release notes on https://cloud.google.com/kubernetes-engine/docs/release-notes, but there's not a simple table. And on Autopilot the version doesn't show up in Pantheon. So I'm not sure what to link to outside of gcloud container get-server-config --zone us-west1
, which is.. not super friendly.
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.
Played with gcloud a bit and you can run gcloud container get-server-config --region=us-central1 --flatten=channels --filter="(channels.channel~REGULAR)" --format="yaml(channels)"
which is neater
In Autopilot clusters, GKE manages the nodes, so you won't create dedicated | ||
node pools for Agones system components or for metrics collection. The node | ||
affinity rules in the Agones installation manifests tell GKE to separate system | ||
and metrics Pods onto dedicated nodes automatically. |
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.
The node
affinity rules in the Agones installation manifests tell GKE to separate system
and metrics Pods onto dedicated nodes automatically.
This part is opposite of what I intended in the UG as well. I might just leave this part out for now - I'm really thinking we need a guide page rather than dumping all of this on the install page, and I have some stuff to say about separation at a greater depth, but it seems cluttery to put it here.
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.
Does that effect not happen then? I noticed the node tolerations in the manifests
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.
Removed
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.
The problem is that for pod anti-affinity, you have to have >0.25 vCPU pods, so I backed off on using them in general. The tolerations are still there, I think, but I don't think they're doing anything right now.
8317a85
to
45a0e11
Compare
Build Failed 😱 Build Id: 1b7f2e00-c41a-443f-8054-0c7189028453 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
45a0e11
to
322ff3e
Compare
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: shannonxtreme, zmerlynn The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
c998ffb
to
a640f1a
Compare
New changes are detected. LGTM label has been removed. |
Build Succeeded 👏 Build Id: 86ebbc83-dd2b-4629-a3fe-981537e70888 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:
|
a640f1a
to
7b8f739
Compare
Build Failed 😱 Build Id: 0e6a3b12-92c8-4323-b1bf-fc811ede27ea To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Succeeded 👏 Build Id: 4dd0d4d5-8770-4338-aa1c-29c3ff56454d 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:
|
Add alpha stage instructions for GKE autopilot in the GKE getting started doc.
Based on the discussion in #2985, which I'll close for now.
Reviewer notes
/kind documentation
/cc @zmerlynn @markmandel