Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(292)

Issue 3919001: Ignore sysinfo pre/post abortions. (Closed)

Created:
10 years, 2 months ago by DaleCurtis
Modified:
9 years, 7 months ago
Reviewers:
ericli
CC:
chromium-os-reviews_chromium.org, sosa+cc_chromium.org, seano+cc_chromium.org, ericli, petkov+cc_chromium.org
Visibility:
Public.

Description

We're encountering a significant number of unexpected aborts in the pre/post-test and pre/post-iteration sysinfo collection. After a week of debugging I'm no closer to determining the underlying cause. In the mean time, I've prepared this CL to prevent Autotest from recording a job as aborted if the abort happens in the pre/post-test or pre/post-iteration sysinfo collection phase. Change-Id: I2082415ce85e73b469934f7ac39035ed490e98d7 BUG=7931 TEST=Running in production. Observe bug does not trigger abort. Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=f9d7d8d

Patch Set 1 #

Patch Set 2 : Removed typo. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -12 lines) Patch
M server/autotest.py View 1 9 chunks +14 lines, -8 lines 0 comments Download
M server/test.py View 4 chunks +4 lines, -4 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
DaleCurtis
PTAL
10 years, 2 months ago (2010-10-19 22:39:37 UTC) #1
ericli
10 years, 2 months ago (2010-10-19 23:04:06 UTC) #2
LGTM.

Powered by Google App Engine
This is Rietveld 408576698