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

Issue 717896 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: ----



Sign in to add a comment

Add Gerrit reporter

Project Member Reported by emso@chromium.org, May 3 2017

Issue description

Add Gerrit reporter capable of reporting progress, as bubbles, and results, as robot comments, in Gerrit. The bubbles should be hierarchical if possible.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 14 2017

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

commit c2d868c7598626e26b6561acba17ca1742b1f89e
Author: Emma <emso@chromium.org>
Date: Wed Jun 14 12:57:41 2017

Adds Gerrit reporter.

This change adds functionality to the Gerrit reporter
module and extracts out Gerrit integration details
to an interface.

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

[modify] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/appengine/common/track/track.go
[modify] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/api/admin/v1/pb.discovery.go
[add] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/appengine/gerrit-reporter/rpc_report_completed.go
[modify] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/api/admin/v1/tracker.pb.go
[add] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/appengine/gerrit-reporter/init.go
[add] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/api/admin/v1/reporter.proto
[add] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/appengine/gerrit-reporter/rpc_report_launched.go
[modify] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/appengine/frontend/rpc_analyze.go
[modify] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/appengine/common/common.go
[add] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/appengine/gerrit-reporter/rpc_report_launched_test.go
[add] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/api/admin/v1/reporter.pb.go
[modify] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/appengine/tracker/tracker.infra_testing
[add] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/appengine/gerrit-reporter/rpc_report_results_test.go
[add] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/appengine/common/gerrit.go
[modify] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/appengine/gerrit-reporter/handlers.go
[modify] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/appengine/common/testing/testing.go
[modify] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/api/admin/v1/config.pb.go
[add] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/appengine/gerrit-reporter/rpc_report_completed_test.go
[modify] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/appengine/tracker/rpc_worker_done.go
[modify] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/api/admin/v1/workflow.pb.go
[add] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/appengine/gerrit-reporter/rpc_report_results.go
[modify] https://crrev.com/c2d868c7598626e26b6561acba17ca1742b1f89e/go/src/infra/tricium/appengine/tracker/rpc_workflow_launched.go

Project Member

Comment 2 by bugdroid1@chromium.org, Jun 15 2017

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

commit 3e9b05855af89505276aad45703feca3ddfebac1
Author: Emma <emso@chromium.org>
Date: Thu Jun 15 12:27:00 2017

Adds Gerrit server implementation.

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

[modify] https://crrev.com/3e9b05855af89505276aad45703feca3ddfebac1/go/src/infra/tricium/appengine/common/track/track.go
[modify] https://crrev.com/3e9b05855af89505276aad45703feca3ddfebac1/go/src/infra/tricium/appengine/gerrit-reporter/rpc_report_completed_test.go
[modify] https://crrev.com/3e9b05855af89505276aad45703feca3ddfebac1/go/src/infra/tricium/appengine/frontend/rpc_analyze.go
[modify] https://crrev.com/3e9b05855af89505276aad45703feca3ddfebac1/go/src/infra/tricium/appengine/gerrit-reporter/rpc_report_completed.go
[modify] https://crrev.com/3e9b05855af89505276aad45703feca3ddfebac1/go/src/infra/tricium/appengine/common/gerrit.go
[modify] https://crrev.com/3e9b05855af89505276aad45703feca3ddfebac1/go/src/infra/tricium/api/v1/pb.discovery.go
[modify] https://crrev.com/3e9b05855af89505276aad45703feca3ddfebac1/go/src/infra/tricium/appengine/gerrit-reporter/rpc_report_results.go
[modify] https://crrev.com/3e9b05855af89505276aad45703feca3ddfebac1/go/src/infra/tricium/api/v1/tricium.proto
[modify] https://crrev.com/3e9b05855af89505276aad45703feca3ddfebac1/go/src/infra/tricium/api/v1/tricium.pb.go
[modify] https://crrev.com/3e9b05855af89505276aad45703feca3ddfebac1/go/src/infra/tricium/appengine/gerrit-reporter/rpc_report_launched.go

Project Member

Comment 3 by bugdroid1@chromium.org, Jun 21 2017

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

commit fe1635fbbc49afcb98b82ceb11d74676d368f60d
Author: Emma <emso@chromium.org>
Date: Wed Jun 21 08:40:16 2017

Renames Reporter to Consumer in tricium.proto

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

[modify] https://crrev.com/fe1635fbbc49afcb98b82ceb11d74676d368f60d/go/src/infra/tricium/appengine/common/track/track.go
[modify] https://crrev.com/fe1635fbbc49afcb98b82ceb11d74676d368f60d/go/src/infra/tricium/api/v1/tricium.pb.go
[modify] https://crrev.com/fe1635fbbc49afcb98b82ceb11d74676d368f60d/go/src/infra/tricium/api/v1/data.pb.go
[modify] https://crrev.com/fe1635fbbc49afcb98b82ceb11d74676d368f60d/go/src/infra/tricium/appengine/tracker/rpc_worker_launched.go
[modify] https://crrev.com/fe1635fbbc49afcb98b82ceb11d74676d368f60d/go/src/infra/tricium/appengine/tracker/rpc_worker_done.go
[modify] https://crrev.com/fe1635fbbc49afcb98b82ceb11d74676d368f60d/go/src/infra/tricium/api/v1/config.pb.go
[modify] https://crrev.com/fe1635fbbc49afcb98b82ceb11d74676d368f60d/go/src/infra/tricium/api/v1/pb.discovery.go
[modify] https://crrev.com/fe1635fbbc49afcb98b82ceb11d74676d368f60d/go/src/infra/tricium/api/v1/tricium.proto
[modify] https://crrev.com/fe1635fbbc49afcb98b82ceb11d74676d368f60d/go/src/infra/tricium/appengine/frontend/rpc_analyze.go
[modify] https://crrev.com/fe1635fbbc49afcb98b82ceb11d74676d368f60d/go/src/infra/tricium/api/v1/platform.pb.go
[modify] https://crrev.com/fe1635fbbc49afcb98b82ceb11d74676d368f60d/go/src/infra/tricium/appengine/tracker/rpc_workflow_launched.go

Comment 4 by emso@chromium.org, Sep 7 2017

Status: Fixed (was: Assigned)

Sign in to add a comment