Skip to content

Commit

Permalink
Fix check for asan logs
Browse files Browse the repository at this point in the history
It needs to have a non-zero exit code when the logs exist, and the logs
should be cleaned up before the job fails.
  • Loading branch information
benjaminwinger committed Jul 18, 2023
1 parent 9f7007a commit 222e5be
Showing 1 changed file with 8 additions and 4 deletions.
12 changes: 8 additions & 4 deletions .github/workflows/ci-workflow.yml
Original file line number Diff line number Diff line change
Expand Up @@ -110,12 +110,16 @@ jobs:
continue-on-error: true

- name: Report ASan log
run: ls /tmp/asan.log* 1>/dev/null 2>&1 && (cat /tmp/asan.log*; exit 1) || exit 0
run: |
if ls /tmp/asan.log* 1>/dev/null 2>&1;
then cat /tmp/asan.log*;
rm -rf /tmp/asan.log*
exit 1;
else
exit 0;
fi
shell: bash

- name: Clean up ASan log
run: rm -rf /tmp/asan.log*

clang-build-test:
name: clang build & test
needs: [clang-formatting-check]
Expand Down

0 comments on commit 222e5be

Please sign in to comment.