Skip to content

Commit af73e7d

Browse files
Merge pull request #4460 from OffchainLabs/backport-v3.9.x-fix-opening-classic-msg
backport: fix opening classic-msg database
2 parents 91bf578 + d2b206e commit af73e7d

File tree

3 files changed

+11
-3
lines changed

3 files changed

+11
-3
lines changed

changelog/mkulawik-nit-4138.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
### Fixed
2+
- Fix opening classic-msg database

execution/gethexec/node.go

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -313,7 +313,13 @@ func CreateExecutionNode(
313313
var classicOutbox *ClassicOutboxRetriever
314314

315315
if l2BlockChain.Config().ArbitrumChainParams.GenesisBlockNum > 0 {
316-
classicMsgDb, err := stack.OpenDatabase("classic-msg", 0, 0, "classicmsg/", true)
316+
classicMsgDb, err := stack.OpenDatabaseWithOptions("classic-msg", node.DatabaseOptions{
317+
MetricsNamespace: "classicmsg/",
318+
Cache: 0, // will be sanitized to minimum
319+
Handles: 0, // will be sanitized to minimum
320+
ReadOnly: true,
321+
NoFreezer: true,
322+
})
317323
if dbutil.IsNotExistError(err) {
318324
log.Warn("Classic Msg Database not found", "err", err)
319325
classicOutbox = nil

system_tests/bold_challenge_protocol_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -852,9 +852,9 @@ func create2ndNodeWithConfigForBoldProtocol(
852852
l2stack, err := node.New(stackConfig)
853853
Require(t, err)
854854

855-
l2chainDb, err := l2stack.OpenDatabase("chaindb", 0, 0, "", false)
855+
l2chainDb, err := l2stack.OpenDatabaseWithOptions("chaindb", node.DatabaseOptions{})
856856
Require(t, err)
857-
l2arbDb, err := l2stack.OpenDatabase("arbdb", 0, 0, "", false)
857+
l2arbDb, err := l2stack.OpenDatabaseWithOptions("arbdb", node.DatabaseOptions{NoFreezer: true})
858858
Require(t, err)
859859

860860
AddValNodeIfNeeded(t, ctx, nodeConfig, true, "", "")

0 commit comments

Comments
 (0)