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

Issue 38873003: Reland "Dynamically pick build directory on the slave side." (Closed)

Created:
7 years, 2 months ago by Nico
Modified:
7 years, 1 month ago
Reviewers:
iannucci, M-A Ruel
CC:
chromium-reviews, cmp-cc_chromium.org, kjellander+cc_chromium.org
Visibility:
Public.

Description

Reland "Dynamically pick build directory on the slave side." This was reviewed at https://codereview.chromium.org/31543002 and is identical to what is there, but since that review landed and got reverted three times already, I'm landing it under a different review URL now. The previous attempts caused the following issues: 1. The chrome/android bots on the main waterfall got confused. r229993 / r230044 addressed this. 1b The android trybots on the webrtc waterfall. r229993 / r230044 will help them too. 2. The Blink Testers started testing binaries that weren't built at the right revision. That was due to them extracting the binaries into src/build but still running layout tests from src/out. This was http://crbug.com/310152 and should've been fixed by r230280. 3. Some internal chrome/android performance bots broke. This was http://crbug.com/310135 and got addressed by internal revisions r43584 / r43590 TBR=maruel@chromium.org, iannucci@chromium.org (who reviewed at the original review URL). Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=230605 4. It then got reverted once more due to causing stale binaries being used in letting local isolate tests (http://crbug.com/311622), which I believe I fixed in r231606 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=231610

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+46 lines, -44 lines) Patch
M build/scripts/slave/build_directory.py View 1 1 chunk +41 lines, -39 lines 0 comments Download
M build/scripts/slave/unittests/telemetry_test.py View 1 3 chunks +5 lines, -5 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thakis@chromium.org/38873003/1
7 years, 2 months ago (2013-10-24 02:25:41 UTC) #1
commit-bot: I haz the power
Presubmit check for 38873003-1 failed and returned exit status 1. Running presubmit commit checks ...
7 years, 2 months ago (2013-10-24 02:26:30 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thakis@chromium.org/38873003/50001
7 years, 2 months ago (2013-10-24 02:35:12 UTC) #3
commit-bot: I haz the power
Presubmit check for 38873003-50001 failed and returned exit status 1. Running presubmit commit checks ...
7 years, 2 months ago (2013-10-24 02:36:00 UTC) #4
Nico
Committed patchset #2 manually as r230605.
7 years, 2 months ago (2013-10-24 02:36:54 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thakis@chromium.org/38873003/50001
7 years, 1 month ago (2013-10-29 19:21:04 UTC) #6
commit-bot: I haz the power
7 years, 1 month ago (2013-10-29 19:21:55 UTC) #7
Message was sent while issue was closed.
Change committed as 231610

Powered by Google App Engine
This is Rietveld 408576698