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

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

Issue 29469614: #1634 - Update file resource syntax in module logrotate (Closed)
Patch Set: Created June 20, 2017, 4:59 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 | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # == Type: logrotate::config 1 # == Type: logrotate::config
2 # 2 #
3 # A shorthand wrapper that sets up a logrotate configuration file resources 3 # A shorthand wrapper that sets up a logrotate configuration file resources
4 # with the same $title, and properly configured attributes like e.g. $path. 4 # with the same $title, and properly configured attributes like e.g. $path.
5 # 5 #
6 # === Parameters: 6 # === Parameters:
7 # 7 #
8 # [*content*] 8 # [*content*]
9 # Translates directly into the configuration file content. 9 # Translates directly into the configuration file content.
10 # Mutually exclusive with $source. 10 # Mutually exclusive with $source.
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
43 $source = undef, 43 $source = undef,
44 ) { 44 ) {
45 45
46 file {$title: 46 file {$title:
47 content => $content, 47 content => $content,
48 ensure => $ensure ? { 48 ensure => $ensure ? {
49 /^(absent|purged)$/ => 'absent', 49 /^(absent|purged)$/ => 'absent',
50 default => 'present', 50 default => 'present',
51 }, 51 },
52 group => 'root', 52 group => 'root',
53 mode => 0644, 53 mode => '0644',
54 owner => 'root', 54 owner => 'root',
55 path => "/etc/logrotate.d/$name", 55 path => "/etc/logrotate.d/$name",
56 source => $source, 56 source => $source,
57 } 57 }
58 } 58 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld