Add missing frontend.env for Blockscout local testnet#22
Closed
Adeyemir wants to merge 1 commit intocirclefin:mainfrom
Closed
Add missing frontend.env for Blockscout local testnet#22Adeyemir wants to merge 1 commit intocirclefin:mainfrom
Adeyemir wants to merge 1 commit intocirclefin:mainfrom
Conversation
deployments/blockscout.yaml declares an env_file dependency on deployments/monitoring/config-blockscout/frontend/frontend.env, but the file is not present in the repo. Without it, `make testnet` fails to bring up the blockscout-frontend service, and the explorer UI never becomes reachable on http://localhost. This adds a minimal working frontend.env matching the committed backend.env in the same directory. Values align with the localdev chain (chain id 1337, currency ETH) and the nginx proxy exposed on port 80 by the blockscout compose file. Verified end to end against a local `make testnet` run: frontend container stays up, proxy returns 200, explorer loads and shows live blocks.
Contributor
|
Thanks for contribution, we identified this issue and fixed in this PR #19 |
Author
|
Thanks @ZhiyuCircle, I missed #19 before opening this. Closing in favor of yours. Glad the fix landed. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
deployments/blockscout.yamldeclares anenv_filedependency ondeployments/monitoring/config-blockscout/frontend/frontend.env, butthat file is not present in the repo. As a result, a fresh clone
running
make testnetbrings up every service exceptblockscout-frontend, which crash loops on boot with:The nginx proxy then returns 502 on http://localhost, so the explorer
is unreachable, even though Reth and Malachite are producing blocks
normally.
The sibling
backend.envin the same directory is committed, thereis no
.example/.sample/.templateforfrontend.envin thetree, no documentation referencing it, and no
.gitignoreruleexcluding it. Based on that, I assumed the file was meant to ship
with the repo and put this PR together. If it was intentionally left
out and users are supposed to author their own, happy to close this
and send a docs patch instead.
Change
Adds a minimal working
frontend.envwith values aligned to thelocaldev chain and the compose topology:
NEXT_PUBLIC_NETWORK_ID=1337(matchesbackend.envand genesis)NEXT_PUBLIC_NETWORK_CURRENCY_*set to ETH / 18NEXT_PUBLIC_APP_*andNEXT_PUBLIC_API_*pointed atlocalhost:80,which is where the blockscout proxy is exposed by
deployments/blockscout.yamlnone, identicon typejazzicon, homepagecharts limited to
daily_txsHappy to rename any of these values if the team prefers a different
network name or branding for localdev.
Test plan
make testnetfrom a cold cachedocker psshowsfrontendinUpstate (not restarting)curl -sI http://localhostreturnsHTTP/1.1 200 OK"Arc Localdev explorer"