-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Setting Goerli flag in YAML does not set network to Prater #11122
Comments
Now that I think about it, my previous comment doesn't make sense. Alias appears to still be checked when goerli used on command line. So, not clear to me why this doesn't work from YAML. |
Can you share the yaml file? and the cli command line to launch? |
Unfortunately, I can no longer match up what my configuration was when I submitted this with the log entry I have that says I was on the wrong network. Current logs seem to be about something different than my original report, but something doesn't seem right. I won't speculate further. Systemd Service File:
prysm-beacon.yaml
Logs
After I remove
Config
Logs:
...and functioning resumes as normal. No problems syncing up. Edit: Fixed some typos after I copied and pasted ExecStart. |
You can see in the first log where is says it is connecting to mainnet. This is what made me initially think it wasn't respecting the Goerli flag in YAML. But from command line it says Prater. |
we do versioning by the fork-version afaik, @kasey can confirm on this. |
Sorry @nisdas, is your comment something actionable for me, or is this a note to devs? |
Hey @metanull-operator , this is mostly a note to kasey |
urfave/cli#1394 looks like this is the underlying issue to this. |
I believe this will be fixed with v3 cli, waiting for release, I fixed the bug in the library. |
prysm/config/features/config.go
Line 114 in 63a8690
Running v2.1.4-rc.0 with Goerli flag set in YAML config file does not set chain for Prater network. Using Goerli flag on command line does work. An alias for Prater was created with issue #11605, but as best as I can tell the cli library doesn't check aliases when using ctx.Bool on the line referenced in the link above.
The text was updated successfully, but these errors were encountered: