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

Unified Diff: modules/adblockplus/manifests/log.pp

Issue 29483572: #2007 - Fix $adblockplus::log::rotations computation (Closed)
Patch Set: For comments 2 to 4 Created July 10, 2017, 3:57 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: modules/adblockplus/manifests/log.pp
===================================================================
--- a/modules/adblockplus/manifests/log.pp
+++ b/modules/adblockplus/manifests/log.pp
@@ -13,7 +13,7 @@
#
# [*rotations*]
# A hash of adblockplus::log::rotation $name => $parameter items
-# to set up in this context, i.e. via Hiera.
+# to set up in this context via Hiera only.
#
# [*trackers*]
# A hash of adblockplus::log::rotation $title => $parameter items
@@ -32,7 +32,6 @@
#
class adblockplus::log (
$ensure = 'present',
- $rotations = hiera('adblockplus::log::rotations', {}),
$trackers = hiera('adblockplus::log::trackers', {}),
) {
@@ -92,6 +91,7 @@
}
# See modules/adblockplus/manifests/log/rotation.pp
+ $rotations = hiera_hash('adblockplus::log::rotations', {})
create_resources('adblockplus::log::rotation', $rotations)
# See modules/adblockplus/manifests/log/tracker.pp
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld