Filippo Giunchedi has uploaded a new change for review.

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

Change subject: cassandra: fix HeapDumpPath and ErrorFile settings
......................................................................

cassandra: fix HeapDumpPath and ErrorFile settings

pointing HeapDumpPath to a directory will make the jvm pick a default filename
containing the pid as well, also change ErrorFile name to its default

Bug: T116814
Change-Id: I808f8978d0550a1014cfa6156a6fef6659f2d663
---
M modules/cassandra/templates/cassandra-env.sh.erb
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/19/249419/1

diff --git a/modules/cassandra/templates/cassandra-env.sh.erb 
b/modules/cassandra/templates/cassandra-env.sh.erb
index 9b80781..931b401 100644
--- a/modules/cassandra/templates/cassandra-env.sh.erb
+++ b/modules/cassandra/templates/cassandra-env.sh.erb
@@ -201,8 +201,8 @@
 
 CASSANDRA_HEAPDUMP_DIR=<%= @heapdump_directory %>
 if [ "x$CASSANDRA_HEAPDUMP_DIR" != "x" ]; then
-    JVM_OPTS="$JVM_OPTS -XX:HeapDumpPath=$CASSANDRA_HEAPDUMP_DIR/java_%p.hprof"
-    JVM_OPTS="$JVM_OPTS -XX:ErrorFile=$CASSANDRA_HEAPDUMP_DIR/hs_err_%p.log"
+    JVM_OPTS="$JVM_OPTS -XX:HeapDumpPath=$CASSANDRA_HEAPDUMP_DIR"
+    JVM_OPTS="$JVM_OPTS -XX:ErrorFile=$CASSANDRA_HEAPDUMP_DIR/hs_err_pid%p.log"
 fi
 
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I808f8978d0550a1014cfa6156a6fef6659f2d663
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Filippo Giunchedi <[email protected]>

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

Reply via email to