Skip to content

Commit db475f1

Browse files
committed
Auto merge of rust-lang#136977 - Kobzol:citool-datadog, r=<try>
[WIP] Upload Datadog metrics with citool Opening as a draft for testing. r? `@ghost`
2 parents 54cdc75 + 091e615 commit db475f1

File tree

19 files changed

+1837
-5428
lines changed

19 files changed

+1837
-5428
lines changed

.github/workflows/ci.yml

+7-8
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
# This file defines our primary CI workflow that runs on pull requests
22
# and also on pushes to special branches (auto, try).
33
#
4-
# The actual definition of the executed jobs is calculated by a Python
5-
# script located at src/ci/github-actions/ci.py, which
4+
# The actual definition of the executed jobs is calculated by the
5+
# `src/ci/citool` crate, which
66
# uses job definition data from src/ci/github-actions/jobs.yml.
77
# You should primarily modify the `jobs.yml` file if you want to modify
88
# what jobs are executed in CI.
@@ -56,7 +56,10 @@ jobs:
5656
- name: Calculate the CI job matrix
5757
env:
5858
COMMIT_MESSAGE: ${{ github.event.head_commit.message }}
59-
run: python3 src/ci/github-actions/ci.py calculate-job-matrix >> $GITHUB_OUTPUT
59+
run: |
60+
cd src/ci/citool
61+
cargo test
62+
cargo run calculate-job-matrix >> $GITHUB_OUTPUT
6063
id: jobs
6164
job:
6265
name: ${{ matrix.full_name }}
@@ -212,13 +215,9 @@ jobs:
212215
- name: upload job metrics to DataDog
213216
if: needs.calculate_matrix.outputs.run_type != 'pr'
214217
env:
215-
DATADOG_SITE: datadoghq.com
216218
DATADOG_API_KEY: ${{ secrets.DATADOG_API_KEY }}
217219
DD_GITHUB_JOB_NAME: ${{ matrix.full_name }}
218-
run: |
219-
cd src/ci
220-
npm ci
221-
python3 scripts/upload-build-metrics.py ../../build/cpu-usage.csv
220+
run: ./src/ci/citool/target/debug/citool upload-build-metrics ../../build/cpu-usage.csv
222221

223222
# This job isused to tell bors the final status of the build, as there is no practical way to detect
224223
# when a workflow is successful listening to webhooks only in our current bors implementation (homu).

0 commit comments

Comments
 (0)