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

Issue 866293003: Cleanup CQ configs. (Closed)

Created:
5 years, 11 months ago by Sergey Berezin
Modified:
5 years, 10 months ago
Reviewers:
Paweł Hajdan Jr.
CC:
chromium-reviews, abw
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Cleanup CQ configs. Specifically, remove triggered builders from the list of "launched" builders. The new CQ verifier will pick up triggered builders from the "triggered" section. This eliminates the duplicate specs. This CL depends on the deployment of the (Google-only) CL: https://chromereviews.googleplex.com/145077013/ R=phajdan.jr@chromium.org BUG=436774 Committed: https://crrev.com/f96a3d3aad0d070d991f770c8451351e43de0e01 Cr-Commit-Position: refs/heads/master@{#313569}

Patch Set 1 #

Patch Set 2 : Rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -2 lines) Patch
M testing/commit_queue/config.json View 1 1 chunk +1 line, -2 lines 0 comments Download

Messages

Total messages: 8 (1 generated)
Sergey Berezin
PTAL.
5 years, 11 months ago (2015-01-23 22:23:37 UTC) #1
Paweł Hajdan Jr.
LGTM
5 years, 11 months ago (2015-01-27 17:01:47 UTC) #2
Sergey Berezin
On 2015/01/27 17:01:47, Paweł Hajdan Jr. wrote: > LGTM Thanks! This depends on the internal ...
5 years, 11 months ago (2015-01-27 17:15:41 UTC) #3
Sergey Berezin
The new verifier landed and is there to stay. Landing this CL.
5 years, 10 months ago (2015-01-28 19:26:40 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/866293003/20001
5 years, 10 months ago (2015-01-28 19:32:06 UTC) #6
commit-bot: I haz the power
Committed patchset #2 (id:20001)
5 years, 10 months ago (2015-01-28 20:17:49 UTC) #7
commit-bot: I haz the power
5 years, 10 months ago (2015-01-28 20:18:30 UTC) #8
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/f96a3d3aad0d070d991f770c8451351e43de0e01
Cr-Commit-Position: refs/heads/master@{#313569}

Powered by Google App Engine
This is Rietveld 408576698