openstack
/
puppet-monasca
Code Issues Proposed changes

Merge "Use validate_legacy"

This commit is contained in:
Zuul 2019-02-25 17:30:38 +00:00 committed by Gerrit Code Review
parent 0102c8e543 c2aa24ac27
commit 146d6cf1ef
4 changed files with 26 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
manifests/config.pp
View File

@ -31,8 +31,9 @@ class monasca::config (
$monasca_config = {},
$monasca_ini = {},
) {
validate_hash($monasca_config)
validate_hash($monasca_ini)
validate_legacy(Hash, 'validate_hash', $monasca_config)
validate_legacy(Hash, 'validate_hash', $monasca_ini)
create_resources('monasca_config', $monasca_config)
create_resources('monasca_ini', $monasca_ini)

18
manifests/params.pp
View File

@ -93,15 +93,17 @@ class monasca::params(
'database_type' => 'influxdb',
}
) {
include ::openstacklib::defaults
validate_string($admin_password)
validate_string($admin_project_name)
validate_string($user_password)
validate_string($agent_password)
validate_string($sql_password)
validate_string($sql_host)
validate_string($api_db_password)
validate_string($pers_db_password)
validate_legacy(String, 'validate_string', $admin_password)
validate_legacy(String, 'validate_string', $admin_project_name)
validate_legacy(String, 'validate_string', $user_password)
validate_legacy(String, 'validate_string', $agent_password)
validate_legacy(String, 'validate_string', $sql_password)
validate_legacy(String, 'validate_string', $sql_host)
validate_legacy(String, 'validate_string', $api_db_password)
validate_legacy(String, 'validate_string', $pers_db_password)
if $::osfamily == 'Debian' {
$agent_package = 'monasca-agent'

9
manifests/virtualenv/agent_instance.pp
View File

@ -36,13 +36,16 @@ define monasca::virtualenv::agent_instance(
$venv_active = false,
$venv_extra_args = undef,
) {
validate_string($ensure)
validate_legacy(String, 'validate_string', $ensure)
$valid_values = [
'^present$',
'^absent$',
]
validate_re($ensure, $valid_values,
"Unknown value '${ensure}' for ensure, must be present or absent")
validate_legacy(Enum['present', 'absent'], 'validate_re', $ensure,
[$valid_values, "Unknown value '${ensure}' for ensure, must be present or absent"])
File[$basedir] -> anchor { 'monasca::virtualenv::instance': }
Package<| name == 'python-virtualenv' |> -> Anchor['monasca::virtualenv::instance']

11
manifests/virtualenv/instance.pp
View File

@ -58,20 +58,23 @@ define monasca::virtualenv::instance(
$venv_active = false,
$venv_extra_args = undef,
) {
validate_string($ensure)
validate_legacy(String, 'validate_string', $ensure)
$valid_values = [
'^present$',
'^absent$',
]
validate_re($ensure, $valid_values,
"Unknown value '${ensure}' for ensure, must be present or absent")
validate_legacy(Enum['present', 'absent'], 'validate_re', $ensure,
[$valid_values, "Unknown value '${ensure}' for ensure, must be present or absent"])
$req_dest = "${basedir}/${venv_prefix}-requirements.txt"
$venv_dir = "${basedir}/${venv_prefix}-venv"
$venv_name = "${venv_prefix}-${name}"
if $ensure == 'present' {
validate_string($venv_requirements)
validate_legacy(String, 'validate_string', $venv_requirements)
file { $req_dest:
ensure => 'file',