Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: modules/geoip/manifests/init.pp

Issue 29783593: #11294 - Drop $::geoip::version parameter (Closed)
Patch Set: Created May 16, 2018, 11:54 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: modules/geoip/manifests/init.pp
diff --git a/modules/geoip/manifests/init.pp b/modules/geoip/manifests/init.pp
index b36d08a92fbedf1d8cae5dac8e9f237f91e909bb..dcf018ef89c29f719a82ed4a39ba6e9313b5c6be 100644
--- a/modules/geoip/manifests/init.pp
+++ b/modules/geoip/manifests/init.pp
@@ -19,9 +19,6 @@
# [*script*]
# Where to store the update script executed by Cron['geoip'].
#
-# [*version*]
-# A specific version to ensure for all $packages, optional.
-#
# === Examples:
#
# class {'geoip':
@@ -39,14 +36,10 @@ class geoip (
$hook = undef,
$packages = ['geoip-database'],
$script = '/usr/local/sbin/update-geoip-database',
- $version = undef,
) {
ensure_resource('package', $packages, {
- ensure => $ensure ? {
- /^(absent|purged)$/ => $ensure,
- default => $version ? {undef => 'present', default => $version},
- },
+ ensure => $ensure,
})
create_resources('cron', {geoip => $cron}, {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld