The hiera function is deprecated and does not work with the latest
hieradata version 5. It should be replaced by the new lookup
function[1].
[1] https://puppet.com/docs/puppet/7/hiera_automatic.html
With the lookup function, we can define value type and merge behavior,
but these are kept default at this moment to limit scope of this change
to just simple replacement. Adding value type might be useful to make
sure the value is in expected type (especially when a boolean value is
expected), but we will revisit that later.
example:
lookup(<NAME>, [<VALUE TYPE>], [<MERGE BEHAVIOR>], [<DEFAULT VALUE>])
Change-Id: I77cb622a7ec1471715bb3e4fcb0157e5a6b80e74
Currently we override default values for backend manifests by undef
but because of this we already need to define each parameters otherwise
puppet-cinder raises validation failure or set empty values.
This change removes that redundant overrides and make puppet-tripleo
rely on defaults in puppet-cinder (and most of them are
$::os_service_default which result in defaults in cinder itself)
Change-Id: I0089a53ce630a7e84b193af427455ab96b04ec48
Enhance tripleo::profile::base::cinder::volume::pure to support
deploying multiple instances of the cinder::backend::pure backend.
The changes are backward compatible with existing single instance
deployments.
The patch updates the $backend_name parameter to support a list
of names instead of a single name. It also adds a new (optional)
$multi_config parameter, which is a hash of values associated with
each backend. The hash keys match the existing THT parameter names.
Co-Authored-By: Alan Bishop <abishop@redhat.com>
Change-Id: Ia7cc82f5eb4e228a43e47624d87e319ac5340268
Add the ability to override the "backend_availability_zone" parameter
in every cinder volume backend.
Implements: blueprint split-controlplane-cinder-volume-az
Depends-On: I11821a38d8ba5afc594b3d601cd1634207a6f093
Change-Id: Ic407b747474b567858ad36beabc8a7d8c5022343
The step is typically set with the hieradata setting an integer value:
{"step": 1}
However it would be useful for the value to be a string so that
substitutions are possible, for example:
{"step": "%{::step}"}
This change ensures the step parameter defaults to an integer by
calling Integer(hiera('step'))
This change was made by manually removing the undef defaults from
fluentd.pp, uchiwa.pp, and sensu.pp then bulk updating with:
find ./ -type f -print0 |xargs -0 sed -i "s/= hiera('step')/= Integer(hiera('step'))/"
Change-Id: I8a47ca53a7dea8391103abcb8960a97036a6f5b3