URL: https://github.com/freeipa/freeipa/pull/182
Title: #182: Use env var IPA_CONFDIR to get confdir for 'cli' context

tomaskrizek commented:
"""
I also think `IPA_CONFDIR` environment variable is the proper way to configure 
the config directory with use cases such as Ansible.

However, with the current solution, if the `AttributeError` is raised, the 
command will fail and show a traceback. I'd really prefer to only see the error 
message itself. Perhaps this could be solved by using `ScriptError`?

```
# IPA_CONFDIR=/root/ipa ipa ping
[2016-11-07T14:38:11Z ipa] <ERROR>: AttributeError: IPA_CONFDIR must be an 
absolute path to an existing directory.
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/ipalib/cli.py", line 1345, in run
    (_options, argv) = api.bootstrap_with_global_options(context='cli')
  File "/usr/lib/python2.7/site-packages/ipalib/plugable.py", line 580, in 
bootstrap_with_global_options
    self.bootstrap(parser, **overrides)
  File "/usr/lib/python2.7/site-packages/ipalib/plugable.py", line 436, in 
bootstrap
    self.env._bootstrap(**overrides)
  File "/usr/lib/python2.7/site-packages/ipalib/config.py", line 469, in 
_bootstrap
    'IPA_CONFDIR must be an absolute path to an '
AttributeError: IPA_CONFDIR must be an absolute path to an existing directory.
[2016-11-07T14:38:11Z ipa] <ERROR>: an internal error has occurred
```
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/182#issuecomment-258855038
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to