Skip to content

fix(env): stop shipping an active Sepolia healthcheck RPC in .env.example #96

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.

Sign up for GitHub

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

Open
feafel wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix(env): stop shipping an active Sepolia healthcheck RPC in .env.example #96

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 6 additions & 3 deletions .env.example
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,9 +35,12 @@ OP_NODE__L1_BEACON=
OP_NODE__RPC_TYPE=basic

# L2 RPC used by the healthcheck sidecar for comparison.
# The per-network env files provide defaults, and this value overrides them.
# For ink-mainnet, switch this to https://rpc-gel.inkonchain.com
HEALTHCHECK__REFERENCE_RPC_PROVIDER=https://rpc-gel-sepolia.inkonchain.com
# Leave this commented out. envs/<network>/healthcheck.env already provides the
# right reference RPC for NETWORK_NAME, and docker-compose.yml loads this file
# last, so a value here silently overrides the per-network one. That makes a
# mainnet node healthcheck against a Sepolia RPC if this is copied unchanged.
# Uncomment only to force a custom reference RPC.
# HEALTHCHECK__REFERENCE_RPC_PROVIDER=https://rpc-gel-sepolia.inkonchain.com

###############################################################################
# ↓ OPTIONAL ↓ #
Expand Down
8 changes: 5 additions & 3 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,6 @@ NODE_TYPE=full
OP_NODE__RPC_ENDPOINT=<your Sepolia execution RPC>
OP_NODE__L1_BEACON=<your Sepolia beacon API>
OP_NODE__RPC_TYPE=basic
HEALTHCHECK__REFERENCE_RPC_PROVIDER=https://rpc-gel-sepolia.inkonchain.com
```

Configuration notes:
Expand All @@ -137,8 +136,11 @@ Configuration notes:
most first-time setups only need the `.env` values above
- `PORT__OP_NODE_P2P` changes the published host port in `docker-compose.yml`.
The in-container `op-node` listener still uses `9003`
- For `ink-mainnet`, switch the healthcheck reference RPC to
`https://rpc-gel.inkonchain.com`
- The healthcheck reference RPC is selected per network by
`envs/<network>/healthcheck.env`. `docker-compose.yml` loads `.env` after
those files, so set `HEALTHCHECK__REFERENCE_RPC_PROVIDER` in `.env` only to
override it, and never leave a hardcoded Sepolia value there when
`NETWORK_NAME=ink-mainnet`
- Advanced wrapper inputs such as `OVERRIDE_HOLOCENE` and `EXTENDED_ARG` live
in `.env.example`. The shell entrypoints append them to both `op-geth` and
`op-node`, so leave them empty unless you know the flag is compatible with
Expand Down