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

Issue 759158 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

"cros tryjob --yet betty-vmtest-informational" fails oddly.

Project Member Reported by dgarr...@chromium.org, Aug 25 2017

Issue description

I thought this was already filed, but I can't find it.

"cros tryjob --yet betty-vmtest-informational" fails because the value "builder_name" is null in the buildbucket request.

Example buildbucket result:

https://apis-explorer.appspot.com/apis-explorer/?base=https://cr-buildbucket.appspot.com/_ah/api#p/buildbucket/v1/buildbucket.get?id=8970326287936457152&_h=1&

 
I believe this will affect any build scheduled with a non-standard name, or which hasn't been added via a trybot waterfall restart.

Comment 2 by pwang@chromium.org, Aug 25 2017

It used to be worked before Aug 22. Did the waterfall restart during that day?

In addition to this, is it possible to add the builder to waterfall(is it implies that the builder will build itself automatically everyday?) so that we  will not get the same issue in the future?
I found that a CL I landed recently had a bug that's helping to cause this. I have a fix up now.

https://chromium-review.googlesource.com/c/chromiumos/chromite/+/636046

Project Member

Comment 4 by bugdroid1@chromium.org, Aug 26 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/b049418f25b1d37f6adfb9232bb0c656976e8a5e

commit b049418f25b1d37f6adfb9232bb0c656976e8a5e
Author: Don Garrett <dgarrett@google.com>
Date: Sat Aug 26 02:54:14 2017

remote_try: Use default builder name if template is None.

My CL:624672 attempted to make "_GetBuilder" more resilient, but
incorrectly handled builders which don't use a template at all.

BUG= chromium:759158 
TEST=run_tests

Change-Id: Ic69770dcf80a59fd2c496f612da4e7a4a985749f
Reviewed-on: https://chromium-review.googlesource.com/636046
Commit-Ready: Don Garrett <dgarrett@chromium.org>
Tested-by: Don Garrett <dgarrett@chromium.org>
Reviewed-by: Ningning Xia <nxia@chromium.org>

[modify] https://crrev.com/b049418f25b1d37f6adfb9232bb0c656976e8a5e/cbuildbot/remote_try.py

Comment 5 by pwang@chromium.org, Sep 5 2017

Status: Verified (was: Started)

Sign in to add a comment