Skip to content

Commit f539bec

Browse files
riptlripatel-fd
authored andcommitted
Remove backtest checksum
Instead of checking whether firedancer-dev backtest randomly modified inputs, just ban backtest from modifying inputs.
1 parent 90f5998 commit f539bec

File tree

4 files changed

+1
-77
lines changed

4 files changed

+1
-77
lines changed

.github/workflows/benchmark.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ jobs:
6060
DUMP_DIR=../dump ./src/flamenco/runtime/tests/run_ledger_backtest.sh \
6161
-l mainnet-368528500-perf \
6262
-y 5 -m 2000000 -e 368529500 -c 3.0.0 \
63-
--log baseline.log --skip-checksum
63+
--log baseline.log
6464
SPS_BASELINE=$(grep "Backtest playback done." baseline.log | awk -F 'sec/slot: ' '{print $2}')
6565
TOTAL_ELAPSED_BASELINE=$(grep "Backtest playback done." baseline.log | awk -F 'elapsed: ' '{print $2}' | awk '{print $1}')
6666
SNAPSHOT_LOAD_BASELINE=$(grep "accounts from snapshot in" baseline.log | awk -F 'accounts from snapshot in ' '{print $2}' | awk '{print $1}')

contrib/test/ledger_common.sh

Lines changed: 0 additions & 53 deletions
Original file line numberDiff line numberDiff line change
@@ -7,56 +7,3 @@ echo_notice() {
77
echo_error() {
88
echo -e "\033[31m$1$2\033[0m"
99
}
10-
11-
create_checksum() {
12-
# generate checksum of ledger
13-
find $DUMP/$LEDGER -type f \( -name "snapshot*" -o -name "genesis*" -o -wholename "*rocksdb*" \) -print0 | xargs -0 md5sum | sort > $DUMP/$LEDGER/checksum.txt
14-
}
15-
16-
redownload_ledger_or_fail() {
17-
if [[ $REDOWNLOAD -eq 1 ]]; then
18-
echo_notice "Redownloading ledger $LEDGER"
19-
rm -rf $DUMP/$LEDGER
20-
download_and_extract_ledger
21-
create_checksum
22-
else
23-
echo_error "If you want to redownload the ledger, please remove the -nr flag and try again."
24-
exit 1
25-
fi
26-
}
27-
28-
check_ledger_checksum() {
29-
if [ ! -f "$DUMP/$LEDGER/checksum.txt" ]; then
30-
echo_error "original checksum file does not exist."
31-
exit 1
32-
fi
33-
34-
# check existing checksum
35-
find $DUMP/$LEDGER -type f \( -name "snapshot*" -o -name "genesis*" -o -wholename "*rocksdb*" \) -print0 | xargs -0 md5sum | sort > $DUMP/$LEDGER/checksum_temp.txt
36-
37-
if ! diff -u $DUMP/$LEDGER/checksum.txt $DUMP/$LEDGER/checksum_temp.txt > /dev/null; then
38-
echo_error "Checksum mismatch for existing ledger $LEDGER. This likely means the ledger is corrupted. \
39-
vimdiff $DUMP/$LEDGER/checksum.txt $DUMP/$LEDGER/checksum_temp.txt to see the differences."
40-
else
41-
rm $DUMP/$LEDGER/checksum_temp.txt
42-
fi
43-
}
44-
45-
check_ledger_checksum_and_redownload() {
46-
if [ ! -f "$DUMP/$LEDGER/checksum.txt" ]; then
47-
echo_error "original checksum file does not exist."
48-
redownload_ledger_or_fail
49-
fi
50-
51-
# check existing checksum
52-
find $DUMP/$LEDGER -type f \( -name "snapshot*" -o -name "genesis*" -o -wholename "*rocksdb*" \) -print0 | xargs -0 md5sum | sort > $DUMP/$LEDGER/checksum_temp.txt
53-
54-
if ! diff -u $DUMP/$LEDGER/checksum.txt $DUMP/$LEDGER/checksum_temp.txt > /dev/null; then
55-
echo_error "Checksum mismatch for existing ledger $LEDGER. This likely means the ledger is corrupted. \
56-
vimdiff $DUMP/$LEDGER/checksum.txt $DUMP/$LEDGER/checksum_temp.txt to see the differences. \
57-
If you are running locally, please disable redownload with -nr and inspect the differences. "
58-
redownload_ledger_or_fail
59-
else
60-
rm $DUMP/$LEDGER/checksum_temp.txt
61-
fi
62-
}

contrib/test/run_solcap_tests.sh

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -44,9 +44,6 @@ download_and_extract_ledger() {
4444

4545
if [[ ! -e $DUMP/$LEDGER && SKIP_INGEST -eq 0 ]]; then
4646
download_and_extract_ledger
47-
create_checksum
48-
else
49-
check_ledger_checksum_and_redownload
5047
fi
5148

5249
rm -rf $DUMP/$LEDGER/devnet-398736132_current.toml
@@ -68,6 +65,3 @@ $OBJDIR/bin/firedancer-dev configure fini all --config $DUMP/$LEDGER/devnet-3987
6865

6966
$OBJDIR/bin/fd_solcap_import $DUMP/$LEDGER/bank_hash_details/ $DUMP/$LEDGER/solana.solcap
7067
$OBJDIR/bin/fd_solcap_diff $DUMP/$LEDGER/solana.solcap $DUMP/$LEDGER/fd.solcap -v 4
71-
72-
# check that the ledger is not corrupted after a run
73-
check_ledger_checksum

src/flamenco/runtime/tests/run_ledger_backtest.sh

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ HUGE_TLBFS_MOUNT_PATH=${HUGE_TLBFS_MOUNT_PATH:="/mnt/.fd"}
2222
HUGE_TLBFS_ALLOW_HUGEPAGE_INCREASE=${HUGE_TLBFS_ALLOW_HUGEPAGE_INCREASE:="true"}
2323
HAS_INCREMENTAL="false"
2424
REDOWNLOAD=1
25-
SKIP_CHECKSUM=0
2625
DEBUG=( )
2726
WATCH=( )
2827

@@ -111,10 +110,6 @@ while [[ $# -gt 0 ]]; do
111110
DEBUG=( gdb -q -x contrib/debug.gdb --args )
112111
shift
113112
;;
114-
--skip-checksum)
115-
SKIP_CHECKSUM=1
116-
shift
117-
;;
118113
--log)
119114
LOG="$2"
120115
shift
@@ -166,13 +161,6 @@ download_and_extract_ledger() {
166161

167162
if [[ ! -e $DUMP/$LEDGER && SKIP_INGEST -eq 0 ]]; then
168163
download_and_extract_ledger
169-
if [[ $SKIP_CHECKSUM -eq 0 ]]; then
170-
create_checksum
171-
fi
172-
else
173-
if [[ $SKIP_CHECKSUM -eq 0 ]]; then
174-
check_ledger_checksum_and_redownload
175-
fi
176164
fi
177165

178166
chmod -R 0700 $DUMP/$LEDGER
@@ -244,11 +232,6 @@ sudo rm -rf $DUMP/$LEDGER/backtest.blockstore $DUMP/$LEDGER/backtest.funk &> /de
244232

245233
echo "Log for ledger $LEDGER at $LOG"
246234

247-
# check that the ledger is not corrupted after a run
248-
if [[ $SKIP_CHECKSUM -eq 0 ]]; then
249-
check_ledger_checksum
250-
fi
251-
252235
if [ "$status" -eq 0 ]; then
253236
echo_notice "Finished on-demand ingest and replay\n"
254237
exit 0

0 commit comments

Comments
 (0)