Skip to content

docs(sonarqube): sync template standards updates#363

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

docs(sonarqube): sync template standards updates#363
mberlofa wants to merge 1 commit into
mainfrom
docs/sonarqube-template-standards-sync

Conversation

@mberlofa

@mberlofa mberlofa commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Document networkPolicy.egress.extraEgress and extraManifests for SonarQube.
  • Add SonarQube playground allowlist coverage and NetworkPolicy extra egress controls.
  • Keep the chart docs aligned with the template standards update in fix(sonarqube): align template standards charts#685.

Related

Validation

  • npm run lint
  • npm run format:check
  • npm run build
  • make site-sync-check CHART=sonarqube
  • 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

This PR adds a Network Policy egress configuration section to the sonarqube playground config, registers the sonarqube chart in the playground site-sync lookup, and updates the sonarqube chart documentation to describe the new networkPolicy.egress.extraEgress and extraManifests capabilities.

Changes

SonarQube egress/extraManifests support

Layer / File(s) Summary
Playground network policy config and registration
src/data/playground-configs.ts, src/pages/playground.astro
Adds a gated Network Policy section with extra egress CIDR/port fields to the sonarqube playground config and registers sonarqube in siteSyncPlaygroundConfigs.
Documentation for extraEgress and extraManifests
src/pages/docs/charts/sonarqube.mdx
Adds a Key Features bullet, extends the production NetworkPolicy example with extraEgress, adds an Extension Manifests section, and expands the Values table with networkPolicy.egress.extraEgress and extraManifests.

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

Possibly related PRs

  • helmforgedev/site#322: Both PRs add a chart slug to the siteSyncPlaygroundConfigs mapping in src/pages/playground.astro.
  • helmforgedev/site#330: Both PRs update the same siteSyncPlaygroundConfigs registry with new chart slugs pointing to src/data/playground-configs.ts.
🚥 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 concise and accurately reflects the SonarQube docs sync with template standards updates.
Description check ✅ Passed The description covers summary, related issue, and validation, but it omits the template's explicit Quality Gates checklist.
✨ 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/sonarqube-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