Skip to content

CI: enable MSAN memory origins, sanitize all ITs - #1695

Open
678098 wants to merge 1 commit into
bloomberg:mainfrom
678098:260805_san
Open

CI: enable MSAN memory origins, sanitize all ITs#1695
678098 wants to merge 1 commit into
bloomberg:mainfrom
678098:260805_san

Conversation

@678098

@678098 678098 commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator
  • Run all ITs in sanitizers CI
  • Run sanitized ITs in parallel
  • Enable MSAN memory origins

Comment thread .github/workflows/sanitize.yaml Outdated
if: ${{ matrix.fuzz == 'off' }}
run: |
docker run --rm sanitizer-${{ github.job }} /blazingmq/cmake.bld/Linux/run-it.sh -k breathing --bmq-tolerate-dirty-shutdown
docker run --rm sanitizer-${{ github.job }} /blazingmq/cmake.bld/Linux/run-it.sh --bmq-tolerate-dirty-shutdown

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I recall, this will take longer than the 6 hour github actions timeout.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My point is just to run it and see if I have more crashes. Breathing tests path doesn't cover everything.
Maybe, there is enough time with matrix parallelization

Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
@678098
678098 marked this pull request as ready for review August 14, 2026 20:50
@678098
678098 requested a review from a team as a code owner August 14, 2026 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants