<div dir="ltr">I found this bug in the liberty branch [1] over the weekend in the handling of openstack client between glance & keystone. As a part of fixing that I've discussed with Clayton and Michael Chapman just what the right way is to include the openstackclient.<div><br></div><div>Keystone does it by conditionally including the class in client.pp [2]. Glance does it with an ensure_resources call in the main class [3].</div><div><br></div><div>Michael Chapman was of the opinion we should just include the openstacklib::openstackclient unconditionally and let hiera figure it out (hope I'm paraphrasing his opinion). That is cleaner but perhaps less flexible.</div><div><br></div><div>Whatever solution we pick, I want to be consistent and back-portable.</div><div><br></div><div>Thoughts?</div><div><br></div><div><br></div><div>[1] - <a href="https://bugs.launchpad.net/puppet-openstacklib/+bug/1523643">https://bugs.launchpad.net/puppet-openstacklib/+bug/1523643</a> </div><div>[2] - <a href="https://github.com/openstack/puppet-keystone/blob/master/manifests/client.pp#L20-L26">https://github.com/openstack/puppet-keystone/blob/master/manifests/client.pp#L20-L26</a></div><div>[3] - <a href="https://github.com/openstack/puppet-glance/blob/master/manifests/init.pp#L33">https://github.com/openstack/puppet-glance/blob/master/manifests/init.pp#L33</a><br><div><br></div></div></div>