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

Issue 3003523002: Add commit queue settings to Dart (Closed)

Created:
3 years, 4 months ago by Bill Hesse
Modified:
3 years, 4 months ago
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Add commit queue settings to Dart These will be used on Gerrit reviews, to trigger try bots and the commit queue. The commit queue is currently set up to commit to GitHub. This will be changed when the mirroring direction changes. BUG=737149 R=athom@google.com Committed: https://github.com/dart-lang/sdk/commit/2149487c0ddecb4661b42ec25ed6ee12ea994c32

Patch Set 1 #

Patch Set 2 : Restore cq status app url #

Patch Set 3 : Sort OWNERS #

Patch Set 4 : Add isolate files #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+62 lines, -9 lines) Patch
A tools/bots/dart2js_tests.isolate View 1 2 3 1 chunk +22 lines, -0 lines 0 comments Download
A tools/bots/dart_tests.isolate View 1 2 3 1 chunk +26 lines, -0 lines 0 comments Download
M tools/infra/config/OWNERS View 1 2 1 chunk +4 lines, -1 line 0 comments Download
M tools/infra/config/cq.cfg View 1 2 3 2 chunks +10 lines, -8 lines 2 comments Download

Messages

Total messages: 9 (3 generated)
Bill Hesse
3 years, 4 months ago (2017-08-21 12:49:57 UTC) #2
Alexander Thomas
lgtm
3 years, 4 months ago (2017-08-21 13:05:15 UTC) #3
Bill Hesse
Committed patchset #4 (id:60001) manually as 2149487c0ddecb4661b42ec25ed6ee12ea994c32 (presubmit successful).
3 years, 4 months ago (2017-08-21 13:36:19 UTC) #5
tandrii(chromium)
https://codereview.chromium.org/3003523002/diff/60001/tools/infra/config/cq.cfg File tools/infra/config/cq.cfg (right): https://codereview.chromium.org/3003523002/diff/60001/tools/infra/config/cq.cfg#newcode6 tools/infra/config/cq.cfg:6: git_repo_url: "https://github.com/dart-lang/sdk.git" this won't work. This is gerrit CQ, ...
3 years, 4 months ago (2017-08-21 15:24:06 UTC) #7
Bill Hesse
https://codereview.chromium.org/3003523002/diff/60001/tools/infra/config/cq.cfg File tools/infra/config/cq.cfg (right): https://codereview.chromium.org/3003523002/diff/60001/tools/infra/config/cq.cfg#newcode6 tools/infra/config/cq.cfg:6: git_repo_url: "https://github.com/dart-lang/sdk.git" On 2017/08/21 15:24:06, tandrii(chromium) wrote: > this ...
3 years, 4 months ago (2017-08-21 15:26:40 UTC) #8
tandrii(chromium)
3 years, 4 months ago (2017-08-21 15:33:57 UTC) #9
Message was sent while issue was closed.
On 2017/08/21 15:26:40, Bill Hesse wrote:
>
https://codereview.chromium.org/3003523002/diff/60001/tools/infra/config/cq.cfg
> File tools/infra/config/cq.cfg (right):
> 
>
https://codereview.chromium.org/3003523002/diff/60001/tools/infra/config/cq.c...
> tools/infra/config/cq.cfg:6: git_repo_url:
> "https://github.com/dart-lang/sdk.git"
> On 2017/08/21 15:24:06, tandrii(chromium) wrote:
> > this won't work. This is gerrit CQ, not a Rietveld one, and Gerrit CQ needs
a
> > Gerrit host here.
> 
> OK, I saw in an earlier post on the gob->github sync issue, and in the
> go/we-need-cq, that groups were managing to make cq commit to github from
their
> gerrit reviews.  Is there no way to make Gerrit commit to Github?

There might be, but i'm not aware of one. In any case, Gerrit CQ will use Gerrit
Submit API to piggy-back on Gerrit security.
Rietveld CQ did "git push" and thus could work with almost any git host offered
under https://.

Powered by Google App Engine
This is Rietveld 408576698