Skip to content

Add PR-commenting job for benchmarks#11255

Draft
sarahchen6 wants to merge 4 commits intomasterfrom
sarahchen6/add-pr-commenting-job
Draft

Add PR-commenting job for benchmarks#11255
sarahchen6 wants to merge 4 commits intomasterfrom
sarahchen6/add-pr-commenting-job

Conversation

@sarahchen6
Copy link
Copy Markdown
Contributor

@sarahchen6 sarahchen6 commented May 1, 2026

What Does This Do

Add CI job that uploads a summary of the dacapo, load, and startup benchmarks as a comment to the PR

Motivation

Provide a summary of benchmark results to PRs, similar to existing local benchmarks

Additional Notes

Contributor Checklist

Jira ticket: [PROJ-IDENT]

Note: Once your PR is ready to merge, add it to the merge queue by commenting /merge. /merge -c cancels the queue request. /merge -f --reason "reason" skips all merge queue checks; please use this judiciously, as some checks do not run at the PR-level. For more information, see this doc.

@sarahchen6 sarahchen6 added tag: no release notes Changes to exclude from release notes comp: tooling Build & Tooling labels May 1, 2026
@dd-octo-sts
Copy link
Copy Markdown
Contributor

dd-octo-sts Bot commented May 1, 2026

🟢 Java Benchmark SLOs — All performance SLOs passed

Suite Status
Startup 🟢 pass
Load 🟢 pass
DaCapo 🟢 pass

Commit: 0b549ba2 · CI Pipeline · Benchmarking Platform UI

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

Labels

comp: tooling Build & Tooling tag: no release notes Changes to exclude from release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant