openstack/puppet-mistral
Code Issues Proposed changes

Merge "Add strict validation about boolean parameters"

Browse Source
This commit is contained in:
Zuul
2023-03-21 15:07:11 +00:00
committed by Gerrit Code Review
parent 6aeb6d3a3b 86bf3f0ec5
commit ac360063ff
5 changed files with 14 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
manifests/api.pp
View File

@@ -76,6 +76,9 @@ class mistral::api (
include mistral::params include mistral::params
include mistral::policy include mistral::policy
validate_legacy(Boolean, 'validate_bool', $manage_service)
validate_legacy(Boolean, 'validate_bool', $enabled)
if $auth_strategy == 'keystone' { if $auth_strategy == 'keystone' {
include mistral::keystone::authtoken include mistral::keystone::authtoken
} }

3
manifests/engine.pp
View File

@@ -69,6 +69,9 @@ class mistral::engine (
include mistral::deps include mistral::deps
include mistral::params include mistral::params
validate_legacy(Boolean, 'validate_bool', $manage_service)
validate_legacy(Boolean, 'validate_bool', $enabled)
package { 'mistral-engine': package { 'mistral-engine':
ensure => $package_ensure, ensure => $package_ensure,
name => $::mistral::params::engine_package_name, name => $::mistral::params::engine_package_name,

3
manifests/event_engine.pp
View File

@@ -40,6 +40,9 @@ class mistral::event_engine (
include mistral::deps include mistral::deps
include mistral::params include mistral::params
validate_legacy(Boolean, 'validate_bool', $manage_service)
validate_legacy(Boolean, 'validate_bool', $enabled)
package { 'mistral-event-engine': package { 'mistral-event-engine':
ensure => $package_ensure, ensure => $package_ensure,
name => $::mistral::params::event_engine_package_name, name => $::mistral::params::event_engine_package_name,

3
manifests/executor.pp
View File

@@ -40,6 +40,9 @@ class mistral::executor (
include mistral::deps include mistral::deps
include mistral::params include mistral::params
validate_legacy(Boolean, 'validate_bool', $manage_service)
validate_legacy(Boolean, 'validate_bool', $enabled)
package { 'mistral-executor': package { 'mistral-executor':
ensure => $package_ensure, ensure => $package_ensure,
name => $::mistral::params::executor_package_name, name => $::mistral::params::executor_package_name,

2
manifests/init.pp
View File

@@ -186,6 +186,8 @@ class mistral(
include mistral::params include mistral::params
include mistral::db include mistral::db
validate_legacy(Boolean, 'validate_bool', $sync_db)
package { 'mistral-common': package { 'mistral-common':
ensure => $package_ensure, ensure => $package_ensure,
name => $::mistral::params::common_package_name, name => $::mistral::params::common_package_name,