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

Side by Side Diff: modules/fluent/manifests/init.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 | « modules/fluent/manifests/config.pp ('k') | modules/fluent/manifests/plugin.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 # == Class: fluent 1 # == Class: fluent
2 # 2 #
3 # Create and maintain Fluentd (https://www.fluentd.org/) setups. 3 # Create and maintain Fluentd (https://www.fluentd.org/) setups.
4 # 4 #
5 # == Parameters: 5 # == Parameters:
6 # 6 #
7 # [*config*] 7 # [*config*]
8 # Overwrite the default concat/file options for the Fluentd agent config. 8 # Overwrite the default concat/file options for the Fluentd agent config.
9 # 9 #
10 # [*key*] 10 # [*key*]
(...skipping 102 matching lines...) Expand 10 before | Expand all | Expand 10 after
113 file {[ 113 file {[
114 "$directory/config.d", 114 "$directory/config.d",
115 "$directory/plugin", 115 "$directory/plugin",
116 ]: 116 ]:
117 before => File[$title], 117 before => File[$title],
118 ensure => getparam(File[$title], 'ensure') ? { 118 ensure => getparam(File[$title], 'ensure') ? {
119 /^(present|link)$/ => 'directory', 119 /^(present|link)$/ => 'directory',
120 default => 'absent', 120 default => 'absent',
121 }, 121 },
122 group => getparam(File[$title], 'group'), 122 group => getparam(File[$title], 'group'),
123 mode => 0755, 123 mode => '0755',
124 owner => getparam(File[$title], 'owner'), 124 owner => getparam(File[$title], 'owner'),
125 require => Package[$title], 125 require => Package[$title],
126 } 126 }
127 127
128 # The only package provider recognized implicitly 128 # The only package provider recognized implicitly
129 if getparam(Package[$title], 'provider') == 'apt' { 129 if getparam(Package[$title], 'provider') == 'apt' {
130 130
131 ensure_resource('apt::key', $title, merge({ 131 ensure_resource('apt::key', $title, merge({
132 ensure => $ensure, 132 ensure => $ensure,
133 name => 'treasure-data', 133 name => 'treasure-data',
134 }, $key)) 134 }, $key))
135 135
136 ensure_resource('apt::source', $title, merge({ 136 ensure_resource('apt::source', $title, merge({
137 ensure => $ensure, 137 ensure => $ensure,
138 include_src => false, 138 include_src => false,
139 name => 'treasure-data', 139 name => 'treasure-data',
140 }, $source)) 140 }, $source))
141 141
142 Apt::Source[$title] <- Apt::Key[$title] 142 Apt::Source[$title] <- Apt::Key[$title]
143 Apt::Source[$title] -> Package[$title] 143 Apt::Source[$title] -> Package[$title]
144 } 144 }
145 } 145 }
OLDNEW
« no previous file with comments | « modules/fluent/manifests/config.pp ('k') | modules/fluent/manifests/plugin.pp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld