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

Side by Side Diff: modules/adblockplus/manifests/log.pp

Issue 29469584: #1634 - Update file resource syntax in module adblockplus (Closed)
Patch Set: Created June 20, 2017, 4:42 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
OLDNEW
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 56 matching lines...) Expand 10 before | Expand all | Expand 10 after
67 content => template('adblockplus/log/fluentd/default.conf.erb'), 67 content => template('adblockplus/log/fluentd/default.conf.erb'),
68 ensure => $ensure, 68 ensure => $ensure,
69 name => '90-adblockplus-log-defaults', 69 name => '90-adblockplus-log-defaults',
70 } 70 }
71 71
72 # Invoke realize(File[$adblockplus::log::directory]) when neccessary 72 # Invoke realize(File[$adblockplus::log::directory]) when neccessary
73 @file {$directory: 73 @file {$directory:
74 before => Service['fluent'], 74 before => Service['fluent'],
75 ensure => 'directory', 75 ensure => 'directory',
76 group => $group, 76 group => $group,
77 mode => 0775, 77 mode => '0775',
78 owner => $user, 78 owner => $user,
79 } 79 }
80 80
81 # Invoke realize(User[$adblockplus::log::user]) when necessary 81 # Invoke realize(User[$adblockplus::log::user]) when necessary
82 @user {$user: 82 @user {$user:
83 before => File[$directory], 83 before => File[$directory],
84 ensure => $ensure, 84 ensure => $ensure,
85 groups => [$group], 85 groups => [$group],
86 home => "$adblockplus::directory/log", 86 home => "$adblockplus::directory/log",
87 managehome => true, 87 managehome => true,
88 require => [ 88 require => [
89 File[$adblockplus::directory], 89 File[$adblockplus::directory],
90 User['fluent'], 90 User['fluent'],
91 ], 91 ],
92 } 92 }
93 93
94 # See modules/adblockplus/manifests/log/rotation.pp 94 # See modules/adblockplus/manifests/log/rotation.pp
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 }
OLDNEW

Powered by Google App Engine
This is Rietveld