Repository navigation
Fix report collection for build tooling projects - #12815
Conversation
🟢 Java Benchmark SLOs — All performance SLOs passed
PR vs. master results
Commit: Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion. |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
More details
The report-root patterns match the JUnit collector, and workspace projects retain their existing archive destinations while build-tooling reports use their actual project paths.
🤖 Bits Code Review · Commit b429f75
|
/merge -f --reason "No need to run MQ, validated in PR" |
|
View all feedbacks in Devflow UI.
The expected merge time in
Warning This change was merged without running any pre merge CI checks Reason: No need to run MQ, validated in PR |
f45dec8
into
master
What Does This Do
Collect reports from
buildSrcandbuild-logicas well asworkspace, using the actual project directory for copy, move, and delete operations. Enablenullglobso unmatched report roots are skipped.Motivation
Build tooling reports outside
workspacewere omitted. Match the project roots used by the JUnit result collector while preserving the existing report layout for workspace projects.Additional Notes
Validation: Bash syntax check; fixture checks covering copy, move, and delete across all three roots (9 combinations), plus an empty-root case. The extracted script is byte-for-byte identical to the source branch version.
With absent crash files, move mode prints harmless
mvusage messages because unmatched globs disappear; commands retain the existing|| truehandling.CI artifact before this fix:

CI artifact after this fix:

Contributor Checklist
type:and (comp:orinst:) labels in addition to any other useful labelsclose,fix, or any linking keywords when referencing an issueUse
solvesinstead, and assign the PR milestone to the issue