You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by up...@apache.org on 2018/08/22 22:05:20 UTC

[geode] branch develop updated: GEODE-5615: Capturing more data in the case of hangs or OOMEs

This is an automated email from the ASF dual-hosted git repository.

upthewaterspout pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/develop by this push:
     new 5602d02  GEODE-5615: Capturing more data in the case of hangs or OOMEs
5602d02 is described below

commit 5602d02800e694afaaa08230e83df4d5c4bffb2e
Author: Dan Smith <ds...@pivotal.io>
AuthorDate: Wed Aug 22 15:05:14 2018 -0700

    GEODE-5615: Capturing more data in the case of hangs or OOMEs
    
    To help us track down builds that fail with OutOfMemory errors, capture
    any hprof files found.
    
    Capturing call stacks for all java processes
    
    We were only capturing the call stacks for ChildVM processes, which
    doesn't include the main test JVM.
---
 ci/scripts/archive_results.sh     | 1 +
 ci/scripts/capture-call-stacks.sh | 4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/ci/scripts/archive_results.sh b/ci/scripts/archive_results.sh
index 58ed6b6..be6116d 100755
--- a/ci/scripts/archive_results.sh
+++ b/ci/scripts/archive_results.sh
@@ -101,6 +101,7 @@ pushd ${GEODE_BUILD}
   find . -type d -name "test-results" >> ${directories_file}
   (find . -type d -name "*Test" | grep "build/[^/]*Test$") >> ${directories_file}
   find . -name "*-progress*txt" >> ${directories_file}
+  find . -name "*.hprof" >> ${directories_file}
   find . -type d -name "callstacks" >> ${directories_file}
   echo "Collecting the following artifacts..."
   cat ${directories_file}
diff --git a/ci/scripts/capture-call-stacks.sh b/ci/scripts/capture-call-stacks.sh
index d162540..af4700b 100755
--- a/ci/scripts/capture-call-stacks.sh
+++ b/ci/scripts/capture-call-stacks.sh
@@ -64,7 +64,7 @@ for (( h=0; h<${COUNT}; h++)); do
 
         for (( i=0; i<${#containers[@]}; i++ )); do
             echo "Container: ${containers[i]}" | tee -a ${logfile};
-            mapfile -t processes < <(docker exec ${containers[i]} jps | grep ChildVM | cut -d ' ' -f 1)
+            mapfile -t processes < <(docker exec ${containers[i]} jps | cut -d ' ' -f 1)
             echo "Got past processes."
             for ((j=0; j<${#processes[@]}; j++ )); do
                   echo "********* Dumping stack for process ${processes[j]}:" | tee -a ${logfile}
@@ -72,7 +72,7 @@ for (( h=0; h<${COUNT}; h++)); do
             done
         done
     else
-        mapfile -t processes < <(jps | grep ChildVM | cut -d ' ' -f 1)
+        mapfile -t processes < <(jps | cut -d ' ' -f 1)
         echo "Got past processes."
         for ((j=0; j<${#processes[@]}; j++ )); do
               echo "********* Dumping stack for process ${processes[j]}:" | tee -a ${logfile}