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

Issue 633846 link

Starred by 2 users
Status: Verified
Owner:
█ emso@chromium.org
Last visit > 30 days ago
Closed: Sep 2017
Cc:
qyears...@chromium.org
█ emso@chromium.org
mar...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Feature

Blocked on:
issue 638493
issue 638495
Blocking:
issue 625781



Sign in to add a comment

Tricium Gerrit Integration

Project Member Reported by emso@chromium.org, Aug 3 2016 
Gerrit Intergration for Tricium. This includes a Gerrit event listener, which typically is a poller, and support for reporting of analyzer results in Gerrit.

Gerrit event listener: this is typically a poller but could also be a subscriber to a PubSub if available.

Analyzer results in Gerrit: this means support for display of robot comments (aka. Tricium analyzer results) in a way that separates them from human comments, support for display and application of fixes, and support for collection of feedback.

Comment 1 by emso@chromium.org, Aug 3 2016

Blocking: 625781

Comment 2 by emso@chromium.org, Aug 3 2016

Labels: -Type-Bug Type-Feature

Comment 3 by emso@chromium.org, Aug 17 2016

Blockedon: 638493

Comment 4 by emso@chromium.org, Aug 17 2016

Blockedon: 638495
Project Member

Comment 5 by bugdroid1@chromium.org, Oct 6 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra.git/+/310594fcf464926915928262c60aebf4dfc67fd6

commit 310594fcf464926915928262c60aebf4dfc67fd6
Author: Emma Soederberg <emso@chromium.org>
Date: Thu Oct 06 09:56:30 2016

Adds dep to golang.org/x/build.

This library provides a go Gerrit client and will be used by the Tricium Gerrit poller.

BUG= 633846 

Change-Id: I00683b6fd261810bfb175f744fcc64fe6cab311c
Reviewed-on: https://chromium-review.googlesource.com/394586
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

[modify] https://crrev.com/310594fcf464926915928262c60aebf4dfc67fd6/go/deps.lock
[modify] https://crrev.com/310594fcf464926915928262c60aebf4dfc67fd6/go/deps.yaml
Project Member

Comment 7 by bugdroid1@chromium.org, Nov 2 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra.git/+/3caa93063acc0aa9ef59bd1741dfd306161359a9

commit 3caa93063acc0aa9ef59bd1741dfd306161359a9
Author: Emma Soederberg <emso@chromium.org>
Date: Wed Nov 02 10:25:40 2016

Adds cron job to Gerrit poller

BUG= 633846 

Change-Id: I9c81b10aeae20207b7c30daab3040b73c3851020
Reviewed-on: https://chromium-review.googlesource.com/406907
Reviewed-by: Paweł Hajdan Jr. <phajdan@google.com>
Commit-Queue: Emma Söderberg <emso@chromium.org>

[add] https://crrev.com/3caa93063acc0aa9ef59bd1741dfd306161359a9/go/src/infra/tricium/service/default/cron.yaml

Comment 9 by emso@chromium.org, Jan 11 2017

Labels: Tricium

Comment 10 by emso@chromium.org, Apr 28 2017

Labels: Milestone-Gerrit

Comment 11 by emso@chromium.org, May 22 2017

Labels: -Pri-2 Pri-1
Project Member

Comment 12 by bugdroid1@chromium.org, Jun 26 2017 

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

commit 97fb41c9cbdc811826754db3e8174f1e13716266
Author: Emma <emso@chromium.org>
Date: Mon Jun 26 15:17:16 2017

Merges Gerrit modules to one.

Bug:633846
Change-Id: Ifd4cb6a1a870bbe248fb24015209b79f9757cea9
Reviewed-on: https://chromium-review.googlesource.com/539415
Commit-Queue: Emma Söderberg <emso@chromium.org>
Reviewed-by: Marc-Antoine Ruel <maruel@chromium.org>

[add] https://crrev.com/97fb41c9cbdc811826754db3e8174f1e13716266/go/src/infra/tricium/appengine/gerrit/poll.go
[delete] https://crrev.com/56b49b8c59765e36c18493dc26650d329e85f319/go/src/infra/tricium/appengine/gerrit-poller/init.go
[rename] https://crrev.com/97fb41c9cbdc811826754db3e8174f1e13716266/go/src/infra/tricium/appengine/gerrit/rpc_report_completed.go
[delete] https://crrev.com/56b49b8c59765e36c18493dc26650d329e85f319/go/src/infra/tricium/appengine/gerrit-poller/gerrit-poller.infra_testing
[delete] https://crrev.com/56b49b8c59765e36c18493dc26650d329e85f319/go/src/infra/tricium/appengine/gerrit-poller/handlers_test.go
[delete] https://crrev.com/56b49b8c59765e36c18493dc26650d329e85f319/go/src/infra/tricium/appengine/gerrit-poller/handlers.go
[rename] https://crrev.com/97fb41c9cbdc811826754db3e8174f1e13716266/go/src/infra/tricium/appengine/gerrit/rpc_report_results_test.go
[rename] https://crrev.com/97fb41c9cbdc811826754db3e8174f1e13716266/go/src/infra/tricium/appengine/gerrit/rpc_report_completed_test.go
[add] https://crrev.com/97fb41c9cbdc811826754db3e8174f1e13716266/go/src/infra/tricium/appengine/gerrit/poll_test.go
[delete] https://crrev.com/56b49b8c59765e36c18493dc26650d329e85f319/go/src/infra/tricium/appengine/gerrit-reporter/handlers.go
[modify] https://crrev.com/97fb41c9cbdc811826754db3e8174f1e13716266/go/src/infra/tricium/appengine/frontend/cron.yaml
[add] https://crrev.com/97fb41c9cbdc811826754db3e8174f1e13716266/go/src/infra/tricium/appengine/gerrit/gerrit.infra_testing
[add] https://crrev.com/97fb41c9cbdc811826754db3e8174f1e13716266/go/src/infra/tricium/appengine/gerrit/handlers.go
[rename] https://crrev.com/97fb41c9cbdc811826754db3e8174f1e13716266/go/src/infra/tricium/appengine/gerrit/init.go
[add] https://crrev.com/97fb41c9cbdc811826754db3e8174f1e13716266/go/src/infra/tricium/appengine/gerrit/gerrit_test.go
[rename] https://crrev.com/97fb41c9cbdc811826754db3e8174f1e13716266/go/src/infra/tricium/appengine/gerrit/rpc_report_launched_test.go
[modify] https://crrev.com/97fb41c9cbdc811826754db3e8174f1e13716266/go/src/infra/tricium/appengine/dispatch.yaml
[rename] https://crrev.com/97fb41c9cbdc811826754db3e8174f1e13716266/go/src/infra/tricium/appengine/gerrit/module-gerrit.yaml
[delete] https://crrev.com/56b49b8c59765e36c18493dc26650d329e85f319/go/src/infra/tricium/appengine/common/gerrit.go
[delete] https://crrev.com/56b49b8c59765e36c18493dc26650d329e85f319/go/src/infra/tricium/appengine/gerrit-reporter/module-gerrit-reporter.yaml
[delete] https://crrev.com/56b49b8c59765e36c18493dc26650d329e85f319/go/src/infra/tricium/appengine/gerrit-reporter/gerrit-reporter.infra_testing
[rename] https://crrev.com/97fb41c9cbdc811826754db3e8174f1e13716266/go/src/infra/tricium/appengine/gerrit/rpc_report_launched.go
[rename] https://crrev.com/97fb41c9cbdc811826754db3e8174f1e13716266/go/src/infra/tricium/appengine/gerrit/rpc_report_results.go
[add] https://crrev.com/97fb41c9cbdc811826754db3e8174f1e13716266/go/src/infra/tricium/appengine/gerrit/gerrit.go
Project Member

Comment 14 by bugdroid1@chromium.org, Sep 7 2017 

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infradata/config/+/c06e6691afeddf3fa4d89675e44ff66678d42d82

commit c06e6691afeddf3fa4d89675e44ff66678d42d82
Author: Emma Soederberg <emso@google.com>
Date: Thu Sep 07 07:39:12 2017
Project Member

Comment 16 by bugdroid1@chromium.org, Sep 8 2017 

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infradata/config/+/b6ae9b3e02294058ec5804fbb2935e0ef611dc17

commit b6ae9b3e02294058ec5804fbb2935e0ef611dc17
Author: Emma Soederberg <emso@google.com>
Date: Fri Sep 08 09:11:07 2017
Project Member

Comment 17 by bugdroid1@chromium.org, Sep 8 2017 

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infradata/config/+/be6e28bcc6803fd2f9759b40c75461ac561c3997

commit be6e28bcc6803fd2f9759b40c75461ac561c3997
Author: Emma Soederberg <emso@google.com>
Date: Fri Sep 08 11:32:35 2017

Comment 18 by emso@chromium.org, Sep 8 2017

Status: Verified (was: Assigned)
►

Sign in to add a comment