CI: pin ubuntu runner to 20.04 to avoid sanitizer failures #767
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Recently tests are failing with
AddressSanitizer:DEADLYSIGNAL
. A ctest timeout occurs and causes the test suite to fail. This does NOT occur when actually running the tests, but instead during the build/linking step. In the test CMakeLists, there is a call towhich defers test discovery until runtime instead of scanning source files. In effect, the causes the test executable to run with the flag
--gtest_list_tests
.The cmake error points to this call as failing:
It appears that some combination of google test, ubuntu, and VM/Containers is causes the address sanitizer to trigger an infinite loop which causes the timeout.
See quantumlib/Stim#717 and google/googletest#4491
Changes
ubuntu-latest
toubuntu-20.04
in github CI runnersTarget Environment support