Closed tayfunelmas closed 1 month ago
Attention: Patch coverage is 0%
with 4 lines
in your changes missing coverage. Please review.
Project coverage is 71.79%. Comparing base (
e883ee2
) to head (5d160f1
).
Files | Patch % | Lines |
---|---|---|
tools/cold-store/src/cli.rs | 0.00% | 4 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This is not a final fix to
split_storage.py
, because there is still some nondeterminism that makes the test fail. But the changes below are needed since without them the test consistently fails.This PR fixes the following: 1) Change
init_cluster
to add a flag to prevent it from initializing cold-storage for archival node, since the test initializes the cold-storage after initializing the cluster. The test checks the state of the archival node before and after it has cold-storage. 2) Update theDBKind
checks for the hot db, since the hot DB seems to be usingDBKind::Hot
only, notDBKind::Archive
anymore. 3) Replacetime.sleep
calls with waiting for a number of blocks to be more deterministic on the chain progress.