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

Issue 468363002: Change the default --retry-limit for telemetry*unittests. (Closed)

Created:
6 years, 4 months ago by Dirk Pranke
Modified:
6 years, 4 months ago
Reviewers:
tonyg
CC:
chromium-reviews, telemetry+watch_chromium.org, dtu
Project:
chromium
Visibility:
Public.

Description

Change the default --retry-limit for telemetry*unittests. Previously you had to pass the flag to get failures to be retried; now we will retry failures by default if you run all of the tests (i.e., do not specify any tests explicitly on the command line). You can still pass --retry-limit=0 to turn off the retries and --retry-limit=3 (or whatever) to force them on. TBR=tonyg@chromium.org BUG=402089 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=289554

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -2 lines) Patch
M tools/telemetry/telemetry/unittest/run_tests.py View 2 chunks +7 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Dirk Pranke
this CL is split off from https://codereview.chromium.org/469533002/ ... tonyg lgtm'ed that change, so I'm TBR'ing ...
6 years, 4 months ago (2014-08-13 20:13:57 UTC) #1
Dirk Pranke
The CQ bit was checked by dpranke@chromium.org
6 years, 4 months ago (2014-08-13 20:16:44 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dpranke@chromium.org/468363002/1
6 years, 4 months ago (2014-08-13 20:17:11 UTC) #3
commit-bot: I haz the power
6 years, 4 months ago (2014-08-14 14:15:18 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 (1) as 289554

Powered by Google App Engine
This is Rietveld 408576698