Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

merge queue: embarking main (22852bc) and #8107 together #8108

Closed
wants to merge 2 commits into from
Closed

merge queue: embarking main (22852bc) and #8107 together #8108

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
c94af36
Ignore segmentation fault errors when shutting down Docker in tests
teor2345 Dec 13, 2023
84d40bf
Merge of #8107
mergify[bot] Dec 13, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 9 additions & 1 deletion .github/workflows/sub-test-zebra-config.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,15 @@ jobs:

# If grep found the pattern, exit with the Docker container exit status
if [ $LOGS_EXIT_STATUS -eq 0 ]; then
exit $EXIT_STATUS;
# We can't diagnose or fix these errors, so we're just ignoring them for now.
# They don't actually impact the test because they happen after it succeeds.
# See ticket #7898 for details.
if [ $EXIT_STATUS -eq 137 ] || [ $EXIT_STATUS -eq 139 ]; then
echo "Warning: ignoring docker exit status $EXIT_STATUS";
exit 0;
else
exit $EXIT_STATUS;
fi
fi

# Handle other potential errors here
Expand Down