docs: fix Firewall / DNS Allowlist page - #2131
Conversation
- Rewrite relay.robusta.dev and sp.robusta.dev per region (EU/AP) in the region selector, matching the existing api/platform handling — EU and AP accounts use relay.eu/.ap.robusta.dev and sp.eu/.ap.robusta.dev - Fix sp.robusta.dev description: platform storage, not platform analytics - Split the allowlist into Runtime / Installation / Add-ons sections - Add sp.robusta.dev to the allowlist verification command - Drop the region selector from the image-mirroring lists, which contain no region-dependent addresses Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016zhBq5qKJEeZNCaWHADRCA
|
✅ Docker image ready for
Use this tag to pull the image for testing. 📋 Copy commandsgcloud auth configure-docker us-central1-docker.pkg.dev
docker pull us-central1-docker.pkg.dev/robusta-development/temporary-builds/robusta-runner:32372c5
docker tag us-central1-docker.pkg.dev/robusta-development/temporary-builds/robusta-runner:32372c5 me-west1-docker.pkg.dev/robusta-development/development/robusta-runner-dev:32372c5
docker push me-west1-docker.pkg.dev/robusta-development/development/robusta-runner-dev:32372c5Patch Helm values in one line: helm upgrade --install robusta robusta/robusta \
--reuse-values \
--set runner.image=me-west1-docker.pkg.dev/robusta-development/development/robusta-runner-dev:32372c5 |
WalkthroughRegion-aware URL handling now recognizes ChangesRegion routing and proxy documentation
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 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 `@docs/setup-robusta/proxies.rst`:
- Around line 34-37: Update the outbound traffic note in the proxy documentation
to state that connections originate from the relevant pod or node, rather than
always from the runner. Ensure the wording covers HolmesGPT pod requests and
installation image pulls from the node or container runtime, while preserving
the existing inbound-connection and HTTPS/TCP 443 details.
- Around line 46-53: Clarify the regional host coverage in the Robusta SaaS
platform section around the *.robusta.dev entry. Add the required regional
wildcards, including *.eu.robusta.dev and *.ap.robusta.dev, or explicitly note
that proxy rules must allow exact region-qualified hosts such as
api.eu.robusta.dev.
🪄 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: 2441f6b7-7008-47a8-86d5-fb5c4b63ad80
📒 Files selected for processing (2)
docs/_static/region-selector.jsdocs/setup-robusta/proxies.rst
Claude-Session: https://claude.ai/code/session_016zhBq5qKJEeZNCaWHADRCA