Dzahn has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/351703 )

Change subject: systemd: use logrotate::conf for logrotate config
......................................................................


systemd: use logrotate::conf for logrotate config

Change-Id: I9b8742cfdc70f7b4fd87d9c54c09735460510fb3
---
M modules/systemd/manifests/syslog.pp
1 file changed, 2 insertions(+), 4 deletions(-)

Approvals:
  jenkins-bot: Verified
  Dzahn: Looks good to me, approved



diff --git a/modules/systemd/manifests/syslog.pp 
b/modules/systemd/manifests/syslog.pp
index 71c109b..9a25a05 100644
--- a/modules/systemd/manifests/syslog.pp
+++ b/modules/systemd/manifests/syslog.pp
@@ -77,10 +77,8 @@
         before   => Base::Service_unit[$title],
     }
 
-    file { "/etc/logrotate.d/${title}":
+    logrotate::conf { $title:
+        ensure  => present,
         content => template('systemd/logrotate.erb'),
-        owner   => 'root',
-        group   => 'root',
-        mode    => '0444',
     }
 }

-- 
To view, visit https://gerrit.wikimedia.org/r/351703
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I9b8742cfdc70f7b4fd87d9c54c09735460510fb3
Gerrit-PatchSet: 4
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Dzahn <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: Giuseppe Lavagetto <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to