Commit graph

20 commits

Author SHA1 Message Date
Thieu Le
563d08ba7d Fix build break.
BUG=6446

TEST=unit test, logging_UserCrash

Change-Id: Iac9f5484d9218ef91ae606ef410409df5ec16382

Review URL: http://codereview.chromium.org/3380022
2010-09-21 17:07:05 -07:00
Kenneth Waters
cd71405362 Revert "Add call to logging::InitLogging() during startup to write to stderr."
This reverts commit 93cb6e57175e8e622c8f417bfad0213b125b57db.

TEST=none
BUG=none

Review URL: http://codereview.chromium.org/3390021

Change-Id: I979ddb3ab2542968e1f9a521344389bc7edf8492
2010-09-21 16:08:54 -07:00
ChromeOS Developer
9dc2fbf616 Add call to logging::InitLogging() during startup to write to stderr.
There's no need to write to a file since we only log messages for commandline parameter validations (ie. usage errors).

BUG=6446

TEST=unit test

Change-Id: I0e55d569534f953526a6b788dbc6b3e284ff8b4f

Review URL: http://codereview.chromium.org/3464008
2010-09-21 14:05:23 -07:00
Ken Mixter
da5db7a830 Increase number of enqueued crash files, sending rate, and fix minor bug.
Up to 32 crashes per spool and 32 sends per day.

BUG=6355,6782

Change-Id: I99e86265c035ad62937aa135467681be8ef70f9a

TEST=UserCrash and CrashSender

Review URL: http://codereview.chromium.org/3414006
2010-09-17 13:50:42 -07:00
Hugh Dickins
6fba658f1f Fix Unexpected crash_sender stdout/stderr when testing Official image
logging_KernelCrashServer and logging_UserCrash tests fail on an Official
image, because crash_sender's is_official() test leaks the "Official" line:
grep needs the quiet option (and does not need a cut).

BUG=837
TEST=logging_UserCrash

Signed-off-by: Hugh Dickins <hughd@chromium.org>

Review URL: http://codereview.chromium.org/3357010
2010-09-08 20:59:58 -07:00
Hugh Dickins
f174fc08dc Fix collection of kernel crash into /var/spool/crash
No kcrash was collected because the IsEnabled() test always failed:
we must do the kernel_collector->Enable() first.  I've not checked
if this might break the Override path (for local testing): it's more
important for the feature to work for real than in an artificial test.

BUG=837,5865
TEST=logging_KernelCrashServer or manual procedure

Signed-off-by: Hugh Dickins <hughd@chromium.org>

Review URL: http://codereview.chromium.org/3317007
2010-09-08 20:55:26 -07:00
Ken Mixter
023e207f13 Fix problem with crash_sender being able to run multiple times concurrently.
BUG=6134
TEST=Run CrashSender and UserCrash, also test starting those tests while crash_sender is already running.

Change-Id: Ie80c450f6b0da1b67398aa67fd9b74903abc430b

Review URL: http://codereview.chromium.org/3197024
2010-08-26 18:42:02 -07:00
Ken Mixter
04ec10fc93 Limit the number of crash reports to enqueue at once
BUG=5357

Change-Id: Ib21086cdd34c938def885d625a165ba2fa0879a4

Review URL: http://codereview.chromium.org/3209003
2010-08-26 16:02:02 -07:00
Ken Mixter
0340316050 Collect and send kernel crash diagnostics
BUG=1512,1914

Review URL: http://codereview.chromium.org/3179006
2010-08-18 15:23:16 -07:00
Ken Mixter
479b148ba7 Hard fail immediately in crash_sender if /usr/bin/find is not installed
BUGS=5778
TEST=BVTs

Review URL: http://codereview.chromium.org/3122024
2010-08-18 14:02:01 -07:00
Ken Mixter
7ac7a70a59 Move leave_core where similar files exist
BUG=5696

Review URL: http://codereview.chromium.org/3121015
2010-08-13 15:43:34 -07:00
Chris Masone
f9322f734f Update to new libchrome APIs
BUG=none
TEST=compile

Change-Id: Ie494a6536665b635b82725dd6a82959974b02a75

Review URL: http://codereview.chromium.org/3138010
2010-08-11 19:56:51 -07:00
J. Richard Barnette
0e6037cfc1 Add LICENSE file 2010-08-05 14:13:48 -07:00
Ken Mixter
777484c41c Start invoking core2md to implement full system crash reporting
BUG=4741

Review URL: http://codereview.chromium.org/3040013
2010-07-23 16:22:44 -07:00
Ken Mixter
6993256118 Fix crash-reporter test build
Review URL: http://codereview.chromium.org/2944003
2010-07-09 12:37:59 -07:00
Ken Mixter
091993c72b Send client ID in crash report
BUG=4110,988

Review URL: http://codereview.chromium.org/2849038
2010-07-01 15:09:24 -07:00
Darin Petkov
c7f8528cdc Notify metrics daemon of kernel crashes.
BUG=none
TEST=unit tests, ran crash_reporter --init on the device

Review URL: http://codereview.chromium.org/2847017
2010-06-24 11:43:12 -07:00
Chris Sosa
522c061f83 Change include paths to reflect new directory.
TEST=Tested with new ebuild.

Review URL: http://codereview.chromium.org/2819009
2010-06-18 11:08:11 -07:00
Chris Sosa
69e37209b3 TBR: Adding petkov's patch committed today in other repo 2010-06-16 17:55:31 -07:00
Chris Sosa
e4a86037f4 Initial commit of crash reporter repo 2010-06-16 17:08:34 -07:00