Skip to content

Commit

Permalink
fix(vm): Fix missing experimental VM config (#2629)
Browse files Browse the repository at this point in the history
## What ❔

Uses default experimental VM config if one is not supplied when
initializing the state keeper.

## Why ❔

Otherwise, the state keeper will panic on start if the config is
missing. Logically, this config shouldn't be required for it to work
(unlike for the VM playground).

## Checklist

- [x] PR title corresponds to the body of PR (we generate changelog
entries from PRs).
- [x] Code has been formatted via `zk fmt` and `zk lint`.
  • Loading branch information
slowli authored Aug 9, 2024
1 parent b752a54 commit e07a39d
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion core/bin/zksync_server/src/node_builder.rs
Original file line number Diff line number Diff line change
Expand Up @@ -249,7 +249,11 @@ impl MainNodeBuilder {
try_load_config!(wallets.state_keeper),
);
let db_config = try_load_config!(self.configs.db_config);
let experimental_vm_config = try_load_config!(self.configs.experimental_vm_config);
let experimental_vm_config = self
.configs
.experimental_vm_config
.clone()
.unwrap_or_default();
let main_node_batch_executor_builder_layer =
MainBatchExecutorLayer::new(sk_config.save_call_traces, OPTIONAL_BYTECODE_COMPRESSION)
.with_fast_vm_mode(experimental_vm_config.state_keeper_fast_vm_mode);
Expand Down

0 comments on commit e07a39d

Please sign in to comment.