<div dir="ltr">I discovered this issue last night and opened a bug on it (<a href="https://bugs.launchpad.net/puppet-tuskar/+bug/1515273">https://bugs.launchpad.net/puppet-tuskar/+bug/1515273</a>).<div><br></div><div>This effects most of the modules, and the short version of it is that the defaults in all the <modulename>::db classes are wrong for max_pool_size and max_overflow.  We’re setting test to 10 and 20, but oslo_db actually has no internal default.  </div><div><br></div><div>The two options I see for fixing this are to either put in place the old traditional:</div><div><br></div><div>if $option_name { </div><div>  # ensure value</div><div>} else {</div><div>  # ensure absent</div><div>}</div><div><br></div><div>and do that for at least the two values with the wrong defaults, or preferably, change the defaults for all of these parameters to use the service default fact.</div><div><br></div><div>I prefer the latter approach.  I know there has been some discussion on Trello about how much we want to be using the service default fact, but as near as I can tell, the concerns seem to mostly about not accidentally reverting intentionally different values and breaking existing installs.</div><div><br></div><div>This scenario seems to be the ideal candidate for just not setting a value unless the deployer has specifically asked for something special.</div></div>