Skip to content

Conversation

sallyom
Copy link

@sallyom sallyom commented Jul 9, 2025

Initial distributed tracing instrumentation
Also, fix nil pointer dereference in KV Cache scorer initialization.

@sallyom sallyom force-pushed the tracing branch 2 times, most recently from 3342ece to 74dac2d Compare July 30, 2025 00:53

var hitRatio float64
if totalPods > 0 {
hitRatio = float64(len(podScores)) / float64(totalPods)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What if podScores == 0?

@sallyom sallyom force-pushed the tracing branch 6 times, most recently from d47479f to b4041d4 Compare August 26, 2025 20:42
- Add OpenTelemetry distributed tracing support to kv-cache-manager
- Instrument Redis operations with automatic OTel tracing and metrics
@sallyom sallyom force-pushed the tracing branch 4 times, most recently from 21794b6 to 41fd565 Compare August 27, 2025 17:41
Signed-off-by: sallyom <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants