-
Notifications
You must be signed in to change notification settings - Fork 2.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
ERROR: unknown flag: --consensus.create_empty_blocks_interval #4065
Comments
Also, same issue with
|
Tendermint does not expose all config parameters via CLI. |
And I don't think it should as it leads to code duplication UNLESS there's a way to avoid it. You can use env variables for simple variables: |
Thank you @melekes So, how exactly does one programatically control the tendermint node instance, say when launching through tendermint-node ? As of now, a tendermint node instance is useless without a proxy app instance. So, launching the node instance as part of the abci_host app is the perfect (only?) way to manage its life programatically. Given that Tendemint Node is supposed to be 'programmable' one would expect the behavior to be controlled at runtime (and not just through config files).
How does one know which ones are exposed and which ones are not? It sounds pretty wierd to send some commands through CLI param, and some through env, when all of them reside in the same config. I am kind of stuck where I could launch the tendermint-node instance from Node.JS programatically, but no way to control its behavior programatically. Unable to move ahead because of this random decision not to include some params on the CLI. Wondering if you are targeting only GO programmers, and moving away from JS/Node.js If so, please let us know so that we would just stay away and not bother you. It is really frustrating to encounter something labelled as built for programmers, and then not being able to do anything with it programmatically. |
The You are right it can be a bit frustrating that some things can be handled directly on the CLI, while the rest require editing the config file. Most folks have just dealt with this, and figured out a way to programmatically manage the config file. It would probably be a bit over whelming to surface the entire config file into the CLI flags. That said, since we already have a CLI flag for Thanks! |
Thank you @ebuchman . Adding To give a bigger picture:
I meant that "programmatically". That is, in an ABCI host built in some other language, which receives the tendermint config file, it would be very difficult to know programmatically which options in that config file has to go into ENV, and which has to go into the command line. The The goal I am trying to achieve is: start, control and stop the tendermint node programmatically (without any human intervention). Since tendermint is consensus-engine for programmers, I am hoping that it would be made possible. |
Yeah, that's a good goal I think. Maybe there's a way to do it using viper. We'll need to research. |
Ok I've marked it as "good first issue". Someone with minimal Go experience should be able to tackle this. Meanwhile, we fully agree with your point re programmability. Would you mind opening a new issue describing this and how you might like to see it be realized? Thanks! |
Ouch!! Now I have seen this new ticket created by @KrishnaPG ! You can close the PR if not needed. |
Tendermint version: 0.32.6-c4ba93a1
ABCI app: local test app
Environment:
What happened:
What you expected to happen: should work without error
Have you tried the latest version: yes
How to reproduce it (as minimally and precisely as possible):
Use the below command line
Anything else we need to know:
Trying to run tendermint with above CLI command is failing with the above error. The
create_empty_blocks
setting on the same command has no problem. Not sure why it does not accept thecreate_empty_blocks_interval
paramWhile it is failing from the CLI command directly, the original requirement is to be able to launch the node with tendermint-node, which obviously is generating the above error. The config there is:
which the
tendermint-node
automatically converts to CLI commandline parameters, where it is failing to launch because of the above error. So, tried running the command directly from CLI then observed that it is failing.The text was updated successfully, but these errors were encountered: