<div dir="ltr">Regarding config opts for keystone, the keystoneclient middleware already registers the opts at <a href="https://github.com/openstack/python-keystoneclient/blob/master/keystoneclient/middleware/auth_token.py#L325">https://github.com/openstack/python-keystoneclient/blob/master/keystoneclient/middleware/auth_token.py#L325</a> under a keystone_authtoken group in the config file.  Currently, Mistral registers the opts again at <a href="https://github.com/stackforge/mistral/blob/master/mistral/config.py#L108">https://github.com/stackforge/mistral/blob/master/mistral/config.py#L108</a> under a different configuration group.  Should we remove the duplicate from Mistral and refactor the reference to keystone configurations to the keystone_authtoken group?  This seems more consistent.</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 15, 2014 at 1:13 PM, W Chan <span dir="ltr"><<a href="mailto:m4d.coder@gmail.com" target="_blank">m4d.coder@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">Currently, the various configurations are registered in ./mistral/config.py.  The configurations are registered when mistral.config is referenced.  Given the way the code is written, PEP8 throws referenced but not used error if mistral.config is referenced but not called in the module.  In various use cases, this is avoided by using importutils to import mistral.config (i.e. <a href="https://github.com/stackforge/mistral/blob/master/mistral/tests/unit/engine/test_transport.py#L34" target="_blank">https://github.com/stackforge/mistral/blob/master/mistral/tests/unit/engine/test_transport.py#L34</a>).  I want to break down registration code in ./mistral/config.py into separate functions for api, engine, db, etc and move the registration closer to the module where the configuration is needed.  Any objections?</div>

</blockquote></div><br></div>