Skip to content

docs: add celestia-app Docker setup guide#2438

Open
jcstein wants to merge 1 commit intomainfrom
codex/issue-1100-celestia-app-docker
Open

docs: add celestia-app Docker setup guide#2438
jcstein wants to merge 1 commit intomainfrom
codex/issue-1100-celestia-app-docker

Conversation

@jcstein
Copy link
Member

@jcstein jcstein commented Mar 6, 2026

Summary

  • add a dedicated celestia-app Docker guide under consensus validators
  • document quick start with persistent storage, genesis download, and peer config
  • add a Docker network example for running celestia-node against containerized celestia-app
  • add cross-links from existing Docker and install pages

Closes #1100

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds a new documentation page for running celestia-app with Docker. The new guide is comprehensive and covers prerequisites, a quick start guide with persistent storage, and instructions for running celestia-node against the containerized celestia-app. The changes also include adding cross-links from other relevant pages. My review focuses on the new documentation file. I've found a few minor issues in the shell commands for configuring persistent peers. The commands can be improved for correctness and consistency by properly handling trailing commas in the peer lists. I've provided suggestions to fix these.

<Tabs items={['Mainnet Beta', 'Mocha', 'Arabica']}>
<Tabs.Tab>
```bash
PERSISTENT_PEERS=$(curl -sL https://raw.githubusercontent.com/cosmos/chain-registry/master/{{constants['mainnetChainId']}}/chain.json | jq -r '.peers.persistent_peers[].address' | tr '\n' ',' | sed 's/,$/\n/')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The sed 's/,$/\n/' command at the end of this pipeline replaces the trailing comma with a newline character. This results in the PERSISTENT_PEERS variable containing a trailing newline, which will be written into the config.toml file. This is likely not the intended behavior. To simply remove the trailing comma, sed 's/,$//' should be used instead.

    PERSISTENT_PEERS=$(curl -sL https://raw.githubusercontent.com/cosmos/chain-registry/master/{{constants['mainnetChainId']}}/chain.json | jq -r '.peers.persistent_peers[].address' | tr '\n' ',' | sed 's/,$//')

</Tabs.Tab>
<Tabs.Tab>
```bash
PERSISTENT_PEERS=$(curl -sL https://raw.githubusercontent.com/celestiaorg/networks/master/{{constants['mochaChainId']}}/peers.txt | tr '\n' ',')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

This command will produce a comma-separated list of peers with a trailing comma. While this may be valid, it's cleaner to remove the trailing comma for correctness and consistency. You can pipe the output to sed 's/,$//' to remove it.

    PERSISTENT_PEERS=$(curl -sL https://raw.githubusercontent.com/celestiaorg/networks/master/{{constants['mochaChainId']}}/peers.txt | tr '\n' ',' | sed 's/,$//')

</Tabs.Tab>
<Tabs.Tab>
```bash
PERSISTENT_PEERS=$(curl -sL https://raw.githubusercontent.com/celestiaorg/networks/master/{{constants['arabicaChainId']}}/peers.txt | tr '\n' ',')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

This command will produce a comma-separated list of peers with a trailing comma. While this may be valid, it's cleaner to remove the trailing comma for correctness and consistency. You can pipe the output to sed 's/,$//' to remove it.

    PERSISTENT_PEERS=$(curl -sL https://raw.githubusercontent.com/celestiaorg/networks/master/{{constants['arabicaChainId']}}/peers.txt | tr '\n' ',' | sed 's/,$//')

@github-actions
Copy link
Contributor

github-actions bot commented Mar 6, 2026

🚀 Preview Deployment

Your preview is ready: https://celestiaorg.github.io/docs-preview/pr-2438/

Copilot AI added a commit that referenced this pull request Mar 6, 2026
Co-authored-by: jcstein <46639943+jcstein@users.noreply.github.com>
@jcstein jcstein self-assigned this Mar 6, 2026
@celestiaorg celestiaorg deleted a comment from gemini-code-assist bot Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: add documentation for running celestia-app docker images

1 participant