Merge pull request #1494 from helium/adt/dont-take-early-snapshots

Handle when the chain height is too low to take a snapshot
helium · Dec 8, 2022 · 4b704a5 · 4b704a5
2 parents 01ccfb1 + 467c821
commit 4b704a5
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/ledger/v1/blockchain_ledger_snapshot_v1.erl b/src/ledger/v1/blockchain_ledger_snapshot_v1.erl
@@ -171,8 +171,12 @@ snapshot(Ledger0, Blocks, Infos) ->
     %% too far behind. Nodes that are reliably syncing will have a rocksdb checkpoint/snapshot at this
     %% height which is safer to use.
     {ok, Height} = blockchain_ledger_v1:current_height(Ledger0),
-    {ok, SnapLedger} = blockchain_ledger_v1:has_snapshot(Height - blockchain_txn:block_delay(), Ledger0),
-    snapshot(SnapLedger, Blocks, Infos, active).
+    case blockchain_ledger_v1:has_snapshot(Height - blockchain_txn:block_delay(), Ledger0) of
+        {ok, SnapLedger} ->
+            snapshot(SnapLedger, Blocks, Infos, active);
+        Error ->
+            Error
+    end.
 
 -spec snapshot(
     blockchain_ledger_v1:ledger(),