Cosmos: downgrade request-status error logs to debug#4711
Cosmos: downgrade request-status error logs to debug#4711analogrelay with Copilot wants to merge 4 commits into
Conversation
Co-authored-by: analogrelay <7574+analogrelay@users.noreply.github.com>
|
Azure Pipelines: Successfully started running 1 pipeline(s). 2 pipeline(s) were filtered out due to trigger conditions. There may be pipelines that require an authorized user to comment /azp run to run. |
There was a problem hiding this comment.
Pull request overview
Downgrades Cosmos HTTP-status logging to reduce noise from expected control-flow responses.
Changes:
- Changes pipeline abort logs from error to debug.
- Changes hedged terminal HTTP logs from warning to debug.
Show a summary per file
| File | Description |
|---|---|
operation_pipeline.rs |
Lowers terminal outcome log severity. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 1
- Review effort level: Medium
| } | ||
|
|
||
| tracing::error!( | ||
| tracing::debug!( |
analogrelay
left a comment
There was a problem hiding this comment.
@copilot While you're fixing up the issues below, look out for any places where we're tracing "event name"-style strings like cosmos.hedge.terminal_http_error. Tracing events should have human-readable log-line-style messages.
There was a problem hiding this comment.
@copilot Fix this up to use a human-readable log line instead of an "event name" style. Something like non-retriable http error in hedging attempt
There was a problem hiding this comment.
Updated in 16ae25a. This now uses a human-readable message: non-retriable http error in hedging attempt.
There was a problem hiding this comment.
@copilot Fix this up to use a human-readable log line instead of an "event name" style. Something like non-retriable transport error in hedging attempt
There was a problem hiding this comment.
Updated in 16ae25a. This now uses a human-readable message: non-retriable transport error in hedging attempt.
Co-authored-by: analogrelay <7574+analogrelay@users.noreply.github.com>
Addressed in 16ae25a. Updated the touched hedging terminal tracing events to use human-readable log-line messages instead of event-name-style strings. |
Create-if-not-existsand similar flows can intentionally treat non-2xx service responses (for example, 409 Conflict) as successful control flow. The Cosmos pipeline was emitting high-severity logs for these outcomes, creating noisyWARN/ERRORoutput under normal usage.Pipeline abort logging severity
operation_pipelineabort-path logging fromtracing::error!totracing::debug!for terminal operation outcomes that include typed Cosmos HTTP status.Hedged terminal HTTP outcome severity
cosmos.hedge.terminal_http_error) fromtracing::warn!totracing::debug!.Behavioral impact