Skip to content

[otel-7/7] Report structured GPU-sniff measurements through the telemetry adapter - #8280

Draft
rhewett-nv wants to merge 7 commits into
NVIDIA:mainfrom
rhewett-nv:rhewett/otel/public-main-07-gpu-sniff-events
Draft

rhewett-nv wants to merge 7 commits into
NVIDIA:mainfrom
rhewett-nv:rhewett/otel/public-main-07-gpu-sniff-events

Conversation

@rhewett-nv

@rhewett-nv rhewett-nv commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor
  • I, the PR author, have personally reviewed every line of this PR.

What does this PR do?

Maps structured GPU-sniff results to canonical telemetry events without changing the benchmark execution path.

This is PR 7 of 7 and is intended to be reviewed relative to PR 6.

Summary

  • Adapts structured benchmark results into the canonical GPU-sniff event schema.
  • Uses the existing result callback/sink and preserves benchmark cadence and all-rank behavior.
  • Leaves collectives, process groups, timers, and application execution unchanged when telemetry is disabled.

Issue tracking

Linked issue: N/A — follow-up integration work.

Contribution process

Pre-checks

  • I have added relevant unit tests
  • I have added relevant functional tests
  • This PR does not add or change a GPU kernel, so no determinism manifest update is required.
  • I have added proper typing to my code (typing guidelines)
  • I have added relevant documentation
  • I have run the autoformatter checks on my PR

Validation

  • Focused telemetry unit suite at this boundary: 240 passed.
  • Scoped Ruff, Black, adjacent-diff, DCO, and signature checks passed.
  • No local GPU or distributed run was performed.

@copy-pr-bot

copy-pr-bot Bot commented Oct 15, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@rhewett-nv rhewett-nv changed the title Report structured GPU-sniff measurements through the telemetry adapter [otel-7/7] Report structured GPU-sniff measurements through the telemetry adapter Oct 15, 2026
@rhewett-nv
rhewett-nv force-pushed the rhewett/otel/public-main-07-gpu-sniff-events branch 2 times, most recently from dac73f2 to b1868b7 Compare October 16, 2026 23:29
Signed-off-by: Russell Hewett <rhewett@nvidia.com>
Signed-off-by: Russell Hewett <rhewett@nvidia.com>
Signed-off-by: Russell Hewett <rhewett@nvidia.com>
Signed-off-by: Russell Hewett <rhewett@nvidia.com>
Signed-off-by: Russell Hewett <rhewett@nvidia.com>
Signed-off-by: Russell Hewett <rhewett@nvidia.com>
Signed-off-by: Russell Hewett <rhewett@nvidia.com>
@rhewett-nv
rhewett-nv force-pushed the rhewett/otel/public-main-07-gpu-sniff-events branch from b1868b7 to b4b8b54 Compare October 16, 2026 23:35

This branch has not been deployed

No deployments
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.

1 participant