openstack
/
puppet-designate
Code Issues Proposed changes

Merge "Use validate_legacy"

This commit is contained in:
Zuul 2019-02-25 18:10:33 +00:00 committed by Gerrit Code Review
parent 4055441565 d83b0dafdc
commit e977484cef
8 changed files with 21 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
manifests/config.pp
View File

@ -38,9 +38,9 @@ class designate::config (
include ::designate::deps
validate_hash($designate_config)
validate_hash($api_paste_ini_config)
validate_hash($rootwrap_config)
validate_legacy(Hash, 'validate_hash', $designate_config)
validate_legacy(Hash, 'validate_hash', $api_paste_ini_config)
validate_legacy(Hash, 'validate_hash', $rootwrap_config)
create_resources('designate_config', $designate_config)
create_resources('designate_api_paste_ini', $api_paste_ini_config)

4
manifests/db.pp
View File

@ -57,8 +57,8 @@ class designate::db (
include ::designate::deps
include ::designate::params
validate_re($database_connection,
'(mysql(\+pymysql)?):\/\/(\S+:\S+@\S+\/\S+)?')
validate_legacy(String, 'validate_re', $database_connection,
['(mysql(\+pymysql)?):\/\/(\S+:\S+@\S+\/\S+)?'])
case $database_connection {
/^mysql(\+pymysql)?:\/\//: {

2
manifests/policy.pp
View File

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

12
manifests/pool.pp
View File

@ -21,11 +21,13 @@ define designate::pool(
include ::designate::deps
validate_re($name, '[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}',
'Pool name should be a UUID.')
validate_array($nameservers)
validate_array($targets)
validate_array($also_notifies)
validate_legacy(Pattern[/[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}/],
'validate_re', $name, ['[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}',
'Pool name should be a UUID.'])
validate_legacy(Array, 'validate_array', $nameservers)
validate_legacy(Array, 'validate_array', $targets)
validate_legacy(Array, 'validate_array', $also_notifies)
designate_config {
"pool:${name}/nameservers": value => join($nameservers,',');

2
manifests/pool_manager_cache/memcache.pp
View File

@ -19,7 +19,7 @@ class designate::pool_manager_cache::memcache(
include ::designate::deps
validate_array($memcached_servers)
validate_legacy(Array, 'validate_array', $memcached_servers)
designate_config {
'service:pool_manager/cache_driver': value => 'memcache';

5
manifests/pool_nameserver.pp
View File

@ -15,8 +15,9 @@ define designate::pool_nameserver(
include ::designate::deps
validate_re($name, '[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}',
'Name should be a UUID.')
validate_legacy(Pattern[/[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}/],
'validate_re', $name, ['[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}',
'Name should be a UUID.'])
designate_config {
"pool_nameserver:${name}/port": value => $port;

5
manifests/pool_target.pp
View File

@ -35,8 +35,9 @@ define designate::pool_target (
include ::powerdns::mysql
}
validate_re($name, '[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}')
validate_array($masters)
validate_legacy(Pattern[/[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}/],
'validate_re', $name, ['[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}'])
validate_legacy(Array, 'validate_array', $masters)
if is_hash($options) {
$options_real = join(join_keys_to_values($options,':'),',')

2
spec/classes/designate_db_spec.rb
View File

@ -39,7 +39,7 @@ describe 'designate::db' do
{ :database_connection => 'redis://designate:designate@localhost/designate', }
end
it_raises 'a Puppet::Error', /validate_re/
it { should raise_error(Puppet::Error) }
end
context 'with specific parameters' do