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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 714373002: Remove old Gerrit instances. (Closed)

Created:
6 years, 1 month ago by Jorge Lucangeli Obes
Modified:
6 years, 1 month ago
Reviewers:
cmp, M-A Ruel
CC:
chromium-reviews, Dirk Pranke, cmp-cc_chromium.org, iannucci+depot_tools_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/tools/depot_tools.git@master
Project:
tools
Visibility:
Public.

Description

Remove old Gerrit instances. BUG=None TEST=my_activity.py still works. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=292935

Patch Set 1 #

Total comments: 2

Patch Set 2 : Only remove gerrit-int. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -4 lines) Patch
M my_activity.py View 1 1 chunk +0 lines, -4 lines 0 comments Download

Messages

Total messages: 9 (2 generated)
Jorge Lucangeli Obes
Title says it all.
6 years, 1 month ago (2014-11-11 23:10:05 UTC) #2
M-A Ruel
https://chromiumcodereview.appspot.com/714373002/diff/1/my_activity.py File my_activity.py (left): https://chromiumcodereview.appspot.com/714373002/diff/1/my_activity.py#oldcode91 my_activity.py:91: 'host': 'gerrit.chromium.org', It's still used by webm and ffpmeg.
6 years, 1 month ago (2014-11-12 01:08:58 UTC) #3
Jorge Lucangeli Obes
https://chromiumcodereview.appspot.com/714373002/diff/1/my_activity.py File my_activity.py (left): https://chromiumcodereview.appspot.com/714373002/diff/1/my_activity.py#oldcode91 my_activity.py:91: 'host': 'gerrit.chromium.org', On 2014/11/12 01:08:58, M-A Ruel wrote: > ...
6 years, 1 month ago (2014-11-12 18:12:57 UTC) #4
Jorge Lucangeli Obes
PTAL
6 years, 1 month ago (2014-11-12 18:13:23 UTC) #5
M-A Ruel
lgtm Sorry for the delay.
6 years, 1 month ago (2014-11-14 02:01:05 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/714373002/20001
6 years, 1 month ago (2014-11-14 02:01:33 UTC) #8
commit-bot: I haz the power
6 years, 1 month ago (2014-11-14 02:03:47 UTC) #9
Message was sent while issue was closed. 
Committed patchset #2 (id:20001) as 292935

Powered by Google App Engine
This is Rietveld 408576698