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

Delta Between Two Patch Sets: modules/discourse/manifests/init.pp

Issue 29469594: #1634 - Update file resource syntax in module discourse (Closed)
Left Patch Set: Created June 20, 2017, 4:49 p.m.
Right Patch Set: For comment number 2 Created June 21, 2017, 10:07 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « no previous file | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 class discourse( 1 class discourse(
2 $domain, 2 $domain,
3 $certificate, 3 $certificate,
4 $private_key, 4 $private_key,
5 $is_default = false 5 $is_default = false
6 ) inherits private::discourse { 6 ) inherits private::discourse {
7 7
8 class { 'postgresql::globals': 8 class { 'postgresql::globals':
9 manage_package_repo => true, 9 manage_package_repo => true,
10 version => '9.3', 10 version => '9.3',
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
64 64
65 exec {'install-bundler': 65 exec {'install-bundler':
66 command => 'rvm default do gem install bundler', 66 command => 'rvm default do gem install bundler',
67 tag => 'rvm', 67 tag => 'rvm',
68 unless => 'rvm default do gem list | grep "^bundler ")', 68 unless => 'rvm default do gem list | grep "^bundler ")',
69 require => Exec['install-ruby'], 69 require => Exec['install-ruby'],
70 } 70 }
71 71
72 file {'/opt/discourse': 72 file {'/opt/discourse':
73 ensure => directory, 73 ensure => directory,
74 mode => '755', 74 mode => '0755',
mathias 2017/06/20 17:04:15 Now that we're at it, can't we make it uniform and
75 owner => discourse, 75 owner => discourse,
76 group => www-data 76 group => www-data
77 } 77 }
78 78
79 file {['/opt/discourse/tmp', '/opt/discourse/tmp/pids']: 79 file {['/opt/discourse/tmp', '/opt/discourse/tmp/pids']:
80 ensure => directory, 80 ensure => directory,
81 mode => '755', 81 mode => '0755',
82 owner => discourse, 82 owner => discourse,
83 group => www-data, 83 group => www-data,
84 require => Exec['fetch-discourse'] 84 require => Exec['fetch-discourse']
85 } 85 }
86 86
87 file {'/opt/discourse/config/discourse.conf': 87 file {'/opt/discourse/config/discourse.conf':
88 mode => '600', 88 mode => '0600',
89 owner => discourse, 89 owner => discourse,
90 group => www-data, 90 group => www-data,
91 content => template('discourse/discourse.conf.erb'), 91 content => template('discourse/discourse.conf.erb'),
92 notify => Service['discourse'], 92 notify => Service['discourse'],
93 require => Exec['fetch-discourse'] 93 require => Exec['fetch-discourse']
94 } 94 }
95 95
96 file {'/usr/local/bin/init-discourse': 96 file {'/usr/local/bin/init-discourse':
97 mode => '0755', 97 mode => '0755',
98 owner => root, 98 owner => root,
(...skipping 248 matching lines...) Expand 10 before | Expand all | Expand 10 after
347 global_config => ' 347 global_config => '
348 upstream discourse { 348 upstream discourse {
349 server localhost:3000; 349 server localhost:3000;
350 }', 350 }',
351 is_default => $is_default, 351 is_default => $is_default,
352 certificate => $certificate, 352 certificate => $certificate,
353 private_key => $private_key, 353 private_key => $private_key,
354 log => 'access_log_intraforum' 354 log => 'access_log_intraforum'
355 } 355 }
356 } 356 }
LEFTRIGHT
« no previous file | no next file » | Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Toggle Comments ('s')

Powered by Google App Engine
This is Rietveld