New issue
Advanced search Search tips

Issue 867655 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Jul 27
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug

Blocking:
issue 867282



Sign in to add a comment

commit chromite/third_party/...swarming client in chormite repo

Project Member Reported by pprabhu@chromium.org, Jul 25

Issue description

Currently, chromite/third_party/swarming.client is populated via repo. Chromite is used in contexts where third_party doesn't get populated by repo (e.g. site_packages of autotest).

We now want to actually use swarming lib from autotest, so include it in the repo directly.
 
Blocking: 867282
Labels: -Pri-2 Pri-1
Cc: dgarr...@chromium.org
There is a weird split where some of the chromite/third_party/ libs come via repo, and some are checked in.

I think it's better to just have everything checked in there. chromite is unusable without these python modules and is used standalone...
Cc: la...@chromium.org
Components: Infra>Client>ChromeOS>Build
we have a strong interest to *not* commit swarming.client to the chromite repo

it's hard to say what contexts you're running into here whereby the swarming client is necessary.  if it's on the DUT, then it sounds like you want an ebuild to install the swarming.client as a standard python package ?
Status: WontFix (was: Started)
I decided to not depend on chromite's swarming_lib.py or the bundled swarming.client checkout at all.

The context here was on a lab server.

Since we don't have a good story for deploying chromite with its dependencies inside autotest, I'm just going to shy away from using chromite's library in this case.

Sign in to add a comment