openstack/puppet-watcher
Code Issues Proposed changes

Merge "Add strict validation about boolean parameters"

Browse Source
This commit is contained in:
Zuul 2023-07-24 15:14:21 +00:00 committed by Gerrit Code Review
commit 6ddecf55e6
2 changed files with 14 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
manifests/api.pp
View File

@ -72,18 +72,18 @@
# Defaults to 'keystone' # Defaults to 'keystone'
# #
class watcher::api ( class watcher::api (
$package_ensure = 'present', $package_ensure = 'present',
$enabled = true, Boolean $enabled = true,
$manage_service = true, Boolean $manage_service = true,
$port = $facts['os_service_default'], $port = $facts['os_service_default'],
$max_limit = $facts['os_service_default'], $max_limit = $facts['os_service_default'],
$bind_host = $facts['os_service_default'], $bind_host = $facts['os_service_default'],
$workers = $facts['os_workers'], $workers = $facts['os_workers'],
$enable_ssl_api = $facts['os_service_default'], $enable_ssl_api = $facts['os_service_default'],
$service_name = $::watcher::params::api_service_name, $service_name = $::watcher::params::api_service_name,
$create_db_schema = false, Boolean $create_db_schema = false,
$upgrade_db = false, Boolean $upgrade_db = false,
$auth_strategy = 'keystone', $auth_strategy = 'keystone',
) inherits watcher::params { ) inherits watcher::params {
include watcher::policy include watcher::policy

4
manifests/applier.pp
View File

@ -38,8 +38,8 @@
# #
class watcher::applier ( class watcher::applier (
$package_ensure = 'present', $package_ensure = 'present',
$enabled = true, Boolean $enabled = true,
$manage_service = true, Boolean $manage_service = true,
$applier_workers = $facts['os_service_default'], $applier_workers = $facts['os_service_default'],
$applier_conductor_topic = $facts['os_service_default'], $applier_conductor_topic = $facts['os_service_default'],
$applier_status_topic = $facts['os_service_default'], $applier_status_topic = $facts['os_service_default'],