Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1631012

Summary: Expose ironic::executor_thread_pool_size in puppet for undercloud tuning
Product: Red Hat OpenStack Reporter: Andreas Karis <akaris>
Component: puppet-ironicAssignee: Riccardo Pittau <rpittau>
Status: CLOSED UPSTREAM QA Contact: mlammon
Severity: low Docs Contact:
Priority: low    
Version: 13.0 (Queens)CC: achernet, bfournie, dvd, jjoyce, jschluet, jslagle, mburns, racedoro, rpittau, slinaber, tvignaud
Target Milestone: Upstream M1Keywords: Triaged
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1631009 Environment:
Last Closed: 2019-05-16 09:57:44 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Andreas Karis 2018-09-19 17:17:10 UTC
Description of problem:
Requesting the following feature enhancement: implement ironic::rpc_thread_pool_size for undercloud tuning

The undercloud allows us to push custom hierdata to make configuration
persistent:
~~~
undercloud.conf:
hieradata_override = /home/stack/custom_hiera.yaml
~~~

However, for https://access.redhat.com/articles/3597351 , I need to modify a custom ironic parameter for which no
puppet manifest exists yet (at least by grepping, I cannot find it):
~~~
ironic::config::ironic_config:
  DEFAULT/rpc_thread_pool_size:
    value: 8
~~~


ironic::config is not included by instack-undercloud and I created BZ1631009 to include this. However, perhaps we can add the same parameter to the ironic class so that e.g. ironic::rpc_thread_pool_size: <x>  works?

Comment 1 Andreas Karis 2018-09-19 17:21:44 UTC
alternatively executor_thread_pool_size as I think that rpc_thread_pool_size is deprecated and superseded by executor_thread_pool_size

Comment 2 Dmitry Tantsur 2019-01-02 12:19:51 UTC
Low hanging fruit, but will need to be exposed in puppet-oslo first (https://github.com/openstack/puppet-oslo/blob/master/manifests/messaging/default.pp), since this option belongs to oslo.messaging, not ironic.