From 2a98d2995877b107f6f8feb90f730d1a5bfa737f Mon Sep 17 00:00:00 2001 From: Michael Sarahan Date: Tue, 17 Dec 2024 13:07:16 -0600 Subject: [PATCH 1/2] update telemetry actions to fluent-bit friendly style --- .github/workflows/pr.yaml | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/.github/workflows/pr.yaml b/.github/workflows/pr.yaml index 5ad18b47e5..51819a26cb 100644 --- a/.github/workflows/pr.yaml +++ b/.github/workflows/pr.yaml @@ -203,11 +203,5 @@ jobs: if: ${{ vars.TELEMETRY_ENABLED == 'true' && !cancelled() }} continue-on-error: true steps: - - name: Load stashed telemetry env vars - uses: rapidsai/shared-actions/telemetry-dispatch-load-base-env-vars@main - with: - load_service_name: true - name: Telemetry summarize - uses: rapidsai/shared-actions/telemetry-dispatch-write-summary@main - with: - cert_concat: "${{ secrets.OTEL_EXPORTER_OTLP_CA_CERTIFICATE }};${{ secrets.OTEL_EXPORTER_OTLP_CLIENT_CERTIFICATE }};${{ secrets.OTEL_EXPORTER_OTLP_CLIENT_KEY }}" + uses: rapidsai/shared-actions/telemetry-dispatch-summarize@main From 3a961508f297ad6543ba789add6823c8a86f0232 Mon Sep 17 00:00:00 2001 From: Michael Sarahan Date: Tue, 17 Dec 2024 18:25:09 -0600 Subject: [PATCH 2/2] fix runner for telemetry summarize to be self-hosted --- .github/workflows/pr.yaml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/pr.yaml b/.github/workflows/pr.yaml index 51819a26cb..95d648f8e6 100644 --- a/.github/workflows/pr.yaml +++ b/.github/workflows/pr.yaml @@ -198,7 +198,8 @@ jobs: sccache -s; telemetry-summarize: - runs-on: ubuntu-latest + # This job must use a self-hosted runner to record telemetry traces. + runs-on: linux-amd64-cpu4 needs: pr-builder if: ${{ vars.TELEMETRY_ENABLED == 'true' && !cancelled() }} continue-on-error: true