Alexandros Kosiaris has submitted this change and it was merged.
Change subject: Specify owner/group/mode for apache::conf
......................................................................
Specify owner/group/mode for apache::conf
Or else files end up with 664 and whatever uid/gid corresponds to the
uid the file has on puppetmaster (or defaults if those exist - but we
don't have any)
Change-Id: I4a79ace8e5c158b5c4e0039b8ede58dc0fe9c0e9
---
M modules/apache/manifests/conf.pp
1 file changed, 3 insertions(+), 0 deletions(-)
Approvals:
Ori.livneh: Looks good to me, but someone else must approve
Alexandros Kosiaris: Looks good to me, approved
jenkins-bot: Verified
Dzahn: Looks good to me, but someone else must approve
diff --git a/modules/apache/manifests/conf.pp b/modules/apache/manifests/conf.pp
index 5ed4209..d813bdc 100644
--- a/modules/apache/manifests/conf.pp
+++ b/modules/apache/manifests/conf.pp
@@ -58,6 +58,9 @@
ensure => $ensure,
content => $content,
source => $source,
+ owner => 'root',
+ group => 'root',
+ mode => '0444',
}
file { "/etc/apache2/${conf_type}-enabled/${conf_file}":
--
To view, visit https://gerrit.wikimedia.org/r/148652
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I4a79ace8e5c158b5c4e0039b8ede58dc0fe9c0e9
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Alexandros Kosiaris <[email protected]>
Gerrit-Reviewer: Alexandros Kosiaris <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: Giuseppe Lavagetto <[email protected]>
Gerrit-Reviewer: Ori.livneh <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
