<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Feb 11, 2014 at 12:46 PM, Aaron Greengrass <span dir="ltr"><<a href="mailto:aaron@greenbtn.com" target="_blank">aaron@greenbtn.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> <div>
<div>2.  Remove hardcoding -- openstack_project is only usable for ... o.o.  Any hardcoded items should be variables, with a hiera lookup and a default as their default value.<br></div>
<div>     ie   $puppetmaster = hiera('puppetmaster', '<a href="http://ci-puppetmaster.openstack.org" target="_blank">ci-puppetmaster.openstack.org</a>') -- then you can set puppetmaster: 'puppethostname' in hiera, in the node that calls the puppetmaster manifest, or default to o.o</div>
</div></blockquote><div><br>+1<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>
<div> </div>
<div>3.  Redesign hiera.yaml to include a second data directory for 'public' data.  Start taking advantage of hiera's ability to load different data based on hierachy.</div>
<div> </div>
<div>See <a href="https://review.openstack.org/#/c/71739/" target="_blank">https://review.openstack.org/#/c/71739/</a> for the open review that addresses this.</div>
<div> </div></div></blockquote><div><br>+1<br><br></div><div>These would do wonders to help make the puppet architecture more portable, and maintainable. :)<br></div><div><br>-- <br></div><div>Jesusaurus<br></div></div><br>
</div></div>