Simplify env vars; don't run ignored tests (MystenLabs#9144)

quyetlq · Mar 12, 2023 · 9f04d86 · 9f04d86
1 parent ce36add
commit 9f04d86
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/.github/workflows/simulator-nightly.yml b/.github/workflows/simulator-nightly.yml
@@ -38,8 +38,6 @@ jobs:
       RUST_LOG: off
       # Runs tests much faster - disables signing and verification
       USE_MOCK_CRYPTO: 1
-      MSIM_TEST_NUM: 30
-      SIM_STRESS_TEST_DURATION_SECS: 300
       CARGO_TERM_COLOR: always
 
     steps:

diff --git a/scripts/simtest/simtest-run.sh b/scripts/simtest/simtest-run.sh
@@ -18,6 +18,7 @@ SEED="$DATE"
 # TODO: this logs directly to stdout since it is not being run in parallel. is that ok?
 MSIM_TEST_SEED="$SEED" \
 MSIM_WATCHDOG_TIMEOUT_MS=60000 \
+MSIM_TEST_NUM=30 \
 scripts/simtest/cargo-simtest simtest \
   --package sui \
   --package sui-core \
@@ -35,11 +36,10 @@ for SUB_SEED in `seq 1 $NUM_CPUS`; do
 
   # --test-threads 1 is important: parallelism is achieved via the for loop
   MSIM_TEST_SEED="$SEED" \
-  MSIM_TEST_NUM=1 \
   MSIM_WATCHDOG_TIMEOUT_MS=60000 \
+  SIM_STRESS_TEST_DURATION_SECS=300 \
   scripts/simtest/cargo-simtest simtest \
     --package sui-benchmark \
-    --run-ignored all \
     --test-threads 1 \
     --profile simtestnightly \
     -E "$TEST_FILTER" > "$LOG_FILE" 2>&1 &