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

Unified Diff: modules/adblockplus/manifests/init.pp

Issue 29469584: #1634 - Update file resource syntax in module adblockplus (Closed)
Patch Set: Created June 20, 2017, 4:42 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | modules/adblockplus/manifests/log.pp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: modules/adblockplus/manifests/init.pp
===================================================================
--- a/modules/adblockplus/manifests/init.pp
+++ b/modules/adblockplus/manifests/init.pp
@@ -60,7 +60,7 @@
# managed via Puppet, but accessible by all users with access to the system
@file {$directory:
ensure => 'directory',
- mode => 0755,
+ mode => '0755',
owner => 'root',
}
@@ -71,7 +71,7 @@
content => 'UTC',
ensure => 'present',
group => 'root',
- mode => 0644,
+ mode => '0644',
notify => Service['cron'],
owner => 'root';
'/etc/localtime':
@@ -94,7 +94,7 @@
content => template('adblockplus/crontab.erb'),
owner => 'root',
group => 'root',
- mode => 0644,
+ mode => '0644',
}
# Work around https://issues.adblockplus.org/ticket/3479
@@ -103,11 +103,11 @@
file {
'/etc/ssh/ssh_host_rsa_key':
source => 'puppet:///modules/adblockplus/development_host_rsa_key',
- mode => 600,
+ mode => '600',
mathias 2017/06/20 17:09:02 Now that we're at it, can't we make it uniform and
notify => Service['ssh'];
'/etc/ssh/ssh_host_rsa_key.pub':
source => 'puppet:///modules/adblockplus/development_host_rsa_key.pub',
- mode => 644;
+ mode => '644';
}
}
@@ -120,7 +120,7 @@
# https://projects.puppetlabs.com/issues/4145
ensure_resource('file', '/etc/ssh/ssh_known_hosts', {
ensure => 'present',
- mode => 0644,
+ mode => '0644',
})
# See modules/adblockplus/manifests/host.pp
« no previous file with comments | « no previous file | modules/adblockplus/manifests/log.pp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld