mirrors/flatcar-scripts
1
0
Fork 0
mirror of https://github.com/flatcar/scripts.git synced 2025-08-08 21:46:58 +02:00
Code Issues Packages Projects Releases Wiki Activity

scripts: Make cros_generate_stacks_bvt user-friendlier.

Browse Source 
This adds support for using debug tarballs located on the
local machine instead of only ones on remote servers.  (If
the first argument is a local file, we use it; otherwise, we
try to download it.)

It also makes us download the (tiny) minidumps before the
(huge) tarball so we can bail out quickly if the minidump
URL is wrong.

BUG=chromium-os:19565
TEST=manual: ran it successfully with both local and remote tarballs and checked that it fails early when given an invalid minidump path

Change-Id: Ie427b50e4b7e37c6c81a1137eb34a28e25f32c8e
Reviewed-on: http://gerrit.chromium.org/gerrit/8910
Tested-by: Daniel Erat <derat@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: Thieu Le <thieule@chromium.org>
Commit-Ready: Daniel Erat <derat@chromium.org>
This commit is contained in:
Daniel Erat 2011-10-06 12:50:22 -07:00 committed by Gerrit
parent c801b10af3
commit 056c3f53bd
1 changed files with 29 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

47
cros_generate_stacks_bvt
View File

@ -11,12 +11,12 @@
assert_inside_chroot "$@" assert_inside_chroot "$@"
function usage() { function usage() {
echo "Usage: $0 url_to_debug_tgz url_to_bvt_test_results" echo "Usage: $0 url_or_path_to_debug_tgz url_to_bvt_test_results"
} }
if [ -z "$1" ] ; then if [ -z "$1" ] ; then
usage usage
die "The URL to symbols tarball (debug.tgz) is required" die "The URL or path to symbols tarball (debug.tgz) is required"
fi fi
if [ -z "$2" ] ; then if [ -z "$2" ] ; then
@ -28,40 +28,51 @@ fi
set -e set -e
BREAKPAD_DIR="debug/breakpad" BREAKPAD_DIR="debug/breakpad"
DEBUG_TGZ=$(basename $1)
STACKS_GENERATED="" STACKS_GENERATED=""
OUTPUT_DIR=$(mktemp -d) OUTPUT_DIR="$(mktemp -d)"
function extract_tarball() {
info "Extracting breakpad symbols from $1..."
tar zxf "$1" -C "${OUTPUT_DIR}" "${BREAKPAD_DIR}"
}
function generate_stacktrace() { function generate_stacktrace() {
echo "$1.txt" echo "$1.txt"
minidump_stackwalk $1 ${OUTPUT_DIR}/${BREAKPAD_DIR} > $1.txt 2> /dev/null minidump_stackwalk "$1" "${OUTPUT_DIR}/${BREAKPAD_DIR}" \
>"$1.txt" 2>/dev/null
} }
function find_and_generate_stacktraces() { function find_and_generate_stacktraces() {
find ${OUTPUT_DIR} -name *.dmp | find "${OUTPUT_DIR}" -name "*.dmp" |
while read filename ; do while read filename ; do
generate_stacktrace ${filename} generate_stacktrace "${filename}"
done done
} }
function cleanup() { function cleanup() {
if [ -n "${OUTPUT_DIR}" -a -z "${STACKS_GENERATED}" ] ; then if [ -n "${OUTPUT_DIR}" -a -z "${STACKS_GENERATED}" ] ; then
rm -rf ${OUTPUT_DIR} rm -rf "${OUTPUT_DIR}"
fi fi
} }
trap cleanup INT TERM EXIT trap cleanup INT TERM EXIT
info "Downloading symbols tarball..." info "Downloading minidumps from $2..."
wget -P ${OUTPUT_DIR} $1 wget -q -nv -r -np -A "*.dmp" -P "${OUTPUT_DIR}" $2
info "Extracting breakpad symbols..." if [[ -z "$(find "${OUTPUT_DIR}" -name "*.dmp")" ]] ; then
tar zxf ${OUTPUT_DIR}/${DEBUG_TGZ} -C ${OUTPUT_DIR} ${BREAKPAD_DIR} die "No minidumps found"
rm ${OUTPUT_DIR}/${DEBUG_TGZ} fi
info "Downloading minidumps from test results..."
wget -q -nv -r -np -A "*.dmp" -P ${OUTPUT_DIR} $2 if [[ -f "$1" ]] ; then
extract_tarball "$1"
else
info "Downloading symbols tarball from $1..."
wget -P "${OUTPUT_DIR}" "$1"
TARBALL="${OUTPUT_DIR}/$(basename $1)"
extract_tarball "$TARBALL"
rm -f "$TARBALL"
fi
info "Generating stack traces..." info "Generating stack traces..."
STACKS_GENERATED=$(find_and_generate_stacktraces) STACKS_GENERATED=$(find_and_generate_stacktraces)
echo $STACKS_GENERATED echo $STACKS_GENERATED
if [ -z "${STACKS_GENERATED}" ] ; then
warn "No minidumps found"
fi