fix(agent): soften binary source fallback log wording - #803
Conversation
AcquireBinary tries a prioritized chain of sources (env override, injected, cache, download) and logs every miss at debug level as it falls through to the next one. The env-var override is checked first specifically so operators can set it, but almost never is -- so "source local path override (DEVSY_AGENT_BINARY) failed: ..." showed up on nearly every run, reading like an error for what's actually the expected, common case of an unused optional source.
✅ Deploy Preview for images-devsy-sh canceled.
|
|
Warning Review limit reached
Next review available in: 14 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
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 devsydev canceled.
|
1 participant
`AcquireBinary` tries a prioritized chain of sources (env override, injected, cache, download) and logs every miss at debug level as it falls through to the next one. The env-var override is checked first specifically so operators can set it, but almost never do — so `"source local path override (DEVSY_AGENT_BINARY) failed: no agent binary path set..."` showed up on nearly every run, reading like an error for what's actually the expected, routine case of an unused optional source.
Changed the message from
"source %s failed: %v"to"source %s unavailable, trying next: %v"— same debug-level diagnostic info, less alarming wording.