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

docs: GKE Autopilot section #20697

Merged
merged 15 commits into from
Feb 22, 2024
Merged

docs: GKE Autopilot section #20697

merged 15 commits into from
Feb 22, 2024

Conversation

david-yu
Copy link
Contributor

@david-yu david-yu commented Feb 21, 2024

Description

GKE Autopilot section for reference when installing Consul K8s. Update API Gateway install guide on K8s to also show GKE Autopilot install.

Testing & Reproduction steps

Links

PR Checklist

  • updated test coverage
  • external facing docs updated
  • appropriate backport labels added
  • not a security concern

@david-yu david-yu added pr/no-changelog PR does not need a corresponding .changelog entry backport/1.17 This release series is no longer active on CE. Use backport/ent/1.17. backport/1.18 labels Feb 21, 2024
@david-yu david-yu requested a review from a team as a code owner February 21, 2024 23:59
@github-actions github-actions bot added the type/docs Documentation needs to be created/updated/clarified label Feb 21, 2024
@david-yu david-yu added the backport/1.16 This release series is no longer active on CE. Use backport/ent/1.16. label Feb 21, 2024
@david-yu david-yu requested a review from curtbushko February 22, 2024 00:00
Copy link
Contributor

@curtbushko curtbushko left a comment

Choose a reason for hiding this comment

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

Thanks David!

Copy link
Contributor

@boruszak boruszak left a comment

Choose a reason for hiding this comment

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

Overall LGTM.

My suggestions mostly involve using active voice. But per my comment, it's not really clear to me what you're communicating with the sentence to "customize the connectInject.apiGateway stanza to accommodate the pre-installed Gateway API CRDs." I can see the difference in the configurations, but can you add a sentence or two about what those configurations are doing/why they're important from the user's perspective?

Approving so you're not blocked. Please let me know if you if you need any additional review!

Approved on behalf of consul-docs

website/content/docs/k8s/installation/install.mdx Outdated Show resolved Hide resolved
website/content/docs/k8s/installation/install.mdx Outdated Show resolved Hide resolved
website/content/docs/k8s/installation/install.mdx Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.16 This release series is no longer active on CE. Use backport/ent/1.16. backport/1.17 This release series is no longer active on CE. Use backport/ent/1.17. pr/no-changelog PR does not need a corresponding .changelog entry type/docs Documentation needs to be created/updated/clarified
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants