Alexandros Kosiaris has submitted this change and it was merged.

Change subject: jenkins: logrotate main file on a daily basis
......................................................................


jenkins: logrotate main file on a daily basis

The Jenkins main log is rotated on a weekly basis. Our setup produce a
lot of logs, so it seems better to rotate on a daily basis (keeping 20
days of archives instead of 52 weeks).

The current log file for four days is 3.1 GBytes.

Also adds ensure => present to /etc/default/jenkins

Change-Id: Id6134525872f292056fd02c7fe290345d2223c14
---
A modules/jenkins/files/jenkins_log.logrotate
M modules/jenkins/manifests/init.pp
2 files changed, 21 insertions(+), 0 deletions(-)

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



diff --git a/modules/jenkins/files/jenkins_log.logrotate 
b/modules/jenkins/files/jenkins_log.logrotate
new file mode 100644
index 0000000..8e238e8
--- /dev/null
+++ b/modules/jenkins/files/jenkins_log.logrotate
@@ -0,0 +1,11 @@
+# vim: set et:
+# Override logrotate files deployed by Jenkins
+/var/log/jenkins/jenkins.log {
+    daily
+    copytruncate
+    missingok
+    rotate 20
+    compress
+    delaycompress
+    notifempty
+}
diff --git a/modules/jenkins/manifests/init.pp 
b/modules/jenkins/manifests/init.pp
index fbbe6cf..cfe0144 100644
--- a/modules/jenkins/manifests/init.pp
+++ b/modules/jenkins/manifests/init.pp
@@ -65,7 +65,17 @@
 # additionally kill -s ALRM kills jenkins instead of making it reopen
 # its files.
 
+  file { '/etc/logrotate.d/jenkins':
+    ensure  => present,
+    owner   => 'root',
+    group   => 'root',
+    mode    => '0444',
+    source  => 'puppet:///modules/jenkins/jenkins_log.logrotate',
+    require => Package['jenkins'],
+  }
+
   file { '/etc/default/jenkins':
+    ensure => present,
     owner  => 'root',
     group  => 'root',
     mode   => '0444',

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id6134525872f292056fd02c7fe290345d2223c14
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Hashar <has...@free.fr>
Gerrit-Reviewer: Alexandros Kosiaris <akosia...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to