Skip to content
This repository was archived by the owner on Jul 3, 2023. It is now read-only.

Commit 2cb7f0a

Browse files
committed
Add a persistent trace using the RocksDB key-value store.
1 parent fec12bb commit 2cb7f0a

File tree

36 files changed

+3309
-450
lines changed

36 files changed

+3309
-450
lines changed

.github/workflows/coverage.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,12 @@ name: Coverage
22

33
on: [push, pull_request]
44

5+
env:
6+
# It's really `--all-features`, but not adding `persistence`, we expect the
7+
# persistence feature to go away again in the future (but if we add it
8+
# unconditionally it changes the code that's run significantly)
9+
ALMOST_ALL_FEATURES: --features "with-serde with-csv with-nexmark"
10+
511
jobs:
612
check:
713
name: Coverage
@@ -29,7 +35,7 @@ jobs:
2935
uses: actions-rs/cargo@v1
3036
with:
3137
command: llvm-cov
32-
args: --all-features --workspace --lcov --output-path coverage.info --ignore-filename-regex tests.rs
38+
args: ${{ env.ALMOST_ALL_FEATURES }} --workspace --lcov --output-path coverage.info --ignore-filename-regex tests.rs
3339

3440
- name: Upload to codecov.io
3541
uses: codecov/codecov-action@v3

.github/workflows/main.yml

Lines changed: 14 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,10 @@ env:
88
CARGO_NET_RETRY: 10
99
RUST_BACKTRACE: short
1010
RUSTUP_MAX_RETRIES: 10
11+
# It's really `--all-features`, but not adding `persistence`, we expect the
12+
# persistence feature to go away again in the future (but if we add it
13+
# unconditionally it changes the code that's run significantly)
14+
ALMOST_ALL_FEATURES: --features "with-serde with-csv with-nexmark"
1115

1216
jobs:
1317
tests:
@@ -82,7 +86,7 @@ jobs:
8286
uses: actions-rs/cargo@v1
8387
with:
8488
command: test
85-
args: --all-features --target ${{ matrix.target }}
89+
args: ${{ env.ALMOST_ALL_FEATURES }} --target ${{ matrix.target }}
8690

8791
# miri:
8892
# name: Miri
@@ -116,7 +120,7 @@ jobs:
116120
# MIRIFLAGS: "-Zmiri-tag-raw-pointers -Zmiri-disable-isolation -Zmiri-preemption-rate=0"
117121
# with:
118122
# command: miri
119-
# args: test --all-features
123+
# args: test ${{ env.ALMOST_ALL_FEATURES }}
120124

121125
test-sanitizers:
122126
name: Sanitizer Tests
@@ -181,13 +185,18 @@ jobs:
181185
RUSTDOCFLAGS: "-Z sanitizer=${{ matrix.sanitizer }}"
182186
RUSTFLAGS: "-Z sanitizer=${{ matrix.sanitizer }}"
183187
ASAN_OPTIONS: detect_stack_use_after_return=1,detect_leaks=1
188+
# Ensure the C++ code (rocksdb etc.) also gets compiled with the correct sanitizer arguments
189+
CC: "clang"
190+
CCFLAGS: "-fsanitize=${{ matrix.sanitizer }}"
191+
CXX: "clang++"
192+
CXXFLAGS: "-fsanitize=${{ matrix.sanitizer }}"
193+
ASAN_SYMBOLIZER_PATH: "/usr/bin/llvm-symbolizer-14"
184194
# Backtraces sometimes mess with sanitizers
185195
RUST_BACKTRACE: 0
186196
with:
187197
command: test
188-
# leak sanitizer is crashing on exchange tests and is very slow on
189-
# many of the proptests.
190-
args: --all-features --target ${{ matrix.target }} -Z build-std -- --skip 'exchange' --skip 'proptest'
198+
# leak sanitizer is crashing on exchange tests.
199+
args: ${{ env.ALMOST_ALL_FEATURES }} --target ${{ matrix.target }} -Z build-std -- --skip 'exchange' --skip 'proptest' --skip 'persistent'
191200

192201
clippy:
193202
name: Clippy

0 commit comments

Comments
 (0)