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

Issue 625781 link

Starred by 4 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Feature

Blocked on:
issue 633846
issue 633848
issue 633851

Blocking:
issue 615236



Sign in to add a comment

Deploy the end-to-end Tricium service

Project Member Reported by emso@chromium.org, Jul 5 2016

Issue description

Deploy the end-to-end Tricium service on App engine.

This is part of the first Tricium milestone, the end-to-end prototype. See the design doc for Tricium (https://docs.google.com/document/d/1uCTK2v5BKcUSMaiObsg_NZSsyULS8K2eTya7ZAPQx_U/edit?usp=sharing) for more details.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 8 2016

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

commit 269063afbcba7c7f870e9522668d789673555f08
Author: emso <emso@chromium.org>
Date: Fri Jul 08 09:57:19 2016

Adds initial Tricium service GAE configs

This change has been deployed to chrome-infra-tricim-dev and tested manually. To re-run the test, go to the landing page and click the dummy analysis request button. The expected result is a task that goes through the three queues of the app (verified on the task queue overview page).

BUG= 625781 

Review-Url: https://codereview.chromium.org/2125603002

[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/default/default.infra_testing
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/default/default.yaml
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/default/handlers.go
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/default/templates
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/dispatch.yaml
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/gerrit-poller/gerrit-poller.infra_testing
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/gerrit-poller/gerrit-poller.yaml
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/gerrit-poller/handlers.go
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/gerrit-poller/templates
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/gerrit-reporter/gerrit-reporter.infra_testing
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/gerrit-reporter/gerrit-reporter.yaml
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/gerrit-reporter/handlers.go
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/gerrit-reporter/templates
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/queue.yaml
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/templates/base.html
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/workflow-launcher/handlers.go
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/workflow-launcher/templates
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/workflow-launcher/workflow-launcher.infra_testing
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/workflow-launcher/workflow-launcher.yaml
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/workflow-listener/handlers.go
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/workflow-listener/templates
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/workflow-listener/workflow-listener.infra_testing
[add] https://crrev.com/269063afbcba7c7f870e9522668d789673555f08/go/src/infra/tricium/service/workflow-listener/workflow-listener.yaml

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

Blockedon: 633846

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

Blockedon: 633848

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

Blockedon: 633851

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

Blockedon: 633856

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

Labels: -Pri-3 Pri-2
Project Member

Comment 7 by bugdroid1@chromium.org, Sep 23 2016

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

commit c9e62de9e3ac99483fa87d0ffe2a3e42fcdc1fd2
Author: Emma Soederberg <emso@chromium.org>
Date: Thu Sep 22 10:31:53 2016

Updates the Tricium service architecture.

Adds a generic reporter module through which all results will pass, and
adds a service queue to the service (default) module through which all
requests will pass. This will allow for tracking of in progress requests
and results at these places. A data store will be added later.

BUG= 625781 

Change-Id: Ic3a91a29883b4c705551fcb75a132da8767562c8
Reviewed-on: https://chromium-review.googlesource.com/388431
Reviewed-by: Paweł Hajdan Jr. <phajdan@google.com>

[modify] https://crrev.com/c9e62de9e3ac99483fa87d0ffe2a3e42fcdc1fd2/go/src/infra/tricium/service/default/default.yaml
[modify] https://crrev.com/c9e62de9e3ac99483fa87d0ffe2a3e42fcdc1fd2/go/src/infra/tricium/service/default/handlers.go
[modify] https://crrev.com/c9e62de9e3ac99483fa87d0ffe2a3e42fcdc1fd2/go/src/infra/tricium/service/dispatch.yaml
[modify] https://crrev.com/c9e62de9e3ac99483fa87d0ffe2a3e42fcdc1fd2/go/src/infra/tricium/service/gerrit-poller/handlers.go
[modify] https://crrev.com/c9e62de9e3ac99483fa87d0ffe2a3e42fcdc1fd2/go/src/infra/tricium/service/queue.yaml
[add] https://crrev.com/c9e62de9e3ac99483fa87d0ffe2a3e42fcdc1fd2/go/src/infra/tricium/service/reporter/default.infra_testing
[add] https://crrev.com/c9e62de9e3ac99483fa87d0ffe2a3e42fcdc1fd2/go/src/infra/tricium/service/reporter/handlers.go
[add] https://crrev.com/c9e62de9e3ac99483fa87d0ffe2a3e42fcdc1fd2/go/src/infra/tricium/service/reporter/reporter.yaml
[add] https://crrev.com/c9e62de9e3ac99483fa87d0ffe2a3e42fcdc1fd2/go/src/infra/tricium/service/reporter/templates
[modify] https://crrev.com/c9e62de9e3ac99483fa87d0ffe2a3e42fcdc1fd2/go/src/infra/tricium/service/workflow-listener/handlers.go

Project Member

Comment 8 by bugdroid1@chromium.org, Oct 3 2016

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

commit 18be8b718b05aaf6099adfe079c5029f4f0036e3
Author: Emma Soederberg <emso@chromium.org>
Date: Mon Oct 03 10:43:30 2016

Adds storing of run data for better tracking.

BUG= 625781 

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

[modify] https://crrev.com/18be8b718b05aaf6099adfe079c5029f4f0036e3/go/src/infra/tricium/service/common/common.go
[modify] https://crrev.com/18be8b718b05aaf6099adfe079c5029f4f0036e3/go/src/infra/tricium/service/default/handlers.go
[modify] https://crrev.com/18be8b718b05aaf6099adfe079c5029f4f0036e3/go/src/infra/tricium/service/gerrit-poller/handlers.go
[modify] https://crrev.com/18be8b718b05aaf6099adfe079c5029f4f0036e3/go/src/infra/tricium/service/gerrit-reporter/handlers.go
[modify] https://crrev.com/18be8b718b05aaf6099adfe079c5029f4f0036e3/go/src/infra/tricium/service/reporter/handlers.go
[modify] https://crrev.com/18be8b718b05aaf6099adfe079c5029f4f0036e3/go/src/infra/tricium/service/templates/base.html
[add] https://crrev.com/18be8b718b05aaf6099adfe079c5029f4f0036e3/go/src/infra/tricium/service/templates/results.html
[modify] https://crrev.com/18be8b718b05aaf6099adfe079c5029f4f0036e3/go/src/infra/tricium/service/validator/handlers.go
[modify] https://crrev.com/18be8b718b05aaf6099adfe079c5029f4f0036e3/go/src/infra/tricium/service/workflow-launcher/handlers.go
[modify] https://crrev.com/18be8b718b05aaf6099adfe079c5029f4f0036e3/go/src/infra/tricium/service/workflow-listener/handlers.go

Project Member

Comment 9 by bugdroid1@chromium.org, Nov 14 2016

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

commit 8afd86073851b561e97080f2190842f5b75250b9
Author: Emma Soederberg <emso@chromium.org>
Date: Mon Nov 14 08:50:23 2016

Adds service UI for request of analysis of Gerrit patch sets.

This change also refactors the HTML UI:
- adds some CSS styling.
- removes unimplemented status pages.
- refactors the interface to the shared base page.

This change enables manual triggering of analysis.

BUG= 625781 

Change-Id: I5793b9d2bdccc9dff35793785862d3b874c38de5
Reviewed-on: https://chromium-review.googlesource.com/410660
Commit-Queue: Emma Söderberg <emso@chromium.org>
Reviewed-by: Nodir Turakulov <nodir@chromium.org>

[modify] https://crrev.com/8afd86073851b561e97080f2190842f5b75250b9/go/src/infra/tricium/service/common/common.go
[modify] https://crrev.com/8afd86073851b561e97080f2190842f5b75250b9/go/src/infra/tricium/service/default/app.yaml
[modify] https://crrev.com/8afd86073851b561e97080f2190842f5b75250b9/go/src/infra/tricium/service/default/handlers.go
[add] https://crrev.com/8afd86073851b561e97080f2190842f5b75250b9/go/src/infra/tricium/service/default/static
[modify] https://crrev.com/8afd86073851b561e97080f2190842f5b75250b9/go/src/infra/tricium/service/gerrit-poller/handlers.go
[modify] https://crrev.com/8afd86073851b561e97080f2190842f5b75250b9/go/src/infra/tricium/service/gerrit-reporter/handlers.go
[add] https://crrev.com/8afd86073851b561e97080f2190842f5b75250b9/go/src/infra/tricium/service/static/style.css
[modify] https://crrev.com/8afd86073851b561e97080f2190842f5b75250b9/go/src/infra/tricium/service/templates/base.html
[modify] https://crrev.com/8afd86073851b561e97080f2190842f5b75250b9/go/src/infra/tricium/service/templates/results.html
[modify] https://crrev.com/8afd86073851b561e97080f2190842f5b75250b9/go/src/infra/tricium/service/validator/handlers.go
[modify] https://crrev.com/8afd86073851b561e97080f2190842f5b75250b9/go/src/infra/tricium/service/workflow-launcher/handlers.go
[modify] https://crrev.com/8afd86073851b561e97080f2190842f5b75250b9/go/src/infra/tricium/service/workflow-listener/handlers.go

Project Member

Comment 10 by bugdroid1@chromium.org, Dec 8 2016

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

commit 3711d198513505ac9058be3621c62d3ce8e22b7d
Author: Emma Soederberg <emso@chromium.org>
Date: Thu Dec 08 07:42:48 2016

Adds go-querystring to dependencies.

This dependency provides a library for encoding of structs as url values.

BUG= 625781 

Change-Id: I5f03151ff52f73330cd396adb5cabb3cdae12ba9
Reviewed-on: https://chromium-review.googlesource.com/417264
Commit-Queue: Dave Sansome <dsansome@chromium.org>
Commit-Queue: Emma Söderberg <emso@chromium.org>
Reviewed-by: Dave Sansome <dsansome@chromium.org>

[modify] https://crrev.com/3711d198513505ac9058be3621c62d3ce8e22b7d/go/deps.lock
[modify] https://crrev.com/3711d198513505ac9058be3621c62d3ce8e22b7d/go/deps.yaml

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

Labels: Tricium
Project Member

Comment 12 by bugdroid1@chromium.org, Apr 20 2017

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

commit 477970020bc52d4400d944fe438a3fdc9289d64d
Author: Emma Soederberg <emso@google.com>
Date: Thu Apr 20 09:33:23 2017

Project Member

Comment 13 by bugdroid1@chromium.org, Apr 20 2017

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

commit 2361bb866324ab4277a9906e6447551e370a5efb
Author: Emma Soederberg <emso@google.com>
Date: Thu Apr 20 09:45:34 2017

Project Member

Comment 14 by bugdroid1@chromium.org, Apr 20 2017

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

commit 2361bb866324ab4277a9906e6447551e370a5efb
Author: Emma Soederberg <emso@google.com>
Date: Thu Apr 20 09:45:34 2017

Project Member

Comment 15 by bugdroid1@chromium.org, Apr 20 2017

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

commit c4256ac986da014c5bef747016f5dce40c5d1fc6
Author: Emma <emso@chromium.org>
Date: Thu Apr 20 21:56:14 2017

Directly address the driver module in the pubsub push URL

Bug:625781

Change-Id: I7297ec7d5d8481395becab22663e154e2977691f
Reviewed-on: https://chromium-review.googlesource.com/483520
Reviewed-by: Ryan Tseng <hinoka@chromium.org>
Commit-Queue: Emma Söderberg <emso@chromium.org>

[modify] https://crrev.com/c4256ac986da014c5bef747016f5dce40c5d1fc6/go/src/infra/tricium/appengine/common/pubsub.go

Project Member

Comment 16 by bugdroid1@chromium.org, Apr 20 2017

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

commit c4256ac986da014c5bef747016f5dce40c5d1fc6
Author: Emma <emso@chromium.org>
Date: Thu Apr 20 21:56:14 2017

Directly address the driver module in the pubsub push URL

Bug:625781

Change-Id: I7297ec7d5d8481395becab22663e154e2977691f
Reviewed-on: https://chromium-review.googlesource.com/483520
Reviewed-by: Ryan Tseng <hinoka@chromium.org>
Commit-Queue: Emma Söderberg <emso@chromium.org>

[modify] https://crrev.com/c4256ac986da014c5bef747016f5dce40c5d1fc6/go/src/infra/tricium/appengine/common/pubsub.go

Project Member

Comment 17 by bugdroid1@chromium.org, Apr 26 2017

Labels: merge-merged-config
The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/c11392ea74c82781b39a6a7dd5ef9cdbbc5f22c9

commit c11392ea74c82781b39a6a7dd5ef9cdbbc5f22c9
Author: Emma <emso@chromium.org>
Date: Wed Apr 26 13:06:26 2017

Adds initial Tricium prod config

Bug:625781

Change-Id: I3a1f0ce7a4a25a3e77355247ea70239352ccdbed
Reviewed-on: https://chromium-review.googlesource.com/488081
Reviewed-by: Paweł Hajdan Jr. <phajdan.jr@chromium.org>

[add] https://crrev.com/c11392ea74c82781b39a6a7dd5ef9cdbbc5f22c9/tricium-prod.cfg

Project Member

Comment 18 by bugdroid1@chromium.org, Apr 27 2017

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

commit aacf5eefede144911ab46f74442fb428b6d09c58
Author: Emma <emso@chromium.org>
Date: Thu Apr 27 08:19:54 2017

Lets error codes bubble up for analyze calls

Bug:625781

Change-Id: Ibb7fd67a8870a61ced9372a33282999eaf411d84
Reviewed-on: https://chromium-review.googlesource.com/488242
Reviewed-by: Scott Lee <ddoman@google.com>
Reviewed-by: Emma Söderberg <emso@chromium.org>
Commit-Queue: Emma Söderberg <emso@chromium.org>

[modify] https://crrev.com/aacf5eefede144911ab46f74442fb428b6d09c58/go/src/infra/tricium/appengine/frontend/rpc_analyze_test.go
[modify] https://crrev.com/aacf5eefede144911ab46f74442fb428b6d09c58/go/src/infra/tricium/appengine/frontend/rpc_analyze.go

Project Member

Comment 19 by bugdroid1@chromium.org, Apr 27 2017

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

commit d9eca2daae6b800a12929ab22b8bb47e2e908d23
Author: Emma Soederberg <emso@google.com>
Date: Thu Apr 27 09:33:08 2017

Project Member

Comment 20 by bugdroid1@chromium.org, Apr 27 2017

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

commit 19087cda165399b0a3bdc5cf67c53901eaf11486
Author: Emma Soederberg <emso@google.com>
Date: Thu Apr 27 09:52:55 2017

Project Member

Comment 21 by bugdroid1@chromium.org, Apr 27 2017

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

commit ec97259c8fd93e404b8bc3483eeed755dea5b736
Author: Emma Soederberg <emso@google.com>
Date: Thu Apr 27 10:13:30 2017

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

Status: Verified (was: Assigned)
Blockedon: -633856

Sign in to add a comment