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

[Feature Request]: read only mod that doesn't require RPC #3544

Closed
joroshiba opened this issue Jul 1, 2024 · 3 comments
Closed

[Feature Request]: read only mod that doesn't require RPC #3544

joroshiba opened this issue Jul 1, 2024 · 3 comments
Assignees
Labels
enhancement New feature or request external Issues created by non node team members

Comments

@joroshiba
Copy link

Implementation ideas

In our use of celestia nodes, we have some nodes which are never used for submission. We just want to participate in DAS network and read data. I believe this is common with rollkit (every node that's not a header producer just reads data). On startup celestia light nodes require an RPC connection, and if for whatever reason that rpc doesn't work it crash loops because it can't read gas price data.

If you are using a node where you never write data this is unnecessary. The light node will crash (and potentially crash loop of configured to auto restart). A read only mode would I believe not require an RPC connection at all?

@joroshiba joroshiba added the enhancement New feature or request label Jul 1, 2024
@github-actions github-actions bot added the external Issues created by non node team members label Jul 1, 2024
@Wondertan
Copy link
Member

I believe it was possible; if not, there might be a new bug. Could you share the logs you see in the crash loop? The rpc flag should be optional.

@ramin
Copy link
Collaborator

ramin commented Jul 2, 2024

@joroshiba this is unexpected, a light node can start with no RPC connection, can you share exactly what your running and i can try and recreate and figure out what might be going on

@ramin ramin self-assigned this Jul 2, 2024
@joroshiba
Copy link
Author

Sorry about delay here, did not realize RPC was optional (maybe it wasn't always?). Closing issue, just not setting the RPC solves our issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request external Issues created by non node team members
Projects
None yet
Development

No branches or pull requests

3 participants