New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 685321 link

Starred by 5 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug

Blocked on:
issue 642484
issue 746079

Blocking:
issue 600469



Sign in to add a comment

Migrate Chromium to PolyGerrit

Project Member Reported by aga...@chromium.org, Jan 25 2017

Issue description

Repos:
https://chromium.googlesource.com/chromium/src/+/refs/meta/config/project.config

Point of contact:
agable@

Lists:
chrome-infrastructure-announce@
infra-announce@chromium.org
chromium-dev@chromium.org

Subdirectories:
blimp/
cc/
chrome/browser/chromeos/net/
chrome/browser/media/
chrome/test/chromedriver/
components/arc/net/
content/browser/download/
media/remoting/
mojo/
net/http/
third_party/WebKit/

Expected date of first round of PSAs and enabling Gerrit:
Tuesday, Jan 31 (Pacific)

Expected date of full migration:
Unknown, depends on feedback
 

Comment 1 by aga...@chromium.org, Jan 25 2017

Components: Infra>Codereview>Gerrit

Comment 2 by aga...@chromium.org, Feb 22 2017

Status: Started (was: Assigned)
Now that gnumbd and cq are taken care of, I'm quietly making all the config changes for this to go live. Once that's done, will slowly introduce a few users/teams and go from there.
Project Member

Comment 4 by bugdroid1@chromium.org, Feb 23 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/19b52c097bfeb38304d4243fea5eff8aeb3140c9

commit 19b52c097bfeb38304d4243fea5eff8aeb3140c9
Author: Aaron Gable <agable@chromium.org>
Date: Thu Feb 23 00:52:30 2017

Enable gerrit cq for chromium/src

BUG= 685321 

Change-Id: I020b72b3c6a3ccd4170327a70386fcad38d57f0f
Reviewed-on: https://chromium-review.googlesource.com/446522
Reviewed-by: Nodir Turakulov <nodir@chromium.org>
Cr-Commit-Position: refs/heads/master@{#452316}
[modify] https://crrev.com/19b52c097bfeb38304d4243fea5eff8aeb3140c9/infra/config/cq.cfg

Project Member

Comment 5 by bugdroid1@chromium.org, Feb 23 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b69969a47c841e67206fe1cc94213daf15830549

commit b69969a47c841e67206fe1cc94213daf15830549
Author: Aaron Gable <agable@chromium.org>
Date: Thu Feb 23 01:23:41 2017

Whitespace change to test the Gerrit CQ

Bug:  685321 
Change-Id: I6dfc239d3a3166bad71977b4c6ca0af16c5a500f
Reviewed-on: https://chromium-review.googlesource.com/446697
Commit-Queue: Aaron Gable <agable@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
Cr-Commit-Position: refs/heads/master@{#452327}
[modify] https://crrev.com/b69969a47c841e67206fe1cc94213daf15830549/build/whitespace_file.txt

Comment 6 by aga...@chromium.org, Feb 23 2017

And it looks like we're ready to begin dogfood.

Comment 7 by rmis...@google.com, Feb 23 2017

"And it looks like we're ready to begin dogfood."
Great! When is it going to be announced to the many lists? (or has it already been announced and I missed it?).

Comment 8 by aga...@chromium.org, Feb 23 2017

Hasn't been announced. As I mentioned in Comment 2 above, I plan to slowly introduce individuals and small subteams without making a full chromium-dev@ announcement at first.

Ideally we'd announce it widely next week. Unfortunately there are still a lot of bugs listed as Dogfood blockers that I need to go through and re-triage to make sure we're okay announcing before they're fixed. Doubly unfortunately, I'm on vacation all week the week after next, and announcing followed by immediately ghosting doesn't seem like a great plan :)

Long story short, you'll definitely know when we make the big announcement, but I can't give you a hard date of when that will be.

Comment 9 by aga...@chromium.org, Mar 27 2017

Labels: -Milestone-Dogfood Milestone-Launch
This has obviously been delayed from the timeline posted above. We believe that all Dogfood milestone bugs on the PolyGerrit side will be closed this week, so the big announcement will also be this week. Ideally.
Project Member

Comment 10 by bugdroid1@chromium.org, May 22 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/depot_tools/+/c4c40d1b5a9a142a50e8b0a1afb24fa2ec05f9e8

commit c4c40d1b5a9a142a50e8b0a1afb24fa2ec05f9e8
Author: Aaron Gable <agable@chromium.org>
Date: Mon May 22 19:39:03 2017

git-cl: Print Rietveld deprecation warning

To encourage folks to upload to Gerrit.

Bug:  685321 
Change-Id: I849ed53f95a2205959237fe825ffe41e349ff8c7
Reviewed-on: https://chromium-review.googlesource.com/510963
Reviewed-by: Andrew Bonventre <andybons@chromium.org>
Commit-Queue: Aaron Gable <agable@chromium.org>

[modify] https://crrev.com/c4c40d1b5a9a142a50e8b0a1afb24fa2ec05f9e8/tests/git_cl_test.py
[modify] https://crrev.com/c4c40d1b5a9a142a50e8b0a1afb24fa2ec05f9e8/git_cl.py

Project Member

Comment 11 by bugdroid1@chromium.org, Jul 6 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ccfbae89e4c283274f59821d3ca5d252f746b33a

commit ccfbae89e4c283274f59821d3ca5d252f746b33a
Author: Aaron Gable <agable@chromium.org>
Date: Thu Jul 06 18:04:08 2017

Make Gerrit the default codereview system

Bug:  685321 
Change-Id: I75d180634a4f41faa15135bbfeb56c5cd25f701a
Reviewed-on: https://chromium-review.googlesource.com/541675
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Aaron Gable <agable@chromium.org>
Cr-Commit-Position: refs/heads/master@{#484666}
[modify] https://crrev.com/ccfbae89e4c283274f59821d3ca5d252f746b33a/codereview.settings

Comment 12 by kbr@chromium.org, Jul 19 2017

Blockedon: 746079
Project Member

Comment 13 by bugdroid1@chromium.org, Jul 24 2017

Labels: merge-merged-3071
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ed998ea209fc0637e4db5e96369d4998ab3503b5

commit ed998ea209fc0637e4db5e96369d4998ab3503b5
Author: Aaron Gable <agable@chromium.org>
Date: Mon Jul 24 22:02:34 2017

Make Gerrit the default codereview system

(cherry picked from commit ccfbae89e4c283274f59821d3ca5d252f746b33a)

Bug:  685321 
Change-Id: I75d180634a4f41faa15135bbfeb56c5cd25f701a
Reviewed-on: https://chromium-review.googlesource.com/583527
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Cr-Commit-Position: refs/branch-heads/3071@{#846}
Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641}
[modify] https://crrev.com/ed998ea209fc0637e4db5e96369d4998ab3503b5/codereview.settings

Project Member

Comment 14 by bugdroid1@chromium.org, Jul 24 2017

Labels: merge-merged-3112
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bf323a36a5f13cafab77f38bfac15d1ec57a1b75

commit bf323a36a5f13cafab77f38bfac15d1ec57a1b75
Author: Aaron Gable <agable@chromium.org>
Date: Mon Jul 24 22:09:58 2017

Make Gerrit the default codereview system

(cherry picked from commit ccfbae89e4c283274f59821d3ca5d252f746b33a)

Bug:  685321 
Change-Id: I75d180634a4f41faa15135bbfeb56c5cd25f701a
Reviewed-on: https://chromium-review.googlesource.com/583528
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Cr-Commit-Position: refs/branch-heads/3112@{#669}
Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}
[modify] https://crrev.com/bf323a36a5f13cafab77f38bfac15d1ec57a1b75/codereview.settings

Cc: -tandrii@chromium.org
Blocking: 600469
Project Member

Comment 17 by bugdroid1@chromium.org, Jul 27 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/3674da8d7a2b89c2d36e5929d8031b8876dc558d

commit 3674da8d7a2b89c2d36e5929d8031b8876dc558d
Author: Aaron Gable <agable@chromium.org>
Date: Thu Jul 27 18:26:14 2017

Rietveld: refuse new changes to chromium and v8

Bug:  685318 , 685321 
Change-Id: I947abe065151e05debf3f624dc173ab224293e5f
Reviewed-on: https://chromium-review.googlesource.com/589813
Reviewed-by: Katie Thomas <katthomas@google.com>
Commit-Queue: Aaron Gable <agable@chromium.org>

[modify] https://crrev.com/3674da8d7a2b89c2d36e5929d8031b8876dc558d/appengine/chromium_rietveld/settings.py

Status: Fixed (was: Started)

Sign in to add a comment