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

Unified Diff: modules/adblockplus/manifests/legacy/filterserver.pp

Issue 29511584: #2762 - Fix provisioning of geoip to filter servers (Closed) Base URL: https://hg1/infrastructure
Patch Set: split the absent package into a legacy class Created Aug. 11, 2017, 12:37 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
« hiera/roles/filterserver.yaml ('K') | « hiera/roles/filterserver.yaml ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: modules/adblockplus/manifests/legacy/filterserver.pp
===================================================================
new file mode 100644
--- /dev/null
+++ b/modules/adblockplus/manifests/legacy/filterserver.pp
@@ -0,0 +1,15 @@
+# == Class: adblockplus::legacy::filterserver
+#
+# A class for maintaining legacy filter servers.
+#
+# See http://hub.eyeo.com/issues/2762 for more information.
+#
+class adblockplus::legacy::filterserver {
+
+ # Remove any manually installed geoip-database-contrib
+ package{'geoip-database-contrib':
+ ensure => 'purged',
+ require => Cron['geoip'],
mathias 2017/08/14 08:04:33 Why should ensuring absence require anything?
+ }
+}
+
« hiera/roles/filterserver.yaml ('K') | « hiera/roles/filterserver.yaml ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld