Skip to content

[APMLP-714] Only check SLO breaches for benchmarks if they ran - #11307

Merged
gh-worker-dd-mergequeue-cf854d[bot] merged 1 commit into
masterfrom
sarahchen6/check-slo-breach-edit
May 7, 2026
Merged

gh-worker-dd-mergequeue-cf854d[bot] merged 1 commit into
masterfrom
sarahchen6/check-slo-breach-edit

Conversation

@sarahchen6

Copy link
Copy Markdown
Contributor

What Does This Do

Ensure Java benchmark SLO breach check jobs only run if the benchmarks themselves also ran

Motivation

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 requested a review from a team as a code owner May 7, 2026 16:12
@sarahchen6
sarahchen6 requested review from pawelchcki and removed request for a team May 7, 2026 16:12
@sarahchen6 sarahchen6 added tag: no release notes Changes to exclude from release notes comp: tooling Build & Tooling labels May 7, 2026
@pawelchcki
pawelchcki self-requested a review May 7, 2026 16:52
@sarahchen6 sarahchen6 added tag: override groovy enforcement Override the "Enforce Groovy Migration" check and removed tag: override groovy enforcement Override the "Enforce Groovy Migration" check labels May 7, 2026
@sarahchen6

Copy link
Copy Markdown
Contributor Author

/merge

@gh-worker-devflow-routing-ef8351

gh-worker-devflow-routing-ef8351 Bot commented May 7, 2026 •

Copy link
Copy Markdown

View all feedbacks in Devflow UI.

2026-05-07 18:37:04 UTC ℹ️ Start processing command /merge


2026-05-07 18:37:09 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in master is approximately 1h (p90).


2026-05-07 19:57:38 UTC ℹ️ MergeQueue: This merge request was merged

@gh-worker-dd-mergequeue-cf854d
gh-worker-dd-mergequeue-cf854d Bot merged commit 488e95c into master May 7, 2026
576 of 581 checks passed
@gh-worker-dd-mergequeue-cf854d
gh-worker-dd-mergequeue-cf854d Bot deleted the sarahchen6/check-slo-breach-edit branch May 7, 2026 19:57
@github-actions github-actions Bot added this to the 1.63.0 milestone May 7, 2026
@mikewoo-dd mikewoo-dd changed the title Only check SLO breaches for benchmarks if they ran [APMLP-714] Only check SLO breaches for benchmarks if they ran Oct 24, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Hi! 👋 Thanks for your pull request! 🎉

To help us review it, please make sure to:

  • Remove the tag from the pull request title

If you need help, please check our contributing guidelines.

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.

3 participants