Skip to content

Commit

Permalink
[simtest]: clean up /tmpfs dir and decrease number of threads (Mysten…
Browse files Browse the repository at this point in the history
…Labs#9273)

## Description 
1. clean up /tmpfs dir
2. Allow setting number of CPU threads in the workflow

## Test Plan 

https://github.com/MystenLabs/sui/actions/runs/4418509222/jobs/7745727449
  • Loading branch information
ebmifa authored Mar 14, 2023
1 parent 50205d6 commit 728d8aa
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 3 deletions.
7 changes: 6 additions & 1 deletion .github/workflows/simulator-nightly.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,13 +39,18 @@ jobs:
# Runs tests much faster - disables signing and verification
USE_MOCK_CRYPTO: 1
CARGO_TERM_COLOR: always
NUM_CPUS: 24

steps:
- name: Cargo clean
run: |
cd ~/sui && cargo clean && git restore Cargo.lock
- name: Checkout sui repo and cargo clean
- name: Tmpfs clean
run: |
sudo rm -rf /tmp/*.*
- name: Checkout sui repo
run: |
cd ~/sui && git fetch origin && git rebase origin/main
Expand Down
7 changes: 5 additions & 2 deletions scripts/simtest/simtest-run.sh
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ NUM_CPUS=$(cat /proc/cpuinfo | grep processor | wc -l) # ubuntu
# NUM_CPUS=64 # We can increase this later if needed

# filter out some tests that give spurious failures.
TEST_FILTER="(not test(test_move_call_args_linter_command)) & (not test(test_package_publish_command))"
TEST_FILTER="(not test(~cli_tests))"

DATE=$(date +%s)
SEED="$DATE"
Expand All @@ -22,7 +22,8 @@ MSIM_TEST_NUM=30 \
scripts/simtest/cargo-simtest simtest \
--package sui \
--package sui-core \
--profile simtestnightly
--profile simtestnightly \
-E "$TEST_FILTER"

# create logs directory
SIMTEST_LOGS_DIR=~/simtest_logs
Expand All @@ -43,6 +44,8 @@ for SUB_SEED in `seq 1 $NUM_CPUS`; do
--test-threads 1 \
--profile simtestnightly \
-E "$TEST_FILTER" > "$LOG_FILE" 2>&1 &

grep -Hn FAIL "$LOG_FILE"
done

# wait for all the jobs to end
Expand Down

0 comments on commit 728d8aa

Please sign in to comment.