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

Issue 695464 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Add Tricium Results RPC

Project Member Reported by emso@chromium.org, Feb 23 2017

Issue description

Mirror the Analyze RPC with a Results RPC returning the current result state. 

Potentially use this RPC to feed data to a later Polymer UI. 
 

Comment 1 by emso@chromium.org, Feb 24 2017

Labels: -Pri-3 Pri-1
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 28 2017

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

commit 08c2238c7d569be566dd02e3b404665756836a6c
Author: Emma <emso@chromium.org>
Date: Tue Mar 28 12:01:58 2017

Adds results and progress RPCs to API

BUG= 695464 

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

[modify] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/appengine/common/track/track.go
[modify] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/appengine/tracker/rpc_worker_launched_test.go
[modify] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/api/v1/tricium.pb.go
[add] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/api/v1/tricium_helper.go
[modify] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/appengine/tracker/rpc_worker_launched.go
[add] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/appengine/frontend/rpc_progress_test.go
[add] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/appengine/frontend/rpc_results_test.go
[add] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/appengine/frontend/rpc_results.go
[modify] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/api/v1/config.pb.go
[modify] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/api/v1/pb.discovery.go
[add] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/appengine/frontend/rpc_progress.go
[add] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/api/v1/tricium_helper_test.go
[modify] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/appengine/frontend/frontend.infra_testing
[modify] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/api/v1/v1.infra_testing
[modify] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/appengine/tracker/rpc_workflow_launched_test.go
[modify] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/api/v1/tricium.proto
[modify] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/appengine/frontend/rpc_analyze.go
[modify] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/appengine/tracker/rpc_worker_done.go
[modify] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/appengine/tracker/rpc_worker_done_test.go
[modify] https://crrev.com/08c2238c7d569be566dd02e3b404665756836a6c/go/src/infra/tricium/appengine/tracker/rpc_workflow_launched.go

Project Member

Comment 3 by bugdroid1@chromium.org, Mar 28 2017

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

commit 94e7c1808611bc091a88eb5391567df4092a50ee
Author: Emma <emso@chromium.org>
Date: Tue Mar 28 17:06:38 2017

Implements Progress RPC.

BUG= 695464 

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

[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/appengine/common/track/track.go
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/api/v1/tricium.pb.go
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/appengine/tracker/rpc_worker_done.go
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/api/v1/data.pb.go
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/api/v1/data.proto
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/appengine/frontend/rpc_progress.go
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/api/v1/pb.discovery.go
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/appengine/frontend/frontend.infra_testing
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/api/v1/tricium.proto
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/appengine/tracker/tracker.infra_testing
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/appengine/frontend/rpc_progress_test.go
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/appengine/tracker/rpc_worker_done_test.go
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/appengine/tracker/rpc_workflow_launched.go

Project Member

Comment 4 by bugdroid1@chromium.org, Mar 28 2017

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

commit 94e7c1808611bc091a88eb5391567df4092a50ee
Author: Emma <emso@chromium.org>
Date: Tue Mar 28 17:06:38 2017

Implements Progress RPC.

BUG= 695464 

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

[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/appengine/common/track/track.go
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/api/v1/tricium.pb.go
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/appengine/tracker/rpc_worker_done.go
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/api/v1/data.pb.go
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/api/v1/data.proto
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/appengine/frontend/rpc_progress.go
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/api/v1/pb.discovery.go
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/appengine/frontend/frontend.infra_testing
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/api/v1/tricium.proto
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/appengine/tracker/tracker.infra_testing
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/appengine/frontend/rpc_progress_test.go
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/appengine/tracker/rpc_worker_done_test.go
[modify] https://crrev.com/94e7c1808611bc091a88eb5391567df4092a50ee/go/src/infra/tricium/appengine/tracker/rpc_workflow_launched.go

Project Member

Comment 5 by bugdroid1@chromium.org, Mar 31 2017

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

commit 809ddc62170d14e374e0c3bf32045c7a0622035f
Author: Emma <emso@chromium.org>
Date: Fri Mar 31 06:42:19 2017

Adds Results RPC.

BUG= 695464 

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

[modify] https://crrev.com/809ddc62170d14e374e0c3bf32045c7a0622035f/go/src/infra/tricium/appengine/common/track/track.go
[modify] https://crrev.com/809ddc62170d14e374e0c3bf32045c7a0622035f/go/src/infra/tricium/api/v1/tricium.pb.go
[modify] https://crrev.com/809ddc62170d14e374e0c3bf32045c7a0622035f/go/src/infra/tricium/appengine/frontend/rpc_progress_test.go
[modify] https://crrev.com/809ddc62170d14e374e0c3bf32045c7a0622035f/go/src/infra/tricium/appengine/frontend/rpc_results_test.go
[modify] https://crrev.com/809ddc62170d14e374e0c3bf32045c7a0622035f/go/src/infra/tricium/appengine/devcfg/services/tricium-dev/service.cfg
[modify] https://crrev.com/809ddc62170d14e374e0c3bf32045c7a0622035f/go/src/infra/tricium/api/v1/pb.discovery.go
[modify] https://crrev.com/809ddc62170d14e374e0c3bf32045c7a0622035f/go/src/infra/tricium/appengine/frontend/rpc_results.go
[modify] https://crrev.com/809ddc62170d14e374e0c3bf32045c7a0622035f/go/src/infra/tricium/api/v1/tricium.proto
[modify] https://crrev.com/809ddc62170d14e374e0c3bf32045c7a0622035f/go/src/infra/tricium/appengine/tracker/rpc_worker_done.go
[modify] https://crrev.com/809ddc62170d14e374e0c3bf32045c7a0622035f/go/src/infra/tricium/appengine/tracker/rpc_worker_done_test.go

Comment 6 by emso@chromium.org, Mar 31 2017

Status: Verified (was: Assigned)
Done with a a bonus Progress RPC.

Sign in to add a comment