Skip to content

docs(oauth2-proxy): sync template standards updates#360

Open
mberlofa wants to merge 1 commit into
mainfrom
docs/oauth2-proxy-template-standards-sync
Open

docs(oauth2-proxy): sync template standards updates#360
mberlofa wants to merge 1 commit into
mainfrom
docs/oauth2-proxy-template-standards-sync

Conversation

@mberlofa

@mberlofa mberlofa commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Document networkPolicy.extraEgress for OAuth2 Proxy.
  • Add OAuth2 Proxy playground controls for custom egress CIDR and port.
  • Correct the documented pinned image tag to v7.15.3.
  • Register OAuth2 Proxy in the playground sync allowlist.

Related

Validation

  • npm run lint
  • npm run format:check
  • npm run build
  • make site-sync-check CHART=oauth2-proxy
  • make release-check REPO=site
  • make attribution-check REPO=site

@coderabbitai

coderabbitai Bot commented Jul 4, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Adds a Network Policy configuration group with extra egress CIDR and port fields to the oauth2-proxy playground chart config, updates related documentation (image version bump and new networkPolicy.extraEgress values), and adds oauth2-proxy to the playground sync allowlist.

Changes

oauth2-proxy Network Policy support

Layer / File(s) Summary
Network Policy field group
src/data/playground-configs.ts
Adds a collapsible Network Policy section gated by networkPolicy.enabled with new Extra Egress CIDR (default 10.80.0.0/16) and Extra Egress Port (default 443) fields.
Documentation updates
src/pages/docs/charts/oauth2-proxy.mdx
Bumps the pinned upstream image tag from v7.15.2 to v7.15.3 and documents networkPolicy.extraEgress (default []) in the production values example and values table.
Playground allowlist
src/pages/playground.astro
Adds oauth2-proxy to siteSyncPlaygroundConfigs, expanding the allowed chart slugs for playgroundCharts.

Estimated code review effort: 1 (Trivial) | ~5 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title is scoped to oauth2-proxy docs and broadly matches the sync/update theme, though it is somewhat generic.
Description check ✅ Passed The description covers summary, related work, and validation steps, but it omits the template's Quality Gates checklist section.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/oauth2-proxy-template-standards-sync

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant