Skip to content

docs(openreel-video): sync template standards updates#362

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

docs(openreel-video): sync template standards updates#362
mberlofa wants to merge 1 commit into
mainfrom
docs/openreel-video-template-standards-sync

Conversation

@mberlofa

@mberlofa mberlofa commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Document networkPolicy.extraEgress for OpenReel Video.
  • Add OpenReel Video playground controls for custom egress CIDR and port.
  • Correct the documented pinned image tag to v0.5.0.
  • Register OpenReel Video in the playground sync allowlist.

Related

Validation

  • npm run lint
  • npm run format:check
  • npm run build
  • make site-sync-check CHART=openreel-video
  • 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 NetworkPolicy extraEgress configuration to the openreel-video playground chart config, updates the chart documentation to describe the new field and image tag, and registers openreel-video in the playground chart list.

Changes

NetworkPolicy extraEgress feature

Layer / File(s) Summary
Playground config: extraEgress fields
src/data/playground-configs.ts
Adds a Network Policy section gated by networkPolicy.enabled with extra egress CIDR and TCP port fields.
Documentation updates for extraEgress and image tag
src/pages/docs/charts/openreel-video.mdx
Bumps the documented image tag to v0.5.0, adds networkPolicy.extraEgress: [] to the production values example, and adds a corresponding row in the values table.
Playground chart registration
src/pages/playground.astro
Adds openreel-video to siteSyncPlaygroundConfigs, allowing it to appear in the playground regardless of maturity status.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • helmforgedev/site#322: Both PRs update src/pages/playground.astro's siteSyncPlaygroundConfigs registry to add a new chart slug.
  • helmforgedev/site#330: Both PRs extend siteSyncPlaygroundConfigs with a new chart slug, affecting the same chart-selection filtering logic.
🚥 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 relevant to the OpenReel Video docs sync, though it is somewhat generic and does not highlight the main changes.
Description check ✅ Passed The description covers the summary, related issue, and validation steps; only the template-specific quality gates section is missing.
✨ 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/openreel-video-template-standards-sync

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

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/data/playground-configs.ts`:
- Around line 285-305: The new Network Policy block was added to the wrong chart
config array, creating a duplicate Apache section and leaving OpenReel Video
without the new fields. Move the Network Policy field group from the apache
config in playground-configs to the openreel-video config array, keeping the
existing networkPolicy.extraEgress field keys and labels intact. Verify the
apache chart only retains its original Network Policy section and that
openreel-video now exposes the CIDR and port controls.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 8f3e9e3e-1b0e-4e7e-8dc9-7fcf4f9641fe

📥 Commits

Reviewing files that changed from the base of the PR and between 365449a and 61dd3f6.

📒 Files selected for processing (3)
  • src/data/playground-configs.ts
  • src/pages/docs/charts/openreel-video.mdx
  • src/pages/playground.astro

Comment thread src/data/playground-configs.ts Outdated
@mberlofa mberlofa force-pushed the docs/openreel-video-template-standards-sync branch from 61dd3f6 to ff8c6a7 Compare July 4, 2026 06:06
@mberlofa mberlofa force-pushed the docs/openreel-video-template-standards-sync branch from ff8c6a7 to 9407125 Compare July 4, 2026 06:20
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