Skip to content

Commit

Permalink
wip: try CI_FAIL_IF_NO_TCPDUMP_FILE
Browse files Browse the repository at this point in the history
  • Loading branch information
vasild committed Dec 23, 2024
1 parent 95fc906 commit f7ce330
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
1 change: 1 addition & 0 deletions ci/test/00_setup_env_native_asan.sh
Original file line number Diff line number Diff line change
Expand Up @@ -33,3 +33,4 @@ export BITCOIN_CONFIG="\
-DAPPEND_CXXFLAGS='-std=c++23' \
-DAPPEND_CPPFLAGS='-DARENA_DEBUG -DDEBUG_LOCKORDER' \
"
export CI_FAIL_IF_NO_TCPDUMP_FILE=1
4 changes: 3 additions & 1 deletion ci/test/03_test_script.sh
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,9 @@ function traffic_monitor_end()

for ifname in $(get_interfaces) ; do
f=$(tcpdump_file "$test_name" "$ifname")
if [ ! -e "$f" ] && [ "$FILE_ENV" != "./ci/test/00_setup_env_native_asan.sh" ] ; then
echo "AAAAAAAAAAAAAAA _${CI_FAIL_IF_NO_TCPDUMP_FILE}_"
exit 1
if [ ! -e "$f" ] && [ "$CI_FAIL_IF_NO_TCPDUMP_FILE" = "1" ] ; then
# In some CI environments this script is not running as root and so the
# tcpdump errors and does not create $f. Skip silently those, but we
# need at least one where tcpdump can run and this is the ASAN one. So
Expand Down

0 comments on commit f7ce330

Please sign in to comment.