Skip to content

test(benchmarks): run NativeAOT jobs with the memory diagnoser - #102

Merged
glennawatson merged 5 commits into
mainfrom
glennawatson/benchmark-nativeaot-memory
Sep 14, 2026
Merged

glennawatson merged 5 commits into
mainfrom
glennawatson/benchmark-nativeaot-memory

Conversation

@glennawatson

@glennawatson glennawatson commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

What kind of change does this PR introduce?

Benchmarks.

What is the new behavior?

NativeAOT benchmark jobs run in their own pass with the memory diagnoser.

  • The .NET and .NET Framework passes skip the NativeAOT jobs.
  • The benchmark workflow takes an optional commit SHA to measure, and measures the latest commit when it is empty.
  • A Benchmarks A/B workflow measures two commits on one runner per benchmark project and compares them.
  • BENCHMARK_PROFILERS set to false skips the EventPipe and ETW profilers.
  • A test covers the handler list Interaction.GetHandlers returns to a subclass.

What is the current behavior?

The EventPipe profiler cannot attach to a NativeAOT benchmark process, and the whole benchmark run stops.

What might this PR break?

None.

Checklist

  • I have read the Contribute guide
  • Tests have been added or updated (for bug fixes / features)
  • Docs have been added or updated (for bug fixes / features)
  • Changes target the main branch
  • PR title follows Conventional Commits

Additional information

@codecov

codecov Bot commented Sep 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.72%. Comparing base (b4968f4) to head (3994a0a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #102      +/-   ##
==========================================
- Coverage   99.73%   99.72%   -0.01%     
==========================================
  Files         288      289       +1     
  Lines       11537    11533       -4     
  Branches     1256     1259       +3     
==========================================
- Hits        11506    11501       -5     
- Misses         18       19       +1     
  Partials       13       13              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@glennawatson
glennawatson merged commit 2a14647 into main Sep 14, 2026
@glennawatson
glennawatson deleted the glennawatson/benchmark-nativeaot-memory branch September 14, 2026 15:25
@sonarqubecloud

Copy link
Copy Markdown

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