Skip to content
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

lnst-ctl fails with "Unknown option: machine_pool_dirs in section environment" #168

Open
jbenc opened this issue Apr 7, 2016 · 1 comment

Comments

@jbenc
Copy link

jbenc commented Apr 7, 2016

After an update, any invocation of lnst-ctl (including with --help) fails with:

$ lnst-ctl -h
Loading config file '/etc/lnst-ctl.conf'
Loading config file '/home/jbenc/.lnst/lnst-ctl.conf'
Traceback (most recent call last):
File "/bin/lnst-ctl", line 375, in
sys.exit(main())
File "/bin/lnst-ctl", line 226, in main
lnst_config.load_config(usr_cfg)
File "/usr/lib/python2.7/site-packages/lnst/Common/Config.py", line 219, in load_config
self.handleSections(sections, abs_path)
File "/usr/lib/python2.7/site-packages/lnst/Common/Config.py", line 227, in handleSections
self.handleOptions(section, sections[section], path)
File "/usr/lib/python2.7/site-packages/lnst/Common/Config.py", line 253, in handleOptions
raise ConfigError(msg)
lnst.Common.Config.ConfigError: Unknown option: machine_pool_dirs in section environment

I understand there was a change that allows greater flexibility but lnst-ctl should handle the current configs gracefully and not fail. I consider this a regression.

At least warn for several releases that the config option is deprecated and provide instructions to update the config. Updating should not break the program completely. Currently, the only option to find out what's going on and how to fix it is grepping the source code.

@roidayan
Copy link
Contributor

roidayan commented Aug 31, 2016

@jbenc if you still need the syntax:

[pools]
pool_name = /path

example:

[pools]
lnst = ./pool

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants