Skip to content

Commit 9d6182c

Browse files
authored
chore(profiling): use install-llvm-action for native tests actions (#14390)
In an attempt to minimize the time to install llvm deps in profile native tests GH actions ## Checklist - [x] PR author has checked that all the criteria below are met - The PR description includes an overview of the change - The PR description articulates the motivation for the change - The change includes tests OR the PR description describes a testing strategy - The PR description notes risks associated with the change, if any - Newly-added code is easy to change - The change follows the [library release note guidelines](https://ddtrace.readthedocs.io/en/stable/releasenotes.html) - The change includes or references documentation updates if necessary - Backport labels are set (if [applicable](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)) ## Reviewer Checklist - [ ] Reviewer has checked that all the criteria below are met - Title is accurate - All changes are related to the pull request's stated goal - Avoids breaking [API](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces) changes - Testing strategy adequately addresses listed risks - Newly-added code is easy to change - Release note makes sense to a user of the library - If necessary, author has acknowledged and discussed the performance implications of this PR as reported in the benchmarks PR comment - Backport labels are set in a manner that is consistent with the [release branch maintenance policy](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)
1 parent f798a28 commit 9d6182c

File tree

1 file changed

+7
-8
lines changed

1 file changed

+7
-8
lines changed

.github/workflows/profiling-native.yml

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -9,12 +9,13 @@ on:
99
paths:
1010
- ddtrace/internal/datadog/profiling/**
1111
- ddtrace/profiling/**
12+
- .github/workflows/profiling-native.yml
1213
workflow_dispatch: {}
1314

1415
jobs:
1516
test:
1617
runs-on: ${{ matrix.os }}
17-
timeout-minutes: 7
18+
timeout-minutes: 9
1819
strategy:
1920
fail-fast: false
2021
matrix:
@@ -32,15 +33,13 @@ jobs:
3233
with:
3334
python-version: ${{ matrix.python-version }}
3435

35-
- name: Install llvm 19
36-
run: |
37-
# Ubuntu-24.04 GH actions image has llvm-18, but we use 19 as it's
38-
# the latest one available.
39-
wget https://apt.llvm.org/llvm.sh
40-
chmod +x llvm.sh
41-
sudo ./llvm.sh 19
36+
- uses: KyleMayes/install-llvm-action@a7a1a882e2d06ebe05d5bb97c3e1f8c984ae96fc # v2.0.7
37+
with:
38+
version: "19"
39+
cached: true
4240

4341
- name: Install Valgrind
42+
if: ${{ matrix.sanitizer == 'valgrind' }}
4443
run: |
4544
sudo apt-get install -y valgrind
4645

0 commit comments

Comments
 (0)