Conversation
Soroban RPC and Horizon breakers shared one global trip policy with no way to diverge. Add CIRCUIT_BREAKER_SOROBAN_RPC_*/ CIRCUIT_BREAKER_HORIZON_* env overrides on top of the existing shared CIRCUIT_BREAKER_* policy, so an operator can tune one upstream's failure ratio, min requests, window, or open duration independently. Unset overrides fall back to the shared policy, so existing deployments are unaffected. Closes Suncrest-Labs#1314 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Instrument the outbox relay with time-to-first-attempt and time-to-delivery observations, keyed by event type. First-attempt is recorded once per event on its real first hand-off to the job queue (a released/retried hand-off does not re-record it); delivery is recorded when the relay reconciles the event to its terminal dispatched state. Both are exposed via StdMetrics.Snapshot as count/mean/max per event type, so a growing backlog or delivery delay is observable ahead of an incident. Closes Suncrest-Labs#1311 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
@Johnalex-hub Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
✅ Deploy Preview for nesterhq canceled.
|
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configurationConfiguration used: Repository: Suncrest-Labs/nester/.coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (4)
✨ 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 |
✅ Deploy Preview for nesterdapp ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Summary
Covers 4 issues. Work in progress — marking ready for review once all 4 are implemented.
internal/breaker) with configurable thresholds, wired at startup, surfaced at 503+Retry-After. This PR adds independent per-upstream threshold overrides (CIRCUIT_BREAKER_SOROBAN_RPC_*/CIRCUIT_BREAKER_HORIZON_*) on top of the existing shared policy.Test plan
go build ./...passesCloses #1311
Closes #1314
Closes #1312
Closes #1313