Skip to content

Commit fcaf3aa

Browse files
authored
add telemetry (#657)
Enables telemetry during cuxfilter CI runs. This is done by parsing GitHub Actions run log metadata and should have no impact on build or test times. xref rapidsai/build-infra#139 Authors: - Mike Sarahan (https://github.com/msarahan) Approvers: - Gil Forsyth (https://github.com/gforsyth) URL: #657
1 parent 329a1b4 commit fcaf3aa

1 file changed

Lines changed: 25 additions & 0 deletions

File tree

.github/workflows/pr.yaml

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,13 +20,26 @@ jobs:
2020
- docs-build
2121
- wheel-build
2222
- wheel-tests
23+
- telemetry-setup
2324
secrets: inherit
2425
uses: rapidsai/shared-workflows/.github/workflows/pr-builder.yaml@nvks-runners
2526
if: always()
2627
with:
2728
needs: ${{ toJSON(needs) }}
29+
telemetry-setup:
30+
runs-on: ubuntu-latest
31+
continue-on-error: true
32+
env:
33+
OTEL_SERVICE_NAME: "pr-cuxfilter"
34+
steps:
35+
- name: Telemetry setup
36+
# This gate is here and not at the job level because we need the job to not be skipped,
37+
# since other jobs depend on it.
38+
if: ${{ vars.TELEMETRY_ENABLED == 'true' }}
39+
uses: rapidsai/shared-actions/telemetry-dispatch-stash-base-env-vars@main
2840
changed-files:
2941
secrets: inherit
42+
needs: telemetry-setup
3043
uses: rapidsai/shared-workflows/.github/workflows/changed-files.yaml@nvks-runners
3144
with:
3245
files_yaml: |
@@ -46,7 +59,10 @@ jobs:
4659
- '!notebooks/**'
4760
checks:
4861
secrets: inherit
62+
needs: telemetry-setup
4963
uses: rapidsai/shared-workflows/.github/workflows/checks.yaml@nvks-runners
64+
with:
65+
ignored_pr_jobs: telemetry-summarize
5066
conda-python-build:
5167
needs: checks
5268
secrets: inherit
@@ -101,3 +117,12 @@ jobs:
101117
script: ci/test_wheel.sh
102118
# This selects "ARCH=amd64 + the latest supported Python + CUDA".
103119
matrix_filter: map(select(.ARCH == "amd64")) | max_by([(.PY_VER|split(".")|map(tonumber)), (.CUDA_VER|split(".")|map(tonumber))]) | [.]
120+
telemetry-summarize:
121+
# This job must use a self-hosted runner to record telemetry traces.
122+
runs-on: linux-amd64-cpu4
123+
needs: pr-builder
124+
if: ${{ vars.TELEMETRY_ENABLED == 'true' && !cancelled() }}
125+
continue-on-error: true
126+
steps:
127+
- name: Telemetry summarize
128+
uses: rapidsai/shared-actions/telemetry-dispatch-summarize@main

0 commit comments

Comments
 (0)