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

Clarify docs on GKE Autopilot and node pools #4048

Merged
merged 1 commit into from
Nov 26, 2024

Conversation

danfairs
Copy link
Contributor

What this PR does / Why we need it:

/kind documentation

Which issue(s) this PR fixes:

This change makes it clearer that the cluster administrator does not need to manually configure
node pools when using GKE Autopilot.

GKE Autopilot manages node pools for you. These changes make it clearer that the steps for setting up node pools should be skipped if you are using Autopilot.
@github-actions github-actions bot added kind/documentation Documentation for Agones size/S labels Nov 25, 2024
@markmandel
Copy link
Member

/gcbrun so we can get a preview.

@agones-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: 32cb945c-e2d6-411e-ba86-f5ec5a2d12d5

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/4048/head:pr_4048 && git checkout pr_4048
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.46.0-dev-1bb03b7

Copy link
Collaborator

@igooch igooch left a comment

Choose a reason for hiding this comment

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

LGTM

@igooch igooch merged commit edde3c3 into googleforgames:main Nov 26, 2024
4 checks passed
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/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants