Skip to content
This repository has been archived by the owner on Feb 29, 2024. It is now read-only.

Commit

Permalink
Expose max_concurrent_builds as a Heat parameter
Browse files Browse the repository at this point in the history
It can now be set via the use of 'NovaMaxConcurrentBuilds'.

Change-Id: If75b1df0e26a7a3f87be1de9026836a2aa590fe7
Co-authored-By: David J Peacock <[email protected]>
Signed-off-by: Luke Short <[email protected]>
  • Loading branch information
LukeShortCloud and davidjpeacock committed Aug 31, 2020
1 parent 95aa30f commit efe1791
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 0 deletions.
9 changes: 9 additions & 0 deletions deployment/nova/nova-ironic-container-puppet.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,14 @@ parameters:
description: The number of times to retry when a request conflicts. If set to 0, only try once, no retries.
type: number
default: -1
IronicMaxConcurrentBuilds:
description: |
The number of builds to process at the same time per each nova-compute instance. If set to 0, it will
be set to unlimited (best effort).
type: number
default: 10
constraints:
- range: { min: 0 }

conditions:

Expand Down Expand Up @@ -109,6 +117,7 @@ outputs:
- nova::compute::force_config_drive: true
nova::compute::reserved_host_memory: '0'
nova::compute::vnc_enabled: false
nova::compute::ironic::max_concurrent_builds: {get_param: IronicMaxConcurrentBuilds}
nova::ironic::common::password: {get_param: IronicPassword}
nova::ironic::common::project_name: 'service'
nova::ironic::common::auth_url: {get_param: [EndpointMap, KeystoneInternal, uri_no_suffix]}
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
---
features:
- |
The nova-ironic setting for 'max_concurrent_builds' can now be set via the
use of a new TripleO Heat templates parameter 'IronicMaxConcurrentBuilds'.
It is set to the service default of 10 by default in TripleO Heat templates.

0 comments on commit efe1791

Please sign in to comment.