Bug 2180684 - Expose executor_thread_pool_size in puppet-heat
Summary: Expose executor_thread_pool_size in puppet-heat
Keywords:
Status: ON_QA
Alias: None
Product: Red Hat OpenStack
Classification: Red Hat
Component: puppet-heat
Version: 16.2 (Train)
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: z6
: 16.2 (Train on RHEL 8.4)
Assignee: Takashi Kajinami
QA Contact: Nobody
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-03-22 05:05 UTC by Ravi Singh
Modified: 2023-07-27 16:10 UTC (History)
6 users (show)

Fixed In Version: puppet-heat-15.5.1-2.20230401024915.1cce62f.el8ost
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
OpenStack gerrit 879122 0 None MERGED Expose executor_thread_pool_size 2023-04-03 02:27:06 UTC
Red Hat Issue Tracker OSP-23569 0 None None None 2023-03-22 05:08:16 UTC

Description Ravi Singh 2023-03-22 05:05:27 UTC
Description of problem:

Configuration of executor_thread_pool_size is failing with duplicate declaration since this param is moved to oslo messaging but we do not see it in puppet-heat.

+++
ControllerExtraConfig:
    heat::config::heat_config:
      DEFAULT/executor_thread_pool_size:
        value:  24

With the same config, this fails in RHOSP16.2 with the following error on the controllers:

        Feb 28 15:10:22 ctrl0 puppet-user[501416]: Error: Evaluation Error: Error while evaluating a Resource Statement, Evaluation Error: Error while evaluating a Function Call, Duplicate declaration: Heat_config[DEFAULT/executor_thread_pool_size] is already declared at (file: /etc/puppet/modules/heat/manifests/config.pp, line: 36); cannot redeclare (file: /etc/puppet/modules/oslo/manifests/messaging/default.pp, line: 47) (file: /etc/puppet/modules/oslo/manifests/messaging/default.pp, line: 47, column: 3) (file: /etc/puppet/modules/heat/manifests/init.pp, line: 445) on node 
+++
Param in oslo
+++
define oslo::messaging::default(
  $executor_thread_pool_size = $::os_service_default,
  $rpc_response_timeout      = $::os_service_default,
  $transport_url             = $::os_service_default,
  $control_exchange          = $::os_service_default,
) {

  $default_options = {
    'DEFAULT/executor_thread_pool_size' => { value => $executor_thread_pool_size }, <<<<<<<<<<<
    'DEFAULT/rpc_response_timeout'      => { value => $rpc_response_timeout },
    'DEFAULT/transport_url'             => { value => $transport_url, secret => true },
    'DEFAULT/control_exchange'          => { value => $control_exchange },
+++

Recently upstream patch was filed & we are looking if it's possible to backport it to OSP16.
https://review.opendev.org/c/openstack/puppet-heat/+/876489

Version-Release number of selected component (if applicable):
16.2

How reproducible:
Everytime

Steps to Reproduce:
1.
2.
3.

Actual results:
Not able to config param

Expected results:
Should be able to config.

Additional info:


Note You need to log in before you can comment on or make changes to this bug.