gladiusio / gladius-node

All services to run a Gladius Node
GNU General Public License v3.0
36 stars 11 forks source link

gladius-controld is incompatible with an empty gladius-controld.toml configuration file #76

Closed sebastianhutter closed 5 years ago

sebastianhutter commented 6 years ago

Whats happening: When starting gladius-controld with an empty gladius-controld.toml file in the gladius root dir the controld service exits with an error message due to missing configuration options:

$ gladius-controld
2018/09/02 13:33:53 [DEBUG] Using dynamic bind port 35907
2018/09/02 13:33:53 Config.Name field is required.

This configuration was created by the pre 0.6.x installer scripts. If I update from pre 0.6.x I have to manually remove the file.

What do you expect: gladius-controld finds the empty config file - sees that no configuration is given and loads its defined defaults. This behaviour is already implemented in gladius-networkd.

Further information: none

overtfuture commented 5 years ago

We removed the need for toml files unless wanting to override a specific config value in the latest version (0.7.0).