Skip to content

Commit

Permalink
Updtae some CI jobs
Browse files Browse the repository at this point in the history
* Move ThreadSanitizer job to the scheduled jobs (once a day): all out tests
  are intrinsically mono-thread and this job takes quite some time
  • Loading branch information
IvanNardi committed Jan 7, 2025
1 parent c3d19be commit 172bea9
Show file tree
Hide file tree
Showing 2 changed files with 25 additions and 10 deletions.
12 changes: 2 additions & 10 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -116,8 +116,8 @@ jobs:
msan: "--with-sanitizer"
nBPF: ""
- compiler: "clang-18" # "Newest" clang easily available. See also below...
ar: "llvm-ar-18"
ranlib: "llvm-ranlib-18"
#ar: "llvm-ar-18"
#ranlib: "llvm-ranlib-18"
os: ubuntu-24.04
arch: "x86_64"
gcrypt: ""
Expand All @@ -126,14 +126,6 @@ jobs:
msan: "--with-sanitizer"
nBPF: ""
lto_gold_linker: "--with-lto-and-gold-linker"
- compiler: "cc"
os: ubuntu-latest
arch: "x86_64"
gcrypt: ""
pcre: "--with-pcre2"
maxminddb: "--with-maxminddb"
msan: "--with-thread-sanitizer"
nBPF: ""
- compiler: "cc"
os: ubuntu-latest
arch: "x86_64"
Expand Down
23 changes: 23 additions & 0 deletions .github/workflows/build_scheduled.yml
Original file line number Diff line number Diff line change
Expand Up @@ -109,3 +109,26 @@ jobs:
name: ndpi-performance
path: ndpi-performance-upload
retention-days: 7

threadsanitizer:
name: Thread Sanitizer (ubuntu-latest)
runs-on: ubuntu-latest
env:
CFLAGS: -Wextra -Werror
steps:
- uses: actions/checkout@v4
with:
fetch-depth: 0
- name: Install Ubuntu Prerequisites
run: |
sudo apt-get update
sudo apt-get install autoconf automake libtool pkg-config gettext flex bison libjson-c libpcap-dev rrdtool librrd-dev parallel
- name: Configure nDPI library
run: |
./autogen.sh --enable-option-checking=fatal --with-thread-sanitizer
- name: Build nDPI library
run: |
make -j $(nproc) all
- name: Tests
run: |
NDPI_FORCE_PARALLEL_UTESTS=1 NDPI_SKIP_PARALLEL_BAR=1 ./tests/do.sh

0 comments on commit 172bea9

Please sign in to comment.