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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2457423002: WebRTC: Add Android more configs bots. (Closed)

Created:
4 years, 1 month ago by kjellander_chromium
Modified:
4 years, 1 month ago
CC:
chromium-reviews, infra-reviews+build_chromium.org, kjellander-cc_chromium.org
Target Ref:
refs/heads/master
Project:
build
Visibility:
Public.

Description

WebRTC: Add Android more configs bots. Adding similar bots as in https://codereview.chromium.org/2460493002 but for Android. BUG=652197, 611054 Committed: https://chromium.googlesource.com/chromium/tools/build/+/1f79efc64c710df6a559505304c0c0998a22d31c

Patch Set 1 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+60 lines, -22 lines) Patch
M masters/master.client.webrtc/master_android_cfg.py View 2 chunks +2 lines, -0 lines 0 comments Download
M masters/master.client.webrtc/slaves.cfg View 1 chunk +1 line, -0 lines 0 comments Download
M masters/master.tryserver.webrtc/master.cfg View 1 chunk +1 line, -0 lines 0 comments Download
M masters/master.tryserver.webrtc/slaves.cfg View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/webrtc/more_configs.py View 3 chunks +27 lines, -0 lines 0 comments Download
A + scripts/slave/recipes/webrtc/more_configs.expected/full_client_webrtc_Android32__more_configs_.json View 8 chunks +14 lines, -11 lines 0 comments Download
A + scripts/slave/recipes/webrtc/more_configs.expected/full_tryserver_webrtc_android_more_configs.json View 8 chunks +14 lines, -11 lines 0 comments Download

Messages

Total messages: 14 (9 generated)
kjellander_chromium
4 years, 1 month ago (2016-10-31 07:25:30 UTC) #6
ehmaldonado_chromium
On 2016/10/31 07:25:30, kjellander_chromium wrote: LGTM
4 years, 1 month ago (2016-10-31 11:52:54 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2457423002/40001
4 years, 1 month ago (2016-10-31 11:54:59 UTC) #11
commit-bot: I haz the power
Committed patchset #1 (id:40001) as https://chromium.googlesource.com/chromium/tools/build/+/1f79efc64c710df6a559505304c0c0998a22d31c
4 years, 1 month ago (2016-10-31 12:07:05 UTC) #13
kjellander_chromium
4 years, 1 month ago (2016-10-31 13:31:12 UTC) #14
Message was sent while issue was closed. 
On 2016/10/31 12:07:05, commit-bot: I haz the power wrote:
> Committed patchset #1 (id:40001) as
>
https://chromium.googlesource.com/chromium/tools/build/+/1f79efc64c710df6a559...

This CL was missing the more_configs recipe configuration for the builders.
Follow-up CL: https://codereview.chromium.org/2466543002/

Powered by Google App Engine
This is Rietveld 408576698