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

Unified Diff: modules/web/manifests/server.pp

Issue 29785567: #11294 - Generate and store anonymized HTTPd logs (Closed)
Patch Set: #11294 - Remove $adblockplus::web::server::geoip parameter Created June 5, 2018, 1:46 p.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 | « modules/private-stub/hiera/base.yaml ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: modules/web/manifests/server.pp
diff --git a/modules/web/manifests/server.pp b/modules/web/manifests/server.pp
index dd88f1083f02bc8eb2175da4bcbb8600b97d6b40..ec81d393eb59f09a58d69408ec5fda620a4b61cf 100644
--- a/modules/web/manifests/server.pp
+++ b/modules/web/manifests/server.pp
@@ -7,11 +7,11 @@ class web::server(
$aliases = undef,
$custom_config = undef,
$multiplexer_locations = undef,
- $geoip = false,
) {
include sitescripts
include adblockplus::web
+ include geoip
$remote = hiera('web::server::remote', "https://hg.adblockplus.org/${repository}")
@@ -35,15 +35,7 @@ class web::server(
}
class {'nginx':
- geoip_country => $geoip ? {
- false => undef,
- default => '/usr/share/GeoIP/GeoIPv6.dat',
- },
- }
-
- class {'geoip':
- cron => {hour => 0, minute => 8, monthday => 15},
- ensure => $geoip ? {false => 'absent', default => 'present'},
+ geoip_country => '/usr/share/GeoIP/GeoIPv6.dat',
}
# Make sure that apt packages corresponding to the pip-installed modules below
« no previous file with comments | « modules/private-stub/hiera/base.yaml ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld