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

Issue 344123002: Download archives contain git or svn revisions in the filename. (Closed)

Created:
6 years, 6 months ago by prasadv
Modified:
6 years, 6 months ago
Reviewers:
tonyg, sh
CC:
chromium-reviews
Project:
chromium
Visibility:
Public.

Description

Download archives contain git or svn revisions in the filename. Due to recent Git mirgration on bots, builds are archived using Git hash in filename instead of SVN revision. Bisects should check for builds either with Git hash or SVN revisions and download. Also this CL allows bisect to continue when a revision under test fails to build (except good or bad revisions). BUG=385734, 385356, 380304 NOTRY=true Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=278767

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+91 lines, -51 lines) Patch
M tools/bisect-perf-regression.py View 11 chunks +91 lines, -51 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
prasadv
6 years, 6 months ago (2014-06-20 00:12:41 UTC) #1
tonyg
lgtm
6 years, 6 months ago (2014-06-20 16:55:35 UTC) #2
prasadv
The CQ bit was checked by prasadv@chromium.org
6 years, 6 months ago (2014-06-20 17:11:48 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/prasadv@chromium.org/344123002/1
6 years, 6 months ago (2014-06-20 17:12:52 UTC) #4
prasadv
The CQ bit was unchecked by prasadv@chromium.org
6 years, 6 months ago (2014-06-20 18:07:32 UTC) #5
prasadv
The CQ bit was checked by prasadv@chromium.org
6 years, 6 months ago (2014-06-20 18:07:50 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/prasadv@chromium.org/344123002/1
6 years, 6 months ago (2014-06-20 18:09:13 UTC) #7
commit-bot: I haz the power
6 years, 6 months ago (2014-06-20 18:13:03 UTC) #8
Message was sent while issue was closed.
Change committed as 278767

Powered by Google App Engine
This is Rietveld 408576698