New issue
Advanced search Search tips

Issue 617472 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: ----

Blocking:
issue 622532



Sign in to add a comment

infra/infra Gerrit CL tryjobs fail on experimental swarming

Project Member Reported by tandrii@chromium.org, Jun 5 2016

Issue description

CL: https://chromium-review.googlesource.com/c/338811/5 (with Poly-Gerrit UI; otherwise see links to all builds https://chromium-cq-status.appspot.com/patch-status/338811/5).

Now compare Trusty 64:

Buildbot (green):
https://build.chromium.org/p/tryserver.infra/builders/Infra%20Linux%20Trusty%2064%20Tester/builds/2017

LUCI (purple(why?)-red)
https://luci-milo.appspot.com/swarming/prod/2f3b3b866fd80410

On a first glance, the difference is clearly in bot_update.
 
+hinoka@ any ideas what happened here?

+estaab@ because this should be a blocker to migrate infra/infra to swarming.

Comment 2 by no...@chromium.org, Jun 6 2016

Owner: no...@chromium.org
Status: Assigned (was: Untriaged)
I will figure it out
FYI, if you want re-trigger CQ, just remove and vote again for CQ label dry run (+1).

Comment 4 by hinoka@google.com, Jun 6 2016

buildbot version passes in:
--gerrit_repo https://chromium.googlesource.com/infra/infra --gerrit_ref refs/changes/11/338811/5

luci one doesn't.  the recipe never told bot update to apply any patches.

Comment 5 by no...@chromium.org, Jun 23 2016

Blocking: 622532
Labels: luci M-53 Pri-1
Status: Started (was: Assigned)

Comment 7 by no...@chromium.org, Jun 23 2016

ok, so the problem is that bot_update recipe module relies on "repository" property which is derived from "changes" build parameters by buildbot.
Perhaps swarmbucket should do the same: pass repo_url of the first change in the "changes" build parameter and copy it to property...

Comment 9 by no...@chromium.org, Jun 24 2016

Status: Fixed (was: Started)
https://luci-milo.appspot.com/swarming/prod/2f98c7bf11802310

Sign in to add a comment