Skip to content

Commit c89a252

Browse files
joseph-isaacsclaude
andcommitted
ci: optimize sanitizer test job for faster builds
- Remove redundant `-Zsanitizer=leak` (ASAN includes leak detection via detect_leaks=1) - Reduce debuginfo from level 2 to 1 (line tables sufficient for sanitizer stack traces) - Change opt-level from 0 to 1 (reduces code size and improves test runtime) - Remove verbose ASAN/LSAN options (print_stats, verbosity reduce overhead) - Remove redundant CARGO_PROFILE_*_DEBUG env vars (debuginfo in RUSTFLAGS is sufficient) - Remove separate cargo build step (nextest builds what it needs) - Remove --verbose flag from nextest (reduces log noise) Signed-off-by: Joe Isaacs <[email protected]> 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent 89b236b commit c89a252

File tree

1 file changed

+5
-12
lines changed

1 file changed

+5
-12
lines changed

.github/workflows/ci.yml

Lines changed: 5 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -396,16 +396,12 @@ jobs:
396396
- extras=s3-cache
397397
- tag=rust-test-sanitizer
398398
env:
399-
# Add debug symbols and enable ASAN/LSAN with better output
400-
RUSTFLAGS: "-A warnings -Zsanitizer=address -Zsanitizer=leak --cfg disable_loom -C debuginfo=2 -C opt-level=0 -C strip=none"
401-
ASAN_OPTIONS: "symbolize=1:print_stats=1:check_initialization_order=1:detect_leaks=1:halt_on_error=0:verbosity=1:leak_check_at_exit=1"
402-
LSAN_OPTIONS: "verbosity=1:report_objects=1"
399+
# Enable ASAN with leak detection (LSAN is included in ASAN via detect_leaks)
400+
RUSTFLAGS: "-A warnings -Zsanitizer=address --cfg disable_loom -C debuginfo=1 -C opt-level=1 -C strip=none"
401+
ASAN_OPTIONS: "symbolize=1:check_initialization_order=1:detect_leaks=1:halt_on_error=0"
403402
ASAN_SYMBOLIZER_PATH: "/usr/bin/llvm-symbolizer"
404403
# Link against DuckDB debug build
405404
VX_DUCKDB_DEBUG: "1"
406-
# Keep frame pointers for better stack traces
407-
CARGO_PROFILE_DEV_DEBUG: "true"
408-
CARGO_PROFILE_TEST_DEBUG: "true"
409405
steps:
410406
- uses: runs-on/action@v2
411407
with:
@@ -431,16 +427,13 @@ jobs:
431427
tool: nextest
432428
- name: Rust Tests
433429
run: |
434-
# Build with full debug info first (helps with caching)
435-
cargo +nightly build --locked --workspace --all-features --target x86_64-unknown-linux-gnu
436-
# Run tests with sanitizers and debug output
430+
# Run tests with sanitizers (nextest builds what it needs)
437431
cargo +nightly nextest run \
438432
--locked \
439433
--workspace \
440434
--all-features \
441435
--no-fail-fast \
442-
--target x86_64-unknown-linux-gnu \
443-
--verbose
436+
--target x86_64-unknown-linux-gnu
444437
445438
rust-test-other:
446439
name: "Rust tests (${{ matrix.os }})"

0 commit comments

Comments
 (0)