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

Delta Between Two Patch Sets: modules/adblockplus/manifests/log.pp

Issue 29483572: #2007 - Fix $adblockplus::log::rotations computation (Closed)
Left Patch Set: Created July 8, 2017, 6:49 p.m.
Right Patch Set: For comments 2 to 4 Created July 10, 2017, 3:57 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « no previous file | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 # == Class: adblockplus::log 1 # == Class: adblockplus::log
2 # 2 #
3 # Default root namespace for integrating custom logging entities. 3 # Default root namespace for integrating custom logging entities.
4 # 4 #
5 # === Parameters: 5 # === Parameters:
6 # 6 #
7 # [*ensure*] 7 # [*ensure*]
8 # Whether associated resources are ment to be 'present' or 'absent'. 8 # Whether associated resources are ment to be 'present' or 'absent'.
9 # 9 #
10 # [*name*] 10 # [*name*]
(...skipping 73 matching lines...) Expand 10 before | Expand all | Expand 10 after
84 groups => [$group], 84 groups => [$group],
85 home => "$adblockplus::directory/log", 85 home => "$adblockplus::directory/log",
86 managehome => true, 86 managehome => true,
87 require => [ 87 require => [
88 File[$adblockplus::directory], 88 File[$adblockplus::directory],
89 User['fluent'], 89 User['fluent'],
90 ], 90 ],
91 } 91 }
92 92
93 # See modules/adblockplus/manifests/log/rotation.pp 93 # See modules/adblockplus/manifests/log/rotation.pp
94 $rotations = hiera('adblockplus::log::rotations', {}) 94 $rotations = hiera_hash('adblockplus::log::rotations', {})
95 create_resources('adblockplus::log::rotation', $rotations) 95 create_resources('adblockplus::log::rotation', $rotations)
96 96
97 # See modules/adblockplus/manifests/log/tracker.pp 97 # See modules/adblockplus/manifests/log/tracker.pp
98 create_resources('adblockplus::log::tracker', $trackers) 98 create_resources('adblockplus::log::tracker', $trackers)
99 } 99 }
LEFTRIGHT
« no previous file | no next file » | Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Toggle Comments ('s')

Powered by Google App Engine
This is Rietveld