Skip to content

Conversation

cmtice
Copy link
Contributor

@cmtice cmtice commented Aug 8, 2025

Update the metrics.py script to collect mertrics on libc++ premerge testing (on linux); also to create aggregate summaries for the three stages; and to upload the libc++ metrics data to Grafana.

cmtice added 4 commits August 8, 2025 14:27
** DRAFT!! DO NOT REVIEW! DO NOT MERGE! **

Work-in-progress, playing around with adding metrics collection
and dashboard for libc++ premerge testing data.
@cmtice cmtice requested a review from boomanaiden154 August 8, 2025 22:55
@cmtice cmtice changed the title [libc++] [WIP] Add metrics collection for libc++ premerge testing. [llvm][CI] Add metrics collection for libc++ premerge testing. Aug 21, 2025
@cmtice cmtice marked this pull request as ready for review August 21, 2025 16:00
Copy link

github-actions bot commented Aug 21, 2025

✅ With the latest revision this PR passed the Python code formatter.

@cmtice cmtice requested review from ldionne and EricWF August 21, 2025 16:08
Copy link
Contributor

@boomanaiden154 boomanaiden154 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code formatting and tests need to be fixed (see the CI).

@cmtice
Copy link
Contributor Author

cmtice commented Aug 22, 2025

I've added tests for all the libc++ work now. This should be ready to review again.

Copy link
Contributor

@boomanaiden154 boomanaiden154 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The formatter still needs to be fixed. https://llvm.org/docs/CodingStandards.html#python-version-and-source-code-formatting has information on what to do in case you aren't already familiar.

Other than that, a couple nits and I think this is ready to go.

@cmtice cmtice merged commit 43f05fb into llvm:main Aug 22, 2025
12 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