X-Git-Url: https://review.fuel-infra.org/gitweb?a=blobdiff_plain;f=manifests%2Fsetting.pp;h=91835cb1b2b88f15387b276660e50a26828bed91;hb=a30791306b26a18d79875dbc1d99b11205802bcc;hp=507ec9495d66ac2e6d305f71596996d34e7e1973;hpb=e014fc04ed8358e4f7c2e87f7434df9817a4c7bb;p=puppet-modules%2Fpuppetlabs-apt.git diff --git a/manifests/setting.pp b/manifests/setting.pp index 507ec94..91835cb 100644 --- a/manifests/setting.pp +++ b/manifests/setting.pp @@ -18,11 +18,15 @@ define apt::setting ( $setting_type = $title_array[0] $base_name = join(delete_at($title_array, 0), '-') - validate_re($setting_type, ['\Aconf\z', '\Apref\z', '\Alist\z'], "apt::setting resource name/title must start with either 'conf-', 'pref-' or 'list-'") + assert_type(Pattern[/\Aconf\z/, /\Apref\z/, /\Alist\z/], $setting_type) |$a, $b| { + fail("apt::setting resource name/title must start with either 'conf-', 'pref-' or 'list-'") + } - unless is_integer($priority) { + if $priority !~ Integer { # need this to allow zero-padded priority. - validate_re($priority, '^\d+$', 'apt::setting priority must be an integer or a zero-padded integer') + assert_type(Pattern[/^\d+$/], $priority) |$a, $b| { + fail('apt::setting priority must be an integer or a zero-padded integer') + } } if ($setting_type == 'list') or ($setting_type == 'pref') {