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

Side by Side Diff: modules/base/manifests/init.pp

Issue 9351014: Add logrotate cronjob to all servers, don`t run logrotate unnecessarily on provisioning and use con… (Closed)
Patch Set: Created Feb. 7, 2013, 3:06 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 base { 1 class base {
2 stage {'pre': before => Stage['main']} 2 stage {'pre': before => Stage['main']}
3 stage {'post': }
4 Stage['main'] -> Stage['post']
Felix Dahlke 2013/02/07 15:38:41 I'd like this to be consistent. So either this: {
3 5
4 class {'apt': 6 class {'apt':
5 stage => 'pre', 7 stage => 'pre',
6 always_apt_update => true 8 always_apt_update => true
7 } 9 }
8 10
9 class {'users': 11 class {'users':
10 stage => 'pre', 12 stage => 'pre',
11 } 13 }
12 14
13 include private::users, postfix 15 include private::users, postfix
14 16
15 package {['mercurial', 'vim', 'emacs']: ensure => present} 17 package {['mercurial', 'vim', 'emacs']: ensure => present}
16 18
17 file {'/etc/timezone': 19 file {'/etc/timezone':
18 ensure => file, 20 ensure => file,
19 owner => root, 21 owner => root,
20 group => root, 22 group => root,
21 mode => 0644, 23 mode => 0644,
22 source => 'puppet:///modules/base/timezone' 24 source => 'puppet:///modules/base/timezone'
23 } 25 }
26
27 class {'logrotate':
28 stage => 'post'
29 }
24 } 30 }
OLDNEW
« no previous file with comments | « no previous file | modules/filterserver/files/logrotate » ('j') | modules/logrotate/manifests/init.pp » ('J')

Powered by Google App Engine
This is Rietveld