Skip to content

Commit 854ea72

Browse files
committed
fix issues in weekly reports
1 parent 28ec119 commit 854ea72

2 files changed

Lines changed: 39 additions & 2 deletions

File tree

.github/workflows/weekly-checks.yml

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -107,9 +107,17 @@ jobs:
107107
- name: Run tests with ${{ matrix.sanitizer.name }}
108108
env:
109109
LD_LIBRARY_PATH: /opt/simgrid/lib:/opt/fsmod/lib:/usr/local/lib
110-
PYTHONPATH: /opt/simgrid/lib/python3.12/site-packages:/usr/local/lib/python3.12/dist-packages
111110
run: |
112111
export ${{ matrix.sanitizer.env_options }}
112+
# Dynamically find Python paths for SimGrid, FSMod, and DTLMod
113+
SIMGRID_PYTHON_PATH=$(find /opt/simgrid/lib -type d -path "*/python*/site-packages" 2>/dev/null | head -1)
114+
[ -z "$SIMGRID_PYTHON_PATH" ] && SIMGRID_PYTHON_PATH=$(find /opt/simgrid/lib -type d -path "*/python*/dist-packages" 2>/dev/null | head -1)
115+
FSMOD_PYTHON_PATH=$(find /opt/fsmod/lib -type d -path "*/python*/site-packages" 2>/dev/null | head -1)
116+
[ -z "$FSMOD_PYTHON_PATH" ] && FSMOD_PYTHON_PATH=$(find /opt/fsmod/lib -type d -path "*/python*/dist-packages" 2>/dev/null | head -1)
117+
DTLMOD_PYTHON_PATH=$(find /usr/local/lib -type d -path "*/python*/dist-packages" 2>/dev/null | head -1)
118+
[ -z "$DTLMOD_PYTHON_PATH" ] && DTLMOD_PYTHON_PATH=$(find /usr/local/lib -type d -path "*/python*/site-packages" 2>/dev/null | head -1)
119+
export PYTHONPATH="$SIMGRID_PYTHON_PATH:$FSMOD_PYTHON_PATH:$DTLMOD_PYTHON_PATH"
120+
echo "PYTHONPATH=$PYTHONPATH"
113121
cd build
114122
./unit_tests 2>&1 | tee sanitizer-output.txt
115123
cd test/python
@@ -298,7 +306,8 @@ jobs:
298306
for report in reports/*-report/*.txt; do
299307
if [ -f "$report" ]; then
300308
echo "### $(basename $(dirname $report))" >> weekly-report.md
301-
if grep -q "ERROR:\|WARNING:" "$report" 2>/dev/null; then
309+
# Filter out benign warnings from SimGrid context switching and Valgrind redirections
310+
if grep -E "ERROR:|WARNING:" "$report" 2>/dev/null | grep -v "ASan is ignoring requested __asan_handle_no_return" | grep -v "new redirection conflicts" | grep -q .; then
302311
echo "❌ Issues detected" >> weekly-report.md
303312
else
304313
echo "✅ No issues detected" >> weekly-report.md

test/valgrind.supp

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,13 +8,27 @@
88
obj:*/libsimgrid.so*
99
}
1010

11+
{
12+
simgrid_boost_context
13+
Memcheck:Leak
14+
...
15+
obj:*/libboost_context.so*
16+
}
17+
1118
{
1219
boost_filesystem
1320
Memcheck:Leak
1421
...
1522
obj:*/libboost_filesystem.so*
1623
}
1724

25+
{
26+
fsmod_library
27+
Memcheck:Leak
28+
...
29+
obj:*/libfsmod.so*
30+
}
31+
1832
{
1933
python_interpreter
2034
Memcheck:Leak
@@ -28,3 +42,17 @@
2842
...
2943
obj:*/libgtest.so*
3044
}
45+
46+
{
47+
libstdc++_allocations
48+
Memcheck:Leak
49+
...
50+
obj:*/libstdc++.so*
51+
}
52+
53+
{
54+
graphviz_cgraph
55+
Memcheck:Leak
56+
...
57+
obj:*/libcgraph.so*
58+
}

0 commit comments

Comments
 (0)