-
Notifications
You must be signed in to change notification settings - Fork 915
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add build metrics report as artifact to cpp-build workflow #12750
Add build metrics report as artifact to cpp-build workflow #12750
Conversation
…df into exper-build-metrics-report
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great @davidwendt.
We can merge this PR and then come back to remove the manual upload steps later after your other PR below is merged.
Appreciate your work on this.
I'd be interested in hearing your thoughts on centralizing sort_ninja_log.py
so that other repositories can make use of these reports.
We could easily add that to our CI containers to make it available to all RAPIDS libs.
Adds `rapids-upload-artifacts-dir` utility script for uploading individual files from the `RAPIDS_ARTIFACTS_DIR` directory. References: rapidsai/shared-workflows#42 rapidsai/cudf#12750 --------- Co-authored-by: AJ Schmidt <[email protected]>
/merge |
Description
Adds the Build Metrics Report into the gitHub actions workflow by uploading a link to the generated report to the CI downloads S3 and publishes a link to the report in the build output.
A follow-on PR will provide a more direct mechanism for locating the report link.
Checklist