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

Delta Between Two Patch Sets: hiera/roles/logprocessor.yaml

Issue 29338675: Issue 3694 - Introduce class adblockplus::log::processor and role logprocessor (Closed)
Left Patch Set: Created March 18, 2016, 6:08 p.m.
Right Patch Set: Issue 3694 - Addressing feedback from code-review Created March 21, 2016, 4:44 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 | modules/adblockplus/manifests/log/processor.pp » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 # This role is meant for development, testing and debugging purpose only 1 # This role is meant for development, testing and debugging purpose only
2 classes: 2 classes:
3 adblockplus::log::processor: 3 adblockplus::log::processor:
4 fluent: 4 fluent:
5 package: 5 package:
6 ensure: "2.3.1-0" 6 ensure: "2.3.1-0"
7 provider: "apt" 7 provider: "apt"
8 user: 8 user:
9 managehome: true 9 managehome: true
10 groups: 10 groups:
(...skipping 25 matching lines...) Expand all
36 content: | 36 content: |
37 <match **> 37 <match **>
38 @type file 38 @type file
39 path /tmp/fluentd-example.log 39 path /tmp/fluentd-example.log
40 </match> 40 </match>
41 ensure: "present" 41 ensure: "present"
42 name: "80-output" 42 name: "80-output"
43 43
44 # modules/fluent/manifests/gem.pp 44 # modules/fluent/manifests/gem.pp
45 fluent::gems: 45 fluent::gems:
46 fluent-plugin-secure-forward: 46 fluent-plugin-secure-forward:
Fred 2016/03/21 16:33:35 Do we really need/want this plugin in the developm
mathias 2016/03/21 16:36:22 Actually I do not want any of these plugins in dev
Fred 2016/03/21 16:42:20 That suggestion sounds reasonable to me.
47 ensure: "present" 47 ensure: "absent"
48 fluent-plugin-grep: 48 fluent-plugin-grep:
49 ensure: "present" 49 ensure: "present"
50 50
LEFTRIGHT

Powered by Google App Engine
This is Rietveld