Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dev mode is not supported for legacy engine #13438

Open
1 task done
PowVT opened this issue Dec 18, 2024 · 5 comments
Open
1 task done

Dev mode is not supported for legacy engine #13438

PowVT opened this issue Dec 18, 2024 · 5 comments
Labels
C-bug An unexpected or incorrect behavior D-good-first-issue Nice and easy! A great choice to get started

Comments

@PowVT
Copy link

PowVT commented Dec 18, 2024

Describe the bug

when running a dev node per the examples/custom-dev-node example using v1.1.3 i get this error:
Dev mode is not supported for legacy engine

Steps to reproduce

cd examples/custom-dev-node

cargo build

cargo run

Node logs


Platform(s)

No response

Container Type

Not running in a container

What version/commit are you on?

v1.1.3

What database version are you on?

idk

Which chain / network are you on?

dev

What type of node are you running?

Archive (default)

What prune config do you use, if any?

dev

If you've built Reth from source, provide the full command you used

No response

Code of Conduct

  • I agree to follow the Code of Conduct
@PowVT PowVT added C-bug An unexpected or incorrect behavior S-needs-triage This issue needs to be labelled labels Dec 18, 2024
@mattsse mattsse added D-good-first-issue Nice and easy! A great choice to get started and removed S-needs-triage This issue needs to be labelled labels Dec 19, 2024
@PowVT
Copy link
Author

PowVT commented Dec 19, 2024

@mattsse any pointers on where/ why this is occurring? Would love to help fix it.

I know it is triggering this line in the node builder:

eyre::bail!("Dev mode is not supported for legacy engine")

but not sure why that line is there and what the context is for a fix. Thanks

@mattsse
Copy link
Collaborator

mattsse commented Dec 19, 2024

@PowVT this currently would require replicating this

let engine_tree_config = TreeConfig::default()
.with_persistence_threshold(engine_args.persistence_threshold)
.with_memory_block_buffer_target(engine_args.memory_block_buffer_target);
let handle = builder
.with_types_and_provider::<EthereumNode, BlockchainProvider2<_>>()
.with_components(EthereumNode::components())
.with_add_ons(EthereumAddOns::default())
.launch_with_fn(|builder| {
let launcher = EngineNodeLauncher::new(
builder.task_executor().clone(),
builder.config().datadir(),
engine_tree_config,
);
builder.launch_with(launcher)
})
.await?;
handle.node_exit_future.await

but legacy is scheduled for deprecating so perhaps not worth doing rn

@PedroRosalba
Copy link

May I try my hand at this?

@PowVT
Copy link
Author

PowVT commented Dec 22, 2024

May I try my hand at this?

I would probably hold off since like @mattsse said they are deprecating legacy and that PR will most likely cover the --dev mode case

@VyuduInc
Copy link

May I try my hand at this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug An unexpected or incorrect behavior D-good-first-issue Nice and easy! A great choice to get started
Projects
Status: Todo
Development

No branches or pull requests

4 participants