openstack/puppet-glare
Code Issues Proposed changes

Merge "Use validate_legacy"

Browse Source
This commit is contained in:
Zuul 2019-02-25 17:21:23 +00:00 committed by Gerrit Code Review
commit 82045d7158
4 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/config.pp
View File

@ -30,8 +30,8 @@ class glare::config (
include ::glare::deps
validate_hash($glare_config)
validate_hash($glare_paste_ini_config)
validate_legacy(Hash, 'validate_hash', $glare_config)
validate_legacy(Hash, 'validate_hash', $glare_paste_ini_config)
create_resources('glare_config', $glare_config)
create_resources('glare_paste_ini', $glare_paste_ini_config)

4
manifests/db.pp
View File

@ -56,8 +56,8 @@ class glare::db (
include ::glare::deps
validate_re($database_connection,
'^(sqlite|mysql(\+pymysql)?|postgresql):\/\/(\S+:\S+@\S+\/\S+)?')
validate_legacy(Oslo::Dbconn, 'validate_re', $database_connection,
['^(sqlite|mysql(\+pymysql)?|postgresql):\/\/(\S+:\S+@\S+\/\S+)?'])
oslo::db { 'glare_config':
connection => $database_connection,

2
manifests/db/mysql.pp
View File

@ -45,7 +45,7 @@ class glare::db::mysql(
include ::glare::deps
validate_string($password)
validate_legacy(String, 'validate_string', $password)
::openstacklib::db::mysql { 'glare':
user => $user,

2
manifests/policy.pp
View File

@ -31,7 +31,7 @@ class glare::policy (
include ::glare::deps
include ::glare::params
validate_hash($policies)
validate_legacy(Hash, 'validate_hash', $policies)
Openstacklib::Policy::Base {
file_path => $policy_path,