New issue
Advanced search Search tips

Issue 829168 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

servo-9040 firmware preflight builder failing

Project Member Reported by nsanders@chromium.org, Apr 5 2018

Issue description

Seems like something is trying to download arc++ stuff, but this build doesn't support arc++?


See:
https://uberchromegw.corp.google.com/i/chromeos.branch/builders/falco%20pre-flight%20firmware-servo-9040.B


GSNoSuchKey: return code: 1; command: /b/c/cbuild/repository/.cache/common/gsutil_4.19.tar.gz/gsutil/gsutil -o 'Boto:num_retries=10' ls -- gs://android-build-chromeos/builds/git_mnc-dr-arc-dev-linux-static_sdk_tools
CommandException: One or more URLs matched no objects.
 
Cc: jclinton@chromium.org
Cc: dgarr...@chromium.org
Unclear to me if this is consistent. The most recent failure has a different signature. I've forced a new build to see what happens. https://uberchromegw.corp.google.com/i/chromeos.branch/builders/falco%20pre-flight%20firmware-servo-9040.B/builds/15
The builder seems to have two failure signatures.

In one of them, it fails trying download a non-existent file. I don't understand why a sync stage would try to fetch that:
  gs://android-build-chromeos/builds/git_mnc-dr-arc-dev-linux-static_sdk_tools

In the other doesn't make any sense at all:
  https://logs.chromium.org/v/?s=chromeos%2Fbb%2Fchromeos.branch%2Ffalco_pre-flight_firmware-servo-9040.B%2F14%2F%2B%2F%2A%2A%2Fstdout&s=chromeos%2Fbb%2Fchromeos.branch%2Ffalco_pre-flight_firmware-servo-9040.B%2F14%2F%2B%2F%2A%2A%2Fstderr


It jumps back and forth between the two failures randomly.


So.... what's this builder supposed to be for? What's it preflighting?
It's the pre flight builder for the servo firmware branch
Who originally brought it up? More the the point, who can sign off if I make radical changes to it?
I brought it up in the fist place. More changes are likely needed.
Thanks.
I own servo releases, so go for it and make radical changes. They will be appreciated =)
Project Member

Comment 8 by bugdroid1@chromium.org, Apr 6 2018

Labels: merge-merged-firmware-servo-9040.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/663723e9bf79642779f3651b667d337545459817

commit 663723e9bf79642779f3651b667d337545459817
Author: YH Lin <yueherngl@google.com>
Date: Fri Apr 06 22:13:17 2018

servo: experiments with the change for pre-flight

BUG= chromium:829168 
TEST=None

Change-Id: I605c32dc9ba04fc5c9d49734004b3654e3501e64
Reviewed-on: https://chromium-review.googlesource.com/1000411
Reviewed-by: YH Lin <yueherngl@chromium.org>
Commit-Queue: YH Lin <yueherngl@chromium.org>
Tested-by: YH Lin <yueherngl@chromium.org>

[modify] https://crrev.com/663723e9bf79642779f3651b667d337545459817/cbuildbot/config_dump.json
[modify] https://crrev.com/663723e9bf79642779f3651b667d337545459817/cbuildbot/chromeos_config.py

Project Member

Comment 9 by bugdroid1@chromium.org, Apr 7 2018

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

commit 723ade5008d24f9af78f352437b0facaf38a1e97
Author: YH Lin <yueherngl@chromium.org>
Date: Sat Apr 07 04:03:15 2018

Revert "servo: experiments with the change for pre-flight"

This reverts commit 663723e9bf79642779f3651b667d337545459817.

Reason for revert: <INSERT REASONING HERE>

Original change's description:
> servo: experiments with the change for pre-flight
> 
> BUG= chromium:829168 
> TEST=None
> 
> Change-Id: I605c32dc9ba04fc5c9d49734004b3654e3501e64
> Reviewed-on: https://chromium-review.googlesource.com/1000411
> Reviewed-by: YH Lin <yueherngl@chromium.org>
> Commit-Queue: YH Lin <yueherngl@chromium.org>
> Tested-by: YH Lin <yueherngl@chromium.org>

Bug:  chromium:829168 
Change-Id: I90effa83a4a4409a32698f3a6f656a1bd27a8ea1
Reviewed-on: https://chromium-review.googlesource.com/1000429
Reviewed-by: YH Lin <yueherngl@chromium.org>
Commit-Queue: YH Lin <yueherngl@chromium.org>
Tested-by: YH Lin <yueherngl@chromium.org>

[modify] https://crrev.com/723ade5008d24f9af78f352437b0facaf38a1e97/cbuildbot/config_dump.json
[modify] https://crrev.com/723ade5008d24f9af78f352437b0facaf38a1e97/cbuildbot/chromeos_config.py

Components: Infra>Client>ChromeOS>Test
Owner: ----
Gathered onto a servo v4 hotlist, removing self as owner.
Owner: dgarr...@chromium.org
I think dgarrett@ was looking at this. 
Labels: -Pri-3 Pri-1
Owner: shapiroc@chromium.org
Need this to release upcoming fixes for FAFT, and new reference boards.
Owner: gmeinke@chromium.org
same running gsutil issue that needs the fix patched in to the respective branch
Status: Started (was: Untriaged)
Above comment #13 is not fully correct, there were some cert errors that were fixed but issue is really that the file gs://android-build-chromeos/builds/git_mnc-dr-arc-dev-linux-static_sdk_tools does not exist anymore.

Not sure why a firmware builder is trying to get arc stuff ...
Project Member

Comment 16 by bugdroid1@chromium.org, May 24 2018

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

commit a450995e04e41fa633ae723b52cf19afef854195
Author: YH Lin <yueherngl@google.com>
Date: Thu May 24 21:58:21 2018

servo: another experiment with the pre-flight change

BUG= chromium:829168 
TEST=None

Change-Id: I380c6c45058d099e961850c76e2eb12abb0df28c
Reviewed-on: https://chromium-review.googlesource.com/1072642
Reviewed-by: YH Lin <yueherngl@chromium.org>
Commit-Queue: YH Lin <yueherngl@chromium.org>
Tested-by: YH Lin <yueherngl@chromium.org>

[modify] https://crrev.com/a450995e04e41fa633ae723b52cf19afef854195/cbuildbot/config_dump.json
[modify] https://crrev.com/a450995e04e41fa633ae723b52cf19afef854195/cbuildbot/chromeos_config.py

Status: Fixed (was: Started)
Thanks Greg. With the change above (#16) we have the first successful pre-flight build.

I am marking the bug fixed. We can re-open if the pre-flight build fail again.


 

Sign in to add a comment