Yuvipanda has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/173484

Change subject: base: Allow overriding core dump path via hiera
......................................................................

base: Allow overriding core dump path via hiera

Change-Id: I1d209df535a0387ba88c58c84bce72e8c4e175d4
Phab: T1259
---
M modules/base/manifests/environment.pp
1 file changed, 5 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/84/173484/1

diff --git a/modules/base/manifests/environment.pp 
b/modules/base/manifests/environment.pp
index 7edd8d5..79f5e20 100644
--- a/modules/base/manifests/environment.pp
+++ b/modules/base/manifests/environment.pp
@@ -82,6 +82,9 @@
 
     # Write core dumps to 
/var/tmp/core/core.<host>.<executable>.<pid>.<timestamp>.
     # Remove core dumps with atime > one week.
+    # Allow overrides of the core dump path with hiera, so places
+    # where we want core dumps to be elsewhere (CWD, etc) can
+    # still have that.
 
     file { '/var/tmp/core':
         ensure => directory,
@@ -90,8 +93,9 @@
         mode   => '1773',
     }
 
+    $core_dump_pattern = hiera('core_dump_pattern', 
'/var/tmp/core/core.%h.%e.%p.%t')
     sysctl::parameters { 'core_dumps':
-        values  => { 'kernel.core_pattern' => 
'/var/tmp/core/core.%h.%e.%p.%t', },
+        values  => { 'kernel.core_pattern' => $core_dump_pattern, },
         require => File['/var/tmp/core'],
     }
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1d209df535a0387ba88c58c84bce72e8c4e175d4
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Yuvipanda <yuvipa...@gmail.com>

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

Reply via email to