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

Issue 802234 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Task



Sign in to add a comment

Onboard the Infra project

Project Member Reported by emso@chromium.org, Jan 16 2018

Issue description

Connect the infra project to Tricium and enable at least the Spacey analyzer.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 17 2018

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

commit 47ed7b56a8b4d2f375fa685243682c4275cb8439
Author: Emma <emso@chromium.org>
Date: Wed Jan 17 09:10:28 2018

Updates Tricium config.

- Removes the tricium-prod config to not be connected to two instances at the same time.
- Update the tricium-dev config to use functions.

Bug:802234
Change-Id: I05de4a98398d9b5347735945fbb2ca48e8c5b02a
Reviewed-on: https://chromium-review.googlesource.com/868130
Reviewed-by: Marc-Antoine Ruel <maruel@chromium.org>
Reviewed-by: Quinten Yearsley <qyearsley@chromium.org>
Commit-Queue: Emma Söderberg <emso@chromium.org>

[modify] https://crrev.com/47ed7b56a8b4d2f375fa685243682c4275cb8439/tricium-dev.cfg
[delete] https://crrev.com/59a80d8e6997c9515b48628cee1680052c1a1f48/tricium-prod.cfg

Project Member

Comment 2 by bugdroid1@chromium.org, Jan 19 2018

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

commit be56446819218d4f18f4e091e72e262a3a61d33e
Author: Emma Soederberg <emso@google.com>
Date: Fri Jan 19 06:35:12 2018

Project Member

Comment 3 by bugdroid1@chromium.org, Mar 6 2018

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

commit 6f0d4c445edf80c2d0ed139a30e5080d874a55cb
Author: Quinten Yearsley <qyearsley@chromium.org>
Date: Tue Mar 06 23:14:52 2018

Labels: -merge-merged-config
Owner: qyears...@chromium.org
Status: Started (was: Available)
Example change:
https://chromium-review.googlesource.com/c/infra/infra/+/952956
This task can include luci-py and luci-go as well as the main infra repo.
Cc: aga...@chromium.org tandrii@chromium.org
The main infra.git repo is now set up with a service config, project config, and gerrit plugin config;
tricium analyzer runs are now visible for CLs there, e.g. https://chromium-review.googlesource.com/c/infra/infra/+/956485

Already done for luci-py and luci-go:

 - Add project configs https://crrev.com/c/966890, https://crrev.com/c/966898

Still left to do here:

 - Add tricium plugin config for luci-py; https://crrev.com/c/969299
 - Add tricium plugin config for luci-go; https://crrev.com/c/966938
 - Add luci-go and luci-py to service config; https://chrome-internal-review.googlesource.com/c/589982
Labels: Milestone-Dogfooding
Status: Fixed (was: Started)
luci-py and luci-go have both had project configs updated. Just confirmed in luci-py that tricium-prod does indeed run (https://chromium-review.googlesource.com/c/infra/luci/luci-py/+/1060563), I expect it should be the same for luci-go.

Sign in to add a comment