Skip to content

feat(avm): test avm-bulk with sanitizers CI job#19548

Open
defkit wants to merge 1 commit into
nextfrom
sn/13_jun_san_build_test_bulk
Open

feat(avm): test avm-bulk with sanitizers CI job#19548
defkit wants to merge 1 commit into
nextfrom
sn/13_jun_san_build_test_bulk

Conversation

@defkit

@defkit defkit commented Jan 13, 2026

Copy link
Copy Markdown
Contributor

With #19512 #19507 we got worried about UBs

This PR introduces CI job which runs avm bulk test (see yarn-project/bb-prover) with ASAN and MSAN sanitizers

@defkit defkit requested a review from dbanks12 January 13, 2026 13:58
@defkit defkit requested a review from charlielye as a code owner January 13, 2026 13:58
cmake --build build-asan --target bb-avm && \
mv build-asan/bin/bb-avm build/bin/bb-avm && \
cd /root/aztec-packages/yarn-project/bb-prover && \
yarn test -t 'AVM proven bulk test' --verbose

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Im not sure how good idea it is to test it during the build

import { AvmProvingTester } from './avm_proving_tester.js';

const TIMEOUT = 180_000;
const TIMEOUT = 1_800_000;

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

running this test with sanitizers enabled takes a little bit more time

@AztecBot

Copy link
Copy Markdown
Collaborator

Flakey Tests

🤖 says: This CI run detected 1 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (8;;http://ci.aztec-labs.com/23db5404d31258f6�23db5404d31258f68;;�): yarn-project/end-to-end/scripts/run_test.sh web3signer src/composed/web3signer/e2e_multi_validator_node_key_store.test.ts (37s) (code: 1) (\033jewelofchaos9\033: feat(avm): test avm-bulk with sanitizers CI job)

@charlielye charlielye removed their request for review February 5, 2026 11:51
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