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

Review and update Full Node How to Guide #517

Open
MSevey opened this issue Nov 20, 2024 · 2 comments · May be fixed by rollkit/rollkit#1952
Open

Review and update Full Node How to Guide #517

MSevey opened this issue Nov 20, 2024 · 2 comments · May be fixed by rollkit/rollkit#1952
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@MSevey
Copy link
Contributor

MSevey commented Nov 20, 2024

Run Full node guide steps don't work. Command has old flags, not sure what minimal flags need to be.

json flag is outdated
needs sequencer rollup id flag

need local da? mock da doesn't seem to work?
might need local sequencer as well?

@MSevey MSevey self-assigned this Nov 20, 2024
@MSevey MSevey added this to Rollkit Nov 20, 2024
@MSevey
Copy link
Contributor Author

MSevey commented Dec 4, 2024

I tested the how to guide using the gm rollup created from the ignite guide: https://rollkit.dev/guides/ignite-rollkit

To better separate the sequencer from the full node I duplicated the gm/ directory to gm_fn/ to start the full node guide.

Then I followed the steps of the guide.

I had to update the start command with 2 changes.

  1. Remove the --json-rpc.ws-address 127.0.0.1:8547 flag as it seems outdated
  2. Add the --minimum-gas-prices 0stake flag to address the error about needing it defined

The resulting command is this:

rollkit start --rollkit.aggregator=false \
  --rollkit.da_address http://127.0.0.1:7980 \
  --rpc.laddr tcp://127.0.0.1:46657 \
  --grpc.address 127.0.0.1:9390 \
  --minimum-gas-prices 0stake \
  --p2p.seeds [email protected]:26656 \
  --p2p.laddr "0.0.0.0:46656" \
  --api.address tcp://localhost:1318

This allows the chain to start.

We should mention explicitly why we are including all the other flags, and that it is to avoid the duplicate port use of the other node running. That might not be obvious to users.

Once it is running, I start seeing these errors.

2:42PM ERR failed to retrieve block from DALC daHeight=53 errors="failed to retrieve block: failed to get IDs: given height is from the future\nfailed to retrieve block: failed to get IDs: given height is from the future\nfailed to retrieve block: failed to get IDs: given height is from the future\nfailed to retrieve block: failed to get IDs: given height is from the future\nfailed to retrieve block: failed to get IDs: given height is from the future\nfailed to retrieve block: failed to get IDs: given height is from the future\nfailed to retrieve block: failed to get IDs: given height is from the future\nfailed to retrieve block: failed to get IDs: given height is from the future\nfailed to retrieve block: failed to get IDs: given height is from the future\nfailed to retrieve block: failed to get IDs: given height is from the future" module=BlockManager

I'm wondering if there is something else that needs to be updated with this guide, maybe a sequencer setting.

I was running the local-da and the local-sequencer.

@MSevey MSevey moved this from Todo to In Progress in Rollkit Dec 4, 2024
@MSevey
Copy link
Contributor Author

MSevey commented Dec 5, 2024

@gupadhyaya here are some minor edits I had locally that would be good to include.

#528

Specifically:

  • Remove json flag
  • Add min gas flag
  • specifically call out the other flags for new ports
  • Add a CHAIN_ID terminal variable to make the commands copy paste.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
Status: In Progress
3 participants