docs: point Docker setup at live consensus gRPC endpoints - #2440
Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request enhances the clarity and accuracy of network endpoint documentation, particularly for Docker setup and testnet configurations. By standardizing terminology to 'consensus endpoint' and explicitly detailing the requirement for both RPC and gRPC services, the changes aim to provide users with more precise and comprehensive guidance for connecting to various networks. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the documentation to clarify the use of consensus endpoints that provide both RPC and gRPC services, particularly for Docker setups. The changes are generally good and improve clarity. I've added two comments: one suggests renaming some environment variables in the Docker setup guide for better readability and consistency, and the other points out a potential inconsistency created by removing an endpoint from a list in the Mocha testnet documentation while it's still being used in examples elsewhere.
I am having trouble creating individual review comments. Click here to see my feedback.
app/operate/getting-started/docker/page.mdx (92-98)
The variable names RPC_URL and RPC_PORT are misleading now that they refer to a consensus host and a gRPC port. To improve clarity for users, consider renaming them.
For example, you could use CONSENSUS_HOST and GRPC_PORT:
export CONSENSUS_HOST=this-is-an-rpc-url.comThen set the gRPC port for the CONSENSUS_HOST host:
export GRPC_PORT=9090If you make this change, please also update the docker run commands further down in the file to use these new variable names (e.g., ... --core.ip $CONSENSUS_HOST --core.port $GRPC_PORT ...).
app/operate/networks/mocha-testnet/page.mdx (112-118)
The endpoint rpc-mocha.pops.one has been removed from this list. However, it is still used as an example endpoint in other documentation pages, such as app/operate/data-availability/light-node/quickstart/page.mdx and app/operate/data-availability/bridge-node/page.mdx.
If rpc-mocha.pops.one is no longer a recommended consensus endpoint, the examples in other files should be updated to use one of the endpoints from the new 'Community consensus endpoints' table. If it is still valid and provides both RPC and gRPC from the same host, it should be added to that table for consistency.
|
🚀 Preview Deployment Your preview is ready: https://celestiaorg.github.io/docs-preview/pr-2440/ |
There was a problem hiding this comment.
Pull request overview
Updates the Docker setup documentation so Mocha and Arabica network links land on sections that explicitly list consensus endpoints where both RPC (26657) and gRPC (9090) are available from the same host (addressing #1625).
Changes:
- Updated Docker quickstart wording and network links to point to “Community consensus endpoints” anchors for Mocha and Arabica.
- Added a “Community consensus endpoints” table to the Mocha network page with
--core.iphosts and RPC/gRPC ports. - Renamed the Arabica “Community RPC endpoints” section heading to “Community consensus endpoints” to match the new Docker link target.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| app/operate/getting-started/docker/page.mdx | Updates wording/links to direct users to consensus endpoints that support both RPC and gRPC; clarifies port intent. |
| app/operate/networks/mocha-testnet/page.mdx | Introduces a dedicated consensus-endpoints table (with both ports) and points bridge-node guidance to it. |
| app/operate/networks/arabica-devnet/page.mdx | Renames the community section heading (and a sentence) so Docker links resolve directly to the intended endpoint list. |
Comments suppressed due to low confidence (1)
app/operate/getting-started/docker/page.mdx:99
- This step says to set the gRPC port, but the environment variable is named
RPC_PORT. To keep terminology consistent (and reduce chances of setting 26657 by mistake), consider renaming it to something likeGRPC_PORT/CORE_PORT, or explicitly note here that--core.portshould be the consensus gRPC port (usually 9090).
Then set the gRPC port for the `RPC_URL` host:
```bash
export RPC_PORT=9090
</details>
---
💡 <a href="/celestiaorg/docs/new/main?filename=.github/instructions/*.instructions.md" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Add Copilot custom instructions</a> for smarter, more guided reviews. <a href="https://docs.github.com/en/copilot/customizing-copilot/adding-repository-custom-instructions-for-github-copilot" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Learn how to get started</a>.
celestia-node connects via --core.ip/--core.port over gRPC only, so link Docker to Mainnet and Mocha community consensus endpoint sections and list currently reachable Mocha --core.ip hosts. Co-authored-by: Cursor <cursoragent@cursor.com>
019c9ab to
19b43bb
Compare
jcstein
changed the title
Match --core.ip/--core.port naming so the consensus endpoint step is not confused with Tendermint RPC URLs. Co-authored-by: Cursor <cursoragent@cursor.com>
Overview
Fixes #1625.
Updates Docker setup so Mainnet and Mocha links land on community consensus endpoint sections that list hosts usable with
--core.ip/--core.port.celestia-node connects to consensus over gRPC only (
--core.port, default9090) after celestia-node#3915 removed the Tendermint RPC dependency. Arabica is left alone (being removed separately).Changes
#community-consensus-endpoints; clarify gRPC port usage--core.iptable (Celestia Labs, P-OPS, ITRocket); remove deadconsensus-full-…hostValidation
nodebuilder/core/flags.go/ constructors:--core.portis gRPC only (default9090)full.consensus.mocha-4…has 26657+9090;rpc-mocha.pops.oneandcelestia-testnet-consensus.itrocket.nethave gRPC on 9090/operate/networks/mainnet-beta#community-consensus-endpoints,/operate/networks/mocha-testnet#community-consensus-endpoints