Skip to content

End to end tests for gpu profiler and fix for flakey attachment#223

Merged
gnurizen merged 7 commits into
mainfrom
e2e-gpu
Mar 10, 2026
Merged

End to end tests for gpu profiler and fix for flakey attachment#223
gnurizen merged 7 commits into
mainfrom
e2e-gpu

Conversation

@gnurizen

@gnurizen gnurizen commented Mar 4, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@gnurizen gnurizen force-pushed the e2e-gpu branch 2 times, most recently from afd89b3 to 8f0ecee Compare March 9, 2026 16:01
@umanwizard umanwizard force-pushed the main branch 2 times, most recently from 83f637e to 579a830 Compare March 9, 2026 17:31
@gnurizen gnurizen changed the title End to end tests for gpu profiler End to end tests for gpu profiler and fix for flakey attachment Mar 9, 2026
@gnurizen gnurizen marked this pull request as ready for review March 9, 2026 23:05
@gnurizen gnurizen requested a review from umanwizard March 9, 2026 23:44
@gnurizen gnurizen force-pushed the e2e-gpu branch 4 times, most recently from 0f15c5d to 83d8d98 Compare March 10, 2026 16:33
@umanwizard

Copy link
Copy Markdown
Collaborator

LGTM assuming the CI issues get fixed

ParseUSDTProbes now calls LoadSections internally so callers cannot
silently get zero probes when sections have not been loaded yet.

Remove the package-level sync.Once for the cuda_progs prog array
update — it prevented subsequent tracer instances from populating
their own maps (in tests).  UpdateProgArray is already idempotent.

Neither of these changes fix anything but they both seemed like good
changes.
@gnurizen gnurizen force-pushed the e2e-gpu branch 2 times, most recently from 5aa102d to ea26a98 Compare March 10, 2026 21:08
@gnurizen gnurizen merged commit e09be72 into main Mar 10, 2026
85 of 96 checks passed
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