-
Notifications
You must be signed in to change notification settings - Fork 20.4k
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
cmd: Fix bug preventing --lightserv NN w/ --syncmode=full #17803
Conversation
FWIW the Appveyor build failures seem to be unrelated failures w/ swarm:
Also FWIW I don't run into these failures when running the tests locally:
|
@karalabe would be nice to get this in 1.8.17, as it lets Archive nodes act as LES servers, whereas currently they can't due to this parsing bug. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Huh, weird corner case and good catch. Took me a long while to understand what's wrong and why your patch fixes it. LGTM!
Currently in
master
this combination of flags fails:However, the intent was to prevent
--lightserv NN
and--syncmode=light
.There was a bug in the validation logic, this PR fixes said logic so that the above works, but the intended conflict still fails:
Note this is effectively a refresh of #16862 which was sitting in PR limbo for awhile.