From 6a96cb8abece3588edba6c6199e001a745241c4b Mon Sep 17 00:00:00 2001 From: Taegyun Kim Date: Thu, 21 Aug 2025 15:42:53 -0400 Subject: [PATCH 1/5] use llvm 19 via actions --- .github/workflows/profiling-native.yml | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/.github/workflows/profiling-native.yml b/.github/workflows/profiling-native.yml index 33ddf60f899..668d35762fd 100644 --- a/.github/workflows/profiling-native.yml +++ b/.github/workflows/profiling-native.yml @@ -32,13 +32,9 @@ jobs: with: python-version: ${{ matrix.python-version }} - - name: Install llvm 19 - run: | - # Ubuntu-24.04 GH actions image has llvm-18, but we use 19 as it's - # the latest one available. - wget https://apt.llvm.org/llvm.sh - chmod +x llvm.sh - sudo ./llvm.sh 19 + - uses: KyleMayes/install-llvm-action@a7a1a882e2d06ebe05d5bb97c3e1f8c984ae96fc # v2.0.7 + with: + version: "19" - name: Install Valgrind run: | From 72deef0b3b5621abea491da8a1c979f32009035a Mon Sep 17 00:00:00 2001 From: Taegyun Kim Date: Fri, 29 Aug 2025 00:12:54 +0900 Subject: [PATCH 2/5] Enable caching for LLVM installation --- .github/workflows/profiling-native.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/profiling-native.yml b/.github/workflows/profiling-native.yml index 668d35762fd..4c467f67662 100644 --- a/.github/workflows/profiling-native.yml +++ b/.github/workflows/profiling-native.yml @@ -35,6 +35,7 @@ jobs: - uses: KyleMayes/install-llvm-action@a7a1a882e2d06ebe05d5bb97c3e1f8c984ae96fc # v2.0.7 with: version: "19" + cached: true - name: Install Valgrind run: | From caede9ef5286ea95600d711b26ab356440faf389 Mon Sep 17 00:00:00 2001 From: Taegyun Kim Date: Thu, 28 Aug 2025 11:30:47 -0400 Subject: [PATCH 3/5] also run when this file is modified --- .github/workflows/profiling-native.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/profiling-native.yml b/.github/workflows/profiling-native.yml index 4c467f67662..30216ffe003 100644 --- a/.github/workflows/profiling-native.yml +++ b/.github/workflows/profiling-native.yml @@ -9,6 +9,7 @@ on: paths: - ddtrace/internal/datadog/profiling/** - ddtrace/profiling/** + - .github/workflows/profiling-native.yml workflow_dispatch: {} jobs: From 56d8093109faeaec63de73784bf593c8789d64f1 Mon Sep 17 00:00:00 2001 From: Taegyun Kim Date: Thu, 28 Aug 2025 11:32:33 -0400 Subject: [PATCH 4/5] install valgrind only if valgrind is used --- .github/workflows/profiling-native.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/profiling-native.yml b/.github/workflows/profiling-native.yml index 30216ffe003..8c1ab575150 100644 --- a/.github/workflows/profiling-native.yml +++ b/.github/workflows/profiling-native.yml @@ -39,6 +39,7 @@ jobs: cached: true - name: Install Valgrind + if: ${{ matrix.sanitizer == 'valgrind' }} run: | sudo apt-get install -y valgrind From 533d7b350b1cbccfac49693bb698bc5bf3210b11 Mon Sep 17 00:00:00 2001 From: Taegyun Kim Date: Thu, 28 Aug 2025 11:34:06 -0400 Subject: [PATCH 5/5] increase timeout --- .github/workflows/profiling-native.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/profiling-native.yml b/.github/workflows/profiling-native.yml index 8c1ab575150..5c0df634f63 100644 --- a/.github/workflows/profiling-native.yml +++ b/.github/workflows/profiling-native.yml @@ -15,7 +15,7 @@ on: jobs: test: runs-on: ${{ matrix.os }} - timeout-minutes: 7 + timeout-minutes: 9 strategy: fail-fast: false matrix: