<div dir="ltr">Hi Clayton!<div><br></div><div>I would like to use the same solution for this problem as we have for <module>::logging class. So, we just leaving those default parameters without $::os_service_default and putting old values (10, 20, etc) [1] and also we should to raise warnings, as suggests Alex Schultz (for example, [2]). For me - it is the best solution of problem.</div><div><br></div><div>[1] <a href="https://github.com/openstack/puppet-cinder/blob/master/manifests/logging.pp#L98">https://github.com/openstack/puppet-cinder/blob/master/manifests/logging.pp#L98</a></div><div>[2] <a href="https://review.openstack.org/#/c/239800/5/manifests/backend/eqlx.pp">https://review.openstack.org/#/c/239800/5/manifests/backend/eqlx.pp</a></div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-11-11 18:04 GMT+03:00 Clayton O'Neill <span dir="ltr"><<a href="mailto:clayton@oneill.net" target="_blank">clayton@oneill.net</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span class="">On Wed, Nov 11, 2015 at 9:50 AM, Clayton O'Neill <span dir="ltr"><<a href="mailto:clayton@oneill.net" target="_blank">clayton@oneill.net</a>></span> wrote:<br></span><div class="gmail_extra"><div class="gmail_quote"><span class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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" target="_blank">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></blockquote><div><br></div></span><div>To clarify: The modules following this pattern are setting max_pool_size and max_overflow to 10 and 20 respectively, but oslo_db has no internal default.</div></div></div></div>
<br>__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div style="color:rgb(136,136,136)"><span style="font-family:arial;font-size:small">Best Regards,</span><br></div><span style="color:rgb(136,136,136)">Egorenko Denis</span>,</div><div><span style="color:rgb(136,136,136)">Deployment Engineer</span><br style="color:rgb(136,136,136)"><span style="color:rgb(136,136,136)">Mirantis</span><br></div></div></div></div>
</div>