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

doc(kic) update admission webhook documentation #5105

Merged
merged 3 commits into from
Feb 2, 2023
Merged

Conversation

rainest
Copy link
Contributor

@rainest rainest commented Jan 31, 2023

Description

Call out the admission webhook more prominently in deploy documentation.

Use "admission webook" consistently. Previously documentation used "admission webhook" and "admission controller" interchangeably.

Add Helm chart instructions to the admission webhook documentation.

Note: this references a chart version that is not yet released. It is ready for review but should not be merged until Kong/charts#727 is merged.

Testing instructions

None. No major changes to formatting, only some link updates that Vale should catch if I botched them

Checklist

  • Review label added
  • PR pointed to correct branch (main for immediate publishing, or a release branch: e.g. release/gateway-3.2, release/deck-1.17)

@rainest rainest added do not merge Issues/ PRs whose changes should not be merged at this time review:copyedit Request for writer review. labels Jan 31, 2023
@rainest rainest requested a review from a team as a code owner January 31, 2023 18:39
@netlify
Copy link

netlify bot commented Jan 31, 2023

Deploy Preview for kongdocs failed.

Name Link
🔨 Latest commit cfdc3f1
🔍 Latest deploy log https://app.netlify.com/sites/kongdocs/deploys/63dbec3ed4921200082eb000

Call out the admission webhook more prominently in deploy documentation.

Use "admission webook" consistently. Previously documentation used
"admission webhook" and "admission controller" interchangeably.

Add Helm chart instructions to the admission webhook documentation.
Copy link
Contributor

@cloudjumpercat cloudjumpercat left a comment

Choose a reason for hiding this comment

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

Left a few grammar suggestions, I'll also make a commit to rename the "Admissions Controller" in the sidebar.

Copy link
Contributor

@cloudjumpercat cloudjumpercat left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for updating this!

@rainest
Copy link
Contributor Author

rainest commented Feb 2, 2023

I apparently can't re-run checks on this repo, but it looks like the failures were all transient github issues, so ignoring. They were passing previously.

@rainest rainest merged commit fe277d8 into main Feb 2, 2023
@rainest rainest deleted the doc/admission branch February 2, 2023 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge Issues/ PRs whose changes should not be merged at this time review:copyedit Request for writer review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants