Add HolmesGPT GitHub Actions integration documentation#2051
Add HolmesGPT GitHub Actions integration documentation#2051
Conversation
Adds a new "GitHub Actions" section to the docs sidebar, next to "HTTP APIs", documenting the HolmesGPT PR review action: prerequisites, Robusta API key and cluster setup, required GitHub secrets, the workflow file, and troubleshooting. https://claude.ai/code/session_01KKnLu24WBHBc4SjbpxPrMM
|
✅ 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:1457c4e
docker tag us-central1-docker.pkg.dev/robusta-development/temporary-builds/robusta-runner:1457c4e me-west1-docker.pkg.dev/robusta-development/development/robusta-runner-dev:1457c4e
docker push me-west1-docker.pkg.dev/robusta-development/development/robusta-runner-dev:1457c4ePatch 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:1457c4e |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
WalkthroughAdds documentation for a GitHub Actions integration that runs HolmesGPT PR Review: availability notes, required Robusta API key and secrets, example workflow (.github/workflows/holmes-pr-review.yaml), prompt customization, end-to-end flow, and troubleshooting. Documentation-only; no code changes. ChangesGitHub Actions docs
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 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: 1
🧹 Nitpick comments (1)
docs/configuration/github-actions/holmes-pr-review.rst (1)
93-97: Add explicit HTTP timeouts to improve workflow reliability.The
curlexample has retries but no timeout bounds. Add connect/overall time limits so failed calls fail fast and don’t stall CI jobs.Suggested hardening for the example command
- curl -fsS --retry 2 \ + curl -fsS --retry 2 --connect-timeout 10 --max-time 90 \ -H "Authorization: Bearer $API_KEY" \ -H "Content-Type: application/json" \ --data-binary `@body.json` \ "$RELAY_BASE_URL/api/holmes/$ACCOUNT_ID/chat" > response.json🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/configuration/github-actions/holmes-pr-review.rst` around lines 93 - 97, The curl example currently retries but lacks timeout bounds; update the curl invocation (the long command that posts body.json to "$RELAY_BASE_URL/api/holmes/$ACCOUNT_ID/chat") to include explicit connect and overall timeouts (e.g., add --connect-timeout 10 and --max-time 30) so requests fail fast in CI, and keep the existing -fsS/--retry flags intact; ensure the timeout options are placed alongside the existing headers and --data-binary options.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/configuration/github-actions/holmes-pr-review.rst`:
- Around line 4-6: Update the GitHub Actions workflow example to remove
ambiguity about the relay endpoint by either (A) documenting the self-hosted
relay URL and showing how to set it via an env var (use RELAY_BASE_URL in the
workflow example and explain the self-hosted value), or (B) explicitly state in
the note that self-hosted commercial deployments use the SaaS relay
(https://api.robusta.dev) so no change is needed; ensure the docs around the
workflow example reference RELAY_BASE_URL and include a short example of setting
it as an environment variable for self-hosted installs.
---
Nitpick comments:
In `@docs/configuration/github-actions/holmes-pr-review.rst`:
- Around line 93-97: The curl example currently retries but lacks timeout
bounds; update the curl invocation (the long command that posts body.json to
"$RELAY_BASE_URL/api/holmes/$ACCOUNT_ID/chat") to include explicit connect and
overall timeouts (e.g., add --connect-timeout 10 and --max-time 30) so requests
fail fast in CI, and keep the existing -fsS/--retry flags intact; ensure the
timeout options are placed alongside the existing headers and --data-binary
options.
🪄 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
Run ID: 90383609-9f3b-4e5c-8ecf-96e505f690da
📒 Files selected for processing (3)
docs/configuration/github-actions/holmes-pr-review.rstdocs/configuration/github-actions/index.rstdocs/index.rst
Summary
This PR adds comprehensive documentation for the new HolmesGPT GitHub Actions integration, enabling users to automatically review pull requests using HolmesGPT within their GitHub workflows.
Key Changes
docs/configuration/github-actions/directory with index and integration guidesholmes-pr-review.rst): Complete setup and usage documentation including:index.rst): Landing page for GitHub Actions integrations with links to available actionsdocs/index.rstto include new GitHub Actions section in the table of contentsNotable Implementation Details
curlto call the Holmes Chat API with PR context and custom review promptshttps://claude.ai/code/session_01KKnLu24WBHBc4SjbpxPrMM