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

Side by Side Diff: modules/fluent/manifests/config.pp

Issue 29469606: #1634 - Update file resource syntax in module fluent (Closed)
Patch Set: Created June 20, 2017, 4:55 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | modules/fluent/manifests/init.pp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # == Type: fluent::config 1 # == Type: fluent::config
2 # 2 #
3 # Maintain Fluentd configuration files. 3 # Maintain Fluentd configuration files.
4 # 4 #
5 # Use "@include config.d/*.conf" in the main configuration ($fluent::config) 5 # Use "@include config.d/*.conf" in the main configuration ($fluent::config)
6 # to include all fragments at once, or "@include config.d/$name.conf" to pick 6 # to include all fragments at once, or "@include config.d/$name.conf" to pick
7 # and choose. 7 # and choose.
8 # 8 #
9 # Type fluent::config is a thin layer around a single Puppet file resource 9 # Type fluent::config is a thin layer around a single Puppet file resource
10 # definition, combining the available parameters with ones computed internally 10 # definition, combining the available parameters with ones computed internally
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 $target = undef, 47 $target = undef,
48 ) { 48 ) {
49 49
50 include fluent 50 include fluent
51 include stdlib 51 include stdlib
52 52
53 file {"fluent::config#$title": 53 file {"fluent::config#$title":
54 content => $content, 54 content => $content,
55 ensure => $ensure, 55 ensure => $ensure,
56 group => $fluent::group, 56 group => $fluent::group,
57 mode => 0640, 57 mode => '0640',
58 notify => Service['fluent'], 58 notify => Service['fluent'],
59 owner => getparam(File['fluent'], 'owner'), 59 owner => getparam(File['fluent'], 'owner'),
60 path => "$fluent::directory/config.d/$name.conf", 60 path => "$fluent::directory/config.d/$name.conf",
61 require => File["$fluent::directory/config.d"], 61 require => File["$fluent::directory/config.d"],
62 source => $source, 62 source => $source,
63 target => $target, 63 target => $target,
64 } 64 }
65 } 65 }
OLDNEW
« no previous file with comments | « no previous file | modules/fluent/manifests/init.pp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld