Ottomata has submitted this change and it was merged.

Change subject: Move Eventlogging logs underneath /srv, which has more free 
space
......................................................................


Move Eventlogging logs underneath /srv, which has more free space

vanadium is seeing daily alerts about the disk space running low on
its root partition [1].
EventLogging writes its logs to the vanadium's root partition.
Since the EventLogging logs have been dialed back from retaining 90
days of data to 45 day some time back (bug 69029), and vanadium's /srv
mount has plenty of free space left anyways [2], it's about time to
solve the spcae problem for good and we hence now write EventLogging
logs into a directory underneath /srv.

[1] See for example 06:26:04 on
  http://bots.wmflabs.org/~wm-bot/logs/%23wikimedia-operations/20141111.txt

[2] Currently, /srv has 250GB free. All of EventLogging logs on
vanadium currently take ~70GB.

RT: 8063
Change-Id: I41f628acdad7506f0c6813648ce3b85cf216cb4b
---
M modules/eventlogging/manifests/init.pp
1 file changed, 13 insertions(+), 1 deletion(-)

Approvals:
  Ori.livneh: Looks good to me, but someone else must approve
  Ottomata: Verified; Looks good to me, approved



diff --git a/modules/eventlogging/manifests/init.pp 
b/modules/eventlogging/manifests/init.pp
index a4c3234..a4c0f72 100644
--- a/modules/eventlogging/manifests/init.pp
+++ b/modules/eventlogging/manifests/init.pp
@@ -42,7 +42,19 @@
     include ::eventlogging::package
     include ::eventlogging::monitoring
 
-    $log_dir = '/var/log/eventlogging'
+    $log_dir = '/srv/log/eventlogging'
+
+    # We ensure the /srv/log (parent of $log_dir) manually here, as
+    # there is no proper class to rely on for this, and starting a
+    # separate would be an overkill for now.
+    if !defined(File['/srv/log']) {
+        file { '/srv/log':
+            ensure  => 'directory',
+            mode    => '755',
+            owner   => 'root',
+            group   => 'root',
+        }
+    }
 
     group { 'eventlogging':
         ensure => present,

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I41f628acdad7506f0c6813648ce3b85cf216cb4b
Gerrit-PatchSet: 5
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: QChris <[email protected]>
Gerrit-Reviewer: Ori.livneh <[email protected]>
Gerrit-Reviewer: Ottomata <[email protected]>
Gerrit-Reviewer: QChris <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to