Skip to content

feat(kvevents): trace raw message processing#640

Open
larainema wants to merge 4 commits into
llm-d:mainfrom
larainema:issue-639-trace-kvevents-processing
Open

feat(kvevents): trace raw message processing#640
larainema wants to merge 4 commits into
llm-d:mainfrom
larainema:issue-639-trace-kvevents-processing

Conversation

@larainema

@larainema larainema commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add an internal OpenTelemetry span around kvevents raw-message processing.
  • Record transport attributes before parsing, then pod/model/event-count attributes after parsing.
  • Mark parse failures with OTel error status.
  • Add unit tests for span attributes and parse-error status.
  • Fix the UDS e2e HostConfig import to match the current testcontainers-go Moby API and keep repo lint green.

Fixes #639
Refs #617

Testing

  • go test ./pkg/kvevents
  • go test ./pkg/...
  • go test ./tests/e2e/uds_tokenizer -run '^$'
  • go run github.com/golangci/golangci-lint/v2/cmd/golangci-lint@v2.9.0 run ./tests/e2e/uds_tokenizer

Signed-off-by: Dong Ma <winterma.dong@gmail.com>
@github-actions github-actions Bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jun 6, 2026
larainema added 3 commits June 6, 2026 14:00
Signed-off-by: Dong Ma <winterma.dong@gmail.com>
Signed-off-by: Dong Ma <winterma.dong@gmail.com>
Signed-off-by: Dong Ma <winterma.dong@gmail.com>
@github-actions

Copy link
Copy Markdown

This PR is marked as stale after 21d of inactivity. After an additional 14d of inactivity (7d to become rotten, then 7d more), it will be closed. To prevent this PR from being closed, add a comment or remove the lifecycle/stale label.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lifecycle/stale size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add OpenTelemetry span for kvevents raw message processing

1 participant