openstack
/
puppet-tripleo
Code Issues Proposed changes

Merge "Designate: Replace hiera by lookup"

This commit is contained in:
Zuul 2022-05-26 01:13:20 +00:00 committed by Gerrit Code Review
parent 68ef8f73db 2acd2f4b0d
commit 040b0c9d59
10 changed files with 72 additions and 72 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

52
manifests/profile/base/designate.pp
View File

@ -20,55 +20,55 @@
# #
# [*step*] # [*step*]
# (Optional) The current step of the deployment # (Optional) The current step of the deployment
# Defaults to hiera('step') # Defaults to Integer(lookup('step'))
# #
# [*oslomsg_rpc_proto*] # [*oslomsg_rpc_proto*]
# Protocol driver for the oslo messaging rpc service # Protocol driver for the oslo messaging rpc service
# Defaults to hiera('oslo_messaging_rpc_scheme', rabbit) # Defaults to lookup('oslo_messaging_rpc_scheme', undef, undef, 'rabbit')
# #
# [*oslomsg_rpc_hosts*] # [*oslomsg_rpc_hosts*]
# list of the oslo messaging rpc host fqdns # list of the oslo messaging rpc host fqdns
# Defaults to hiera('oslo_messaging_rpc_node_names') # Defaults to any2array(lookup('oslo_messaging_rpc_node_names', undef, undef, undef))
# #
# [*oslomsg_rpc_port*] # [*oslomsg_rpc_port*]
# IP port for oslo messaging rpc service # IP port for oslo messaging rpc service
# Defaults to hiera('oslo_messaging_rpc_port', 5672) # Defaults to lookup('oslo_messaging_rpc_port', undef, undef, '5672')
# #
# [*oslomsg_rpc_username*] # [*oslomsg_rpc_username*]
# Username for oslo messaging rpc service # Username for oslo messaging rpc service
# Defaults to hiera('oslo_messaging_rpc_user_name', 'guest') # Defaults to lookup('oslo_messaging_rpc_user_name', undef, undef, 'guest')
# #
# [*oslomsg_rpc_password*] # [*oslomsg_rpc_password*]
# Password for oslo messaging rpc service # Password for oslo messaging rpc service
# Defaults to hiera('oslo_messaging_rpc_password') # Defaults to lookup('oslo_messaging_rpc_password')
# #
# [*oslomsg_rpc_use_ssl*] # [*oslomsg_rpc_use_ssl*]
# Enable ssl oslo messaging services # Enable ssl oslo messaging services
# Defaults to hiera('oslo_messaging_rpc_use_ssl', '0') # Defaults to lookup('oslo_messaging_rpc_use_ssl', undef, undef, '0')
# #
# [*oslomsg_notify_proto*] # [*oslomsg_notify_proto*]
# Protocol driver for the oslo messaging notify service # Protocol driver for the oslo messaging notify service
# Defaults to hiera('oslo_messaging_notify_scheme', rabbit) # Defaults to lookup('oslo_messaging_notify_scheme', undef, undef, 'rabbit')
# #
# [*oslomsg_notify_hosts*] # [*oslomsg_notify_hosts*]
# list of the oslo messaging notify host fqdns # list of the oslo messaging notify host fqdns
# Defaults to hiera('oslo_messaging_notify_node_names') # Defaults to any2array(lookup('oslo_messaging_notify_node_names', undef, undef, undef))
# #
# [*oslomsg_notify_port*] # [*oslomsg_notify_port*]
# IP port for oslo messaging notify service # IP port for oslo messaging notify service
# Defaults to hiera('oslo_messaging_notify_port', 5672) # Defaults to lookup('oslo_messaging_notify_port', undef, undef, '5672')
# #
# [*oslomsg_notify_username*] # [*oslomsg_notify_username*]
# Username for oslo messaging notify service # Username for oslo messaging notify service
# Defaults to hiera('oslo_messaging_notify_user_name', 'guest') # Defaults to lookup('oslo_messaging_notify_user_name', undef, undef, 'guest')
# #
# [*oslomsg_notify_password*] # [*oslomsg_notify_password*]
# Password for oslo messaging notify service # Password for oslo messaging notify service
# Defaults to hiera('oslo_messaging_notify_password') # Defaults to lookup('oslo_messaging_notify_password')
# #
# [*oslomsg_notify_use_ssl*] # [*oslomsg_notify_use_ssl*]
# Enable ssl oslo messaging services # Enable ssl oslo messaging services
# Defaults to hiera('oslo_messaging_notify_use_ssl', '0') # Defaults to lookup('oslo_messaging_notify_use_ssl', undef, undef, '0')
# #
# [* DEPRECATED PARAMETERS *] # [* DEPRECATED PARAMETERS *]
# #
@ -89,19 +89,19 @@
# Defaults to undef # Defaults to undef
# #
class tripleo::profile::base::designate ( class tripleo::profile::base::designate (
$step = Integer(hiera('step')), $step = Integer(lookup('step')),
$oslomsg_rpc_proto = hiera('oslo_messaging_rpc_scheme', 'rabbit'), $oslomsg_rpc_proto = lookup('oslo_messaging_rpc_scheme', undef, undef, 'rabbit'),
$oslomsg_rpc_hosts = any2array(hiera('oslo_messaging_rpc_node_names', undef)), $oslomsg_rpc_hosts = any2array(lookup('oslo_messaging_rpc_node_names', undef, undef, undef)),
$oslomsg_rpc_password = hiera('oslo_messaging_rpc_password'), $oslomsg_rpc_password = lookup('oslo_messaging_rpc_password'),
$oslomsg_rpc_port = hiera('oslo_messaging_rpc_port', '5672'), $oslomsg_rpc_port = lookup('oslo_messaging_rpc_port', undef, undef, '5672'),
$oslomsg_rpc_username = hiera('oslo_messaging_rpc_user_name', 'guest'), $oslomsg_rpc_username = lookup('oslo_messaging_rpc_user_name', undef, undef, 'guest'),
$oslomsg_rpc_use_ssl = hiera('oslo_messaging_rpc_use_ssl', '0'), $oslomsg_rpc_use_ssl = lookup('oslo_messaging_rpc_use_ssl', undef, undef, '0'),
$oslomsg_notify_proto = hiera('oslo_messaging_notify_scheme', 'rabbit'), $oslomsg_notify_proto = lookup('oslo_messaging_notify_scheme', undef, undef, 'rabbit'),
$oslomsg_notify_hosts = any2array(hiera('oslo_messaging_notify_node_names', undef)), $oslomsg_notify_hosts = any2array(lookup('oslo_messaging_notify_node_names', undef, undef, undef)),
$oslomsg_notify_password = hiera('oslo_messaging_notify_password'), $oslomsg_notify_password = lookup('oslo_messaging_notify_password'),
$oslomsg_notify_port = hiera('oslo_messaging_notify_port', '5672'), $oslomsg_notify_port = lookup('oslo_messaging_notify_port', undef, undef, '5672'),
$oslomsg_notify_username = hiera('oslo_messaging_notify_user_name', 'guest'), $oslomsg_notify_username = lookup('oslo_messaging_notify_user_name', undef, undef, 'guest'),
$oslomsg_notify_use_ssl = hiera('oslo_messaging_notify_use_ssl', '0'), $oslomsg_notify_use_ssl = lookup('oslo_messaging_notify_use_ssl', undef, undef, '0'),
# DEPRECATED PARAMETERS # DEPRECATED PARAMETERS
$rndc_host = undef, $rndc_host = undef,
$rndc_port = undef, $rndc_port = undef,

16
manifests/profile/base/designate/api.pp
View File

@ -21,7 +21,7 @@
# [*step*] # [*step*]
# (Optional) The current step in deployment. See tripleo-heat-templates # (Optional) The current step in deployment. See tripleo-heat-templates
# for more details. # for more details.
# Defaults to hiera('step') # Defaults to Integer(lookup('step'))
# #
# [*certificates_specs*] # [*certificates_specs*]
# (Optional) The specifications to give to certmonger for the certificate(s) # (Optional) The specifications to give to certmonger for the certificate(s)
@ -33,16 +33,16 @@
# service_certificate: <service certificate path> # service_certificate: <service certificate path>
# service_key: <service key path> # service_key: <service key path>
# principal: "haproxy/<overcloud controller fqdn>" # principal: "haproxy/<overcloud controller fqdn>"
# Defaults to hiera('apache_certificate_specs', {}). # Defaults to lookup('apache_certificates_specs', undef, undef, {}).
# #
# [*enable_internal_tls*] # [*enable_internal_tls*]
# (Optional) Whether TLS in the internal network is enabled or not. # (Optional) Whether TLS in the internal network is enabled or not.
# Defaults to hiera('enable_internal_tls', false) # Defaults to lookup('enable_internal_tls', undef, undef, false)
# #
# [*designate_network*] # [*designate_network*]
# (Optional) The network name where the designate endpoint is listening on. # (Optional) The network name where the designate endpoint is listening on.
# This is set by t-h-t. # This is set by t-h-t.
# Defaults to hiera('designate_api_network', undef) # Defaults to lookup('designate_api_network', undef, undef, undef)
# #
# DEPRECATED PARAMETERS # DEPRECATED PARAMETERS
# #
@ -57,10 +57,10 @@
# Defaults to undef # Defaults to undef
# #
class tripleo::profile::base::designate::api ( class tripleo::profile::base::designate::api (
$step = Integer(hiera('step')), $step = Integer(lookup('step')),
$certificates_specs = hiera('apache_certificates_specs', {}), $certificates_specs = lookup('apache_certificates_specs', undef, undef, {}),
$enable_internal_tls = hiera('enable_internal_tls', false), $enable_internal_tls = lookup('enable_internal_tls', undef, undef, false),
$designate_network = hiera('designate_api_network', undef), $designate_network = lookup('designate_api_network', undef, undef, undef),
$listen_ip = undef, $listen_ip = undef,
$listen_port = undef $listen_port = undef
) { ) {

24
manifests/profile/base/designate/authtoken.pp
View File

@ -21,28 +21,28 @@
# [*step*] # [*step*]
# (Optional) The current step in deployment. See tripleo-heat-templates # (Optional) The current step in deployment. See tripleo-heat-templates
# for more details. # for more details.
# Defaults to hiera('step') # Defaults to Integer(lookup('step'))
# #
# [*memcached_hosts*] # [*memcached_hosts*]
# (Optional) Array of hostnames, ipv4 or ipv6 addresses for memcache. # (Optional) Array of hostnames, ipv4 or ipv6 addresses for memcache.
# Defaults to hiera('memcached_node_names', []) # Defaults to lookup('memcached_node_names', undef, undef, [])
# #
# [*memcached_port*] # [*memcached_port*]
# (Optional) Memcached port to use. # (Optional) Memcached port to use.
# Defaults to hiera('memcached_authtoken_port', 11211) # Defaults to lookup('memcached_authtoken_port', undef, undef, 11211)
# #
# [*memcached_ipv6*] # [*memcached_ipv6*]
# (Optional) Whether Memcached uses IPv6 network instead of IPv4 network. # (Optional) Whether Memcached uses IPv6 network instead of IPv4 network.
# Defauls to hiera('memcached_ipv6', false) # Defauls to lookup('memcached_ipv6', undef, undef, false)
# #
# [*security_strategy*] # [*security_strategy*]
# (Optional) Memcached (authtoken) security strategy. # (Optional) Memcached (authtoken) security strategy.
# Defaults to hiera('memcached_authtoken_security_strategy', undef) # Defaults to lookup('memcached_authtoken_security_strategy', undef, undef, undef)
# #
# [*secret_key*] # [*secret_key*]
# (Optional) Memcached (authtoken) secret key, used with security_strategy. # (Optional) Memcached (authtoken) secret key, used with security_strategy.
# The key is hashed with a salt, to isolate services. # The key is hashed with a salt, to isolate services.
# Defaults to hiera('memcached_authtoken_secret_key', undef) # Defaults to lookup('memcached_authtoken_secret_key', undef, undef, undef)
# #
# DEPRECATED PARAMETERS # DEPRECATED PARAMETERS
# #
@ -51,12 +51,12 @@
# Defaults to undef # Defaults to undef
# #
class tripleo::profile::base::designate::authtoken ( class tripleo::profile::base::designate::authtoken (
$step = Integer(hiera('step')), $step = Integer(lookup('step')),
$memcached_hosts = hiera('memcached_node_names', []), $memcached_hosts = lookup('memcached_node_names', undef, undef, []),
$memcached_port = hiera('memcached_authtoken_port', 11211), $memcached_port = lookup('memcached_authtoken_port', undef, undef, 11211),
$memcached_ipv6 = hiera('memcached_ipv6', false), $memcached_ipv6 = lookup('memcached_ipv6', undef, undef, false),
$security_strategy = hiera('memcached_authtoken_security_strategy', undef), $security_strategy = lookup('memcached_authtoken_security_strategy', undef, undef, undef),
$secret_key = hiera('memcached_authtoken_secret_key', undef), $secret_key = lookup('memcached_authtoken_secret_key', undef, undef, undef),
# DEPRECATED PARAMETERS # DEPRECATED PARAMETERS
$memcached_ips = undef $memcached_ips = undef
) { ) {

8
manifests/profile/base/designate/backend.pp
View File

@ -21,15 +21,15 @@
# [*step*] # [*step*]
# (Optional) The current step in deployment. See tripleo-heat-templates # (Optional) The current step in deployment. See tripleo-heat-templates
# for more details. # for more details.
# Defaults to hiera('step') # Defaults to Integer(lookup('step'))
# #
# [*backend*] # [*backend*]
# (Optional) Specify a backend used. # (Optional) Specify a backend used.
# Defaults to 'bind9' # Defaults to lookup('designate_backend', undef, undef, 'bind9'),
# #
class tripleo::profile::base::designate::backend ( class tripleo::profile::base::designate::backend (
$step = Integer(hiera('step')), $step = Integer(lookup('step')),
$backend = hiera('designate_backend', 'bind9'), $backend = lookup('designate_backend', undef, undef, 'bind9'),
) { ) {
if $step >= 4 { if $step >= 4 {
if $backend == 'bind9' { if $backend == 'bind9' {

8
manifests/profile/base/designate/central.pp
View File

@ -20,12 +20,12 @@
# #
# [*bootstrap_node*] # [*bootstrap_node*]
# (Optional) The hostname of the node responsible for bootstrapping tasks # (Optional) The hostname of the node responsible for bootstrapping tasks
# Defaults to hiera('designate_central_short_bootstrap_node_name') # Defaults to lookup('designate_central_short_bootstrap_node_name', undef, undef, undef)
# #
# [*step*] # [*step*]
# (Optional) The current step in deployment. See tripleo-heat-templates # (Optional) The current step in deployment. See tripleo-heat-templates
# for more details. # for more details.
# Defaults to hiera('step') # Defaults to Integer(lookup('step'))
# #
# DEPRECATED PARAMETERS # DEPRECATED PARAMETERS
# #
@ -34,8 +34,8 @@
# Defaults to the content of templates/designate/pools.yaml.erb # Defaults to the content of templates/designate/pools.yaml.erb
# #
class tripleo::profile::base::designate::central ( class tripleo::profile::base::designate::central (
$bootstrap_node = hiera('designate_central_short_bootstrap_node_name', undef), $bootstrap_node = lookup('designate_central_short_bootstrap_node_name', undef, undef, undef),
$step = Integer(hiera('step')), $step = Integer(lookup('step')),
# DEPRECATED PARAMETERS # DEPRECATED PARAMETERS
$pools_file_content = undef, $pools_file_content = undef,
) { ) {

16
manifests/profile/base/designate/coordination.pp
View File

@ -22,25 +22,25 @@
# [*step*] # [*step*]
# (Optional) The current step in deployment. See tripleo-heat-templates # (Optional) The current step in deployment. See tripleo-heat-templates
# for more details. # for more details.
# Defaults to hiera('step') # Defaults to Integer(lookup('step'))
# #
# [*designate_redis_password*] # [*designate_redis_password*]
# (Optional) Password for the neutron redis user for the coordination url # (Optional) Password for the neutron redis user for the coordination url
# Defaults to hiera('designate_redis_password', undef), # Defaults to lookup('designate_redis_password', undef, undef, undef),
# #
# [*redis_vip*] # [*redis_vip*]
# (Optional) Redis ip address for the coordination url # (Optional) Redis ip address for the coordination url
# Defaults to hiera('redis_vip', undef), # Defaults to lookup('redis_vip', undef, undef, undef),
# #
# [*enable_internal_tls*] # [*enable_internal_tls*]
# (Optional) Whether TLS in the internal network is enabled or not. # (Optional) Whether TLS in the internal network is enabled or not.
# Defaults to hiera('enable_internal_tls', false) # Defaults to lookup('enable_internal_tls', undef, undef, false)
# #
class tripleo::profile::base::designate::coordination ( class tripleo::profile::base::designate::coordination (
$step = Integer(hiera('step')), $step = Integer(lookup('step')),
$designate_redis_password = hiera('designate_redis_password', undef), $designate_redis_password = lookup('designate_redis_password', undef, undef, undef),
$redis_vip = hiera('redis_vip', undef), $redis_vip = lookup('redis_vip', undef, undef, undef),
$enable_internal_tls = hiera('enable_internal_tls', false), $enable_internal_tls = lookup('enable_internal_tls', undef, undef, false),
) { ) {
if $step >= 4 { if $step >= 4 {
if $redis_vip { if $redis_vip {

4
manifests/profile/base/designate/mdns.pp
View File

@ -21,10 +21,10 @@
# [*step*] # [*step*]
# (Optional) The current step in deployment. See tripleo-heat-templates # (Optional) The current step in deployment. See tripleo-heat-templates
# for more details. # for more details.
# Defaults to hiera('step') # Defaults to Integer(lookup('step'))
# #
class tripleo::profile::base::designate::mdns ( class tripleo::profile::base::designate::mdns (
$step = Integer(hiera('step')), $step = Integer(lookup('step')),
) { ) {
include tripleo::profile::base::designate include tripleo::profile::base::designate
if $step >= 4 { if $step >= 4 {

4
manifests/profile/base/designate/producer.pp
View File

@ -21,10 +21,10 @@
# [*step*] # [*step*]
# (Optional) The current step in deployment. See tripleo-heat-templates # (Optional) The current step in deployment. See tripleo-heat-templates
# for more details. # for more details.
# Defaults to hiera('step') # Defaults to Integer(lookup('step'))
# #
class tripleo::profile::base::designate::producer ( class tripleo::profile::base::designate::producer (
$step = Integer(hiera('step')), $step = Integer(lookup('step')),
) { ) {
include tripleo::profile::base::designate include tripleo::profile::base::designate
include tripleo::profile::base::designate::coordination include tripleo::profile::base::designate::coordination

4
manifests/profile/base/designate/sink.pp
View File

@ -21,10 +21,10 @@
# [*step*] # [*step*]
# (Optional) The current step in deployment. See tripleo-heat-templates # (Optional) The current step in deployment. See tripleo-heat-templates
# for more details. # for more details.
# Defaults to hiera('step') # Defaults to Integer(lookup('step'))
# #
class tripleo::profile::base::designate::sink ( class tripleo::profile::base::designate::sink (
$step = Integer(hiera('step')), $step = Integer(lookup('step')),
) { ) {
include tripleo::profile::base::designate include tripleo::profile::base::designate
if $step >= 4 { if $step >= 4 {

8
manifests/profile/base/designate/worker.pp
View File

@ -21,18 +21,18 @@
# [*step*] # [*step*]
# (Optional) The current step in deployment. See tripleo-heat-templates # (Optional) The current step in deployment. See tripleo-heat-templates
# for more details. # for more details.
# Defaults to hiera('step') # Defaults to Integer(lookup('step'))
# #
# DEPRECATED PARAMETERS # DEPRECATED PARAMETERS
# #
# [*rndc_key*] # [*rndc_key*]
# (Optional) The base64-encoded key secret for /etc/rndc.key. # (Optional) The base64-encoded key secret for /etc/rndc.key.
# Defaults to hiera('designate_rndc_key') # Defaults to lookup('designate_rndc_key', undef, undef, false)
# #
class tripleo::profile::base::designate::worker ( class tripleo::profile::base::designate::worker (
$step = Integer(hiera('step')), $step = Integer(lookup('step')),
# DEPRECATED PARAMETERS # DEPRECATED PARAMETERS
$rndc_key = hiera('designate_rndc_key', false), $rndc_key = lookup('designate_rndc_key', undef, undef, false),
) { ) {
include tripleo::profile::base::designate include tripleo::profile::base::designate