Skip to content

Commit

Permalink
fix: remove path based storage when builder is used (#327)
Browse files Browse the repository at this point in the history
barnabasbusa authored Oct 20, 2023

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
1 parent 4fa4937 commit d3cf3f4
Showing 1 changed file with 12 additions and 2 deletions.
14 changes: 12 additions & 2 deletions src/el/geth/geth_launcher.star
Original file line number Diff line number Diff line change
@@ -155,6 +155,11 @@ def get_config(
EXECUTION_DATA_DIRPATH_ON_CLIENT_CONTAINER,
constants.GENESIS_CONFIG_MOUNT_PATH_ON_CONTAINER + "/genesis.json",
)
elif "--builder" in extra_params:
init_datadir_cmd_str = "geth init --datadir={0} {1}".format(
EXECUTION_DATA_DIRPATH_ON_CLIENT_CONTAINER,
constants.GENESIS_CONFIG_MOUNT_PATH_ON_CONTAINER + "/genesis.json",
)
else:
init_datadir_cmd_str = "geth init --state.scheme=path --datadir={0} {1}".format(
EXECUTION_DATA_DIRPATH_ON_CLIENT_CONTAINER,
@@ -163,8 +168,13 @@ def get_config(

cmd = [
"geth",
# Disable path based storage scheme for electra fork
"{0}".format("--state.scheme=path" if electra_fork_epoch == None else ""),
# Disable path based storage scheme for electra fork or when builder image is used
# TODO: REMOVE Once geth default db is path based, and builder rebased
"{0}".format(
"--state.scheme=path"
if electra_fork_epoch != None or "--builder" not in extra_params
else ""
),
# Override prague fork timestamp for electra fork
"{0}".format(
"--override.prague=" + final_genesis_timestamp

0 comments on commit d3cf3f4

Please sign in to comment.