Skip to content

fix a policy annotation syntax example - #2352

Merged
kenjenkins merged 1 commit into
mainfrom
kenjenkins/ingress-annotations
Aug 25, 2026
Merged

fix a policy annotation syntax example#2352
kenjenkins merged 1 commit into
mainfrom
kenjenkins/ingress-annotations

Conversation

@kenjenkins

@kenjenkins kenjenkins commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Summary

All the k8s object annotation values need to be strings, so we have to quote 'true' so it doesn't parse as a boolean.

I searched for other cases where we had an ingress.pomerium.io/ annotation with an unquoted true but didn't find any.

Related

AI disclosure

none

Checklist

  • reference any related issues
  • disclosed AI usage (or wrote "none") per AI_POLICY.md

All the k8s object annotation values need to be strings, so we have to
quote 'true' so it doesn't parse as a boolean.

I searched for other cases where we had an 'ingress.pomerium.io/'
annotation with an unquoted 'true' but didn't find any.
@kenjenkins
kenjenkins requested a review from a team as a code owner August 24, 2026 22:01
@kenjenkins
kenjenkins requested review from nickytonline and removed request for a team August 24, 2026 22:01
@netlify

netlify Bot commented Aug 24, 2026

Copy link
Copy Markdown

Deploy Preview for pomerium-docs ready!

Name Link
🔨 Latest commit f32e2d6
🔍 Latest deploy log https://app.netlify.com/projects/pomerium-docs/deploys/6a8cbf4303154a0008c936bf
😎 Deploy Preview https://deploy-preview-2352--pomerium-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@greptile-apps

greptile-apps Bot commented Aug 24, 2026

Copy link
Copy Markdown

Greptile Summary

Corrects the Kubernetes Ingress annotation example so its boolean-like value is represented as the required YAML string.

  • Quotes true in the allow_any_authenticated_user annotation example.

Confidence Score: 5/5

The PR appears safe to merge.

The changed value is valid YAML string syntax for a Kubernetes annotation and is consistent with the repository’s canonical Ingress examples and templates.

Important Files Changed

Filename Overview
content/docs/reference/routes/allow-any-authenticated-user.mdx Updates the annotation example to valid Kubernetes string-map syntax without introducing documentation or behavior issues.

Reviews (1): Last reviewed commit: "fix a policy annotation syntax example" | Re-trigger Greptile

@kenjenkins
kenjenkins merged commit 435a067 into main Aug 25, 2026
11 checks passed
@kenjenkins
kenjenkins deleted the kenjenkins/ingress-annotations branch August 25, 2026 16:26
kenjenkins added a commit that referenced this pull request Aug 25, 2026
fix a policy annotation syntax example (#2352)

All the k8s object annotation values need to be strings, so we have to
quote 'true' so it doesn't parse as a boolean.

Co-authored-by: Kenneth Jenkins <51246568+kenjenkins@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants