You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There's currently some ongoing work to clean up ASAN warnings on Ignition Math. In order not to introduce regressions in the future, we should set up CI that runs ASAN periodically.
Desired behavior
ASAN is run periodically and its status is tracked by the build farmer.
We considered using colcon's sanitizers like we have explained in our contribution guide, but decided for an ign-cmake native approach so this can be run without colcon.
The text was updated successfully, but these errors were encountered:
There's currently some ongoing work to clean up ASAN warnings on Ignition Math. In order not to introduce regressions in the future, we should set up CI that runs ASAN periodically.
Desired behavior
ASAN is run periodically and its status is tracked by the build farmer.
Alternatives considered
People could keep running it manually...
Implementation suggestion
The current idea is to:
BUILD_TYPE
toign-cmake
based on http://www.stablecoder.ca/2018/02/01/analyzer-build-types.htmlMost of the work should go into
ign-cmake
, and we may create a new GitHub action on https://github.com/ignition-tooling, similar to https://github.com/ignition-tooling/action-ignition-ci, or add a parameter to the existing action that enables ASAN.Additional context
We considered using colcon's sanitizers like we have explained in our contribution guide, but decided for an
ign-cmake
native approach so this can be run without colcon.The text was updated successfully, but these errors were encountered: