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

Add autopilot instructions to doc as Alpha #3004

Merged
merged 2 commits into from
Mar 4, 2023

Conversation

shannonxtreme
Copy link
Contributor

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

  • Need the correct Agones release version for the publishVersion widget
  • Need reviews around style
  • This PR doesnt clean up older branding and UI instructions on the page (but might be a followup)

/kind documentation
/cc @zmerlynn @markmandel

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: affe623d-371f-4ffe-86a0-1294ebbb0e1e

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@shannonxtreme shannonxtreme force-pushed the autopilot-alpha branch 2 times, most recently from 0ea3e62 to 349e907 Compare March 1, 2023 17:52
@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 3283f84f-b88a-4f7a-8847-c65110a5cdc0

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@shannonxtreme
Copy link
Contributor Author

Is it my link structure?

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 5a9c93a7-c031-4496-b2a0-451d076c4ee8

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@zmerlynn
Copy link
Collaborator

zmerlynn commented Mar 1, 2023

@shannonxtreme I think it's an improper ref?

Error: Error building site: "/go/src/agones.dev/agones/site/content/en/docs/Installation/Creating Cluster/gke.md:107:39": failed to render shortcode "relref": failed to process shortcode: template: _internal/shortcodes/relref.html:1:3: executing "_internal/shortcodes/relref.html" at <relref . .Params>: error calling relref: invalid number of arguments to ref

@shannonxtreme shannonxtreme force-pushed the autopilot-alpha branch 2 times, most recently from a3daef4 to 9104bd5 Compare March 1, 2023 19:28
@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 2f738820-2186-4413-9548-0e5a5f827535

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 36e32383-5650-4155-b950-4c64a5ab6e40

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: b64b1a08-4f4e-48d3-8fcb-a42da70c75ea

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@agones-bot
Copy link
Collaborator

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:

  • git fetch https://github.com/googleforgames/agones.git pull/3004/head:pr_3004 && git checkout pr_3004
  • helm install agones ./install/helm/agones --namespace agones-system --agones.image.release=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.30.0-6bdd1b3-amd64

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 4ac89402-701d-437a-9838-6f82639c65a9

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@shannonxtreme shannonxtreme force-pushed the autopilot-alpha branch 2 times, most recently from 9371344 to a5a9280 Compare March 2, 2023 17:43
@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 87ce1801-9f2f-4414-b3ee-c8c1303cf5a9

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 19bc547e-0ad1-4b5f-8b05-50c244100990

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: d67baf84-4c69-40d9-96e4-2861ae3254e1

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@zmerlynn
Copy link
Collaborator

zmerlynn commented Mar 3, 2023

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">}}).
Copy link
Collaborator

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.

Copy link
Contributor Author

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.
Copy link
Collaborator

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.

Copy link
Contributor Author

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

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed

Copy link
Collaborator

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.

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 1b7f2e00-c41a-443f-8054-0c7189028453

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@google-oss-prow
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: shannonxtreme, zmerlynn
Once this PR has been reviewed and has the lgtm label, please assign ericfortin for approval. For more information see the Kubernetes Code Review Process.

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@google-oss-prow
Copy link

New changes are detected. LGTM label has been removed.

@zmerlynn zmerlynn enabled auto-merge (squash) March 3, 2023 21:25
@agones-bot
Copy link
Collaborator

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:

  • git fetch https://github.com/googleforgames/agones.git pull/3004/head:pr_3004 && git checkout pr_3004
  • helm install agones ./install/helm/agones --namespace agones-system --agones.image.release=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.31.0-a640f1a-amd64

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 0e6a3b12-92c8-4323-b1bf-fc811ede27ea

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@agones-bot
Copy link
Collaborator

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:

  • git fetch https://github.com/googleforgames/agones.git pull/3004/head:pr_3004 && git checkout pr_3004
  • helm install agones ./install/helm/agones --namespace agones-system --agones.image.release=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.31.0-7b8f739-amd64

@zmerlynn zmerlynn merged commit 5face54 into googleforgames:main Mar 4, 2023
@Kalaiselvi84 Kalaiselvi84 added the kind/documentation Documentation for Agones label Apr 10, 2023
@Kalaiselvi84 Kalaiselvi84 added this to the 1.31.0 milestone Apr 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/documentation Documentation for Agones size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants