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

Issue 920096 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

-release: TestSimpleChromeWorkflow - VMTest fails

Project Member Reported by drinkcat@chromium.org, Jan 9

Issue description

A bunch of recent -release builds fail in TestSimpleChromeWorkflow:

daisy-release: https://logs.chromium.org/logs/chromeos/buildbucket/cr-buildbucket.appspot.com/8924846644289184960/+/steps/TestSimpleChromeWorkflow/0/stdout
peppy-release: https://logs.chromium.org/logs/chromeos/buildbucket/cr-buildbucket.appspot.com/8924846583288327136/+/steps/TestSimpleChromeWorkflow/0/stdout
clapper-release: https://logs.chromium.org/logs/chromeos/buildbucket/cr-buildbucket.appspot.com/8924846647462527392/+/steps/TestSimpleChromeWorkflow/0/stdout
peach_pit-release: https://logs.chromium.org/logs/chromeos/buildbucket/cr-buildbucket.appspot.com/8924846584395417200/+/steps/TestSimpleChromeWorkflow/0/stdout

Killing compiler proxy.
  compiler proxy status: http://127.0.0.1:8088 quit!
  21:06:39: INFO: RunCommand: cros --log-level debug --cache-dir /b/swarming/w/ir/tmp/t/cbuildbot-tmpeKk8nM/chrome-sdk-cacheXpcWoI/cache chrome-sdk --board peach_pit --cwd /b/swarming/w/ir/cache/cbuild/repository/.cache/distfiles/target-master/chrome-src-internal/src --sdk-path /b/swarming/w/ir/cache/cbuild/repository/buildbot_archive/peach_pit-release/R73-11549.0.0 --nogn-gen --nostart-goma --gomadir /b/swarming/w/ir/cache/goma/client --internal --chrome-src /b/swarming/w/ir/cache/cbuild/repository/.cache/distfiles/target-master/chrome-src-internal/src -- /b/swarming/w/ir/cache/cbuild/repository/chromite/bin/deploy_chrome --build-dir /b/swarming/w/ir/cache/cbuild/repository/.cache/distfiles/target-master/chrome-src-internal/src/out_peach_pit/Release --staging-only --staging-dir /b/swarming/w/ir/tmp/t/cbuildbot-tmpeKk8nM/chrome-sdk-stageBIiJfD in /b/swarming/w/ir/cache/cbuild/repository
  21:06:39: DEBUG: Configured cache_dir to '/b/swarming/w/ir/tmp/t/cbuildbot-tmpeKk8nM/chrome-sdk-cacheXpcWoI/cache'
  21:06:40: DEBUG: RunCommand: /b/swarming/w/ir/tmp/t/cbuildbot-tmpeKk8nM/chrome-sdk-cacheXpcWoI/cache/common/gsutil_4.33.tar.gz/gsutil/gsutil -o 'Boto:num_retries=10' ls gs://chromeos-authentication-bucket/
  gs://chromeos-authentication-bucket/README.txt
  21:06:41: WARNING: app-emulation/qemu not found.
  21:06:41: WARNING: sys-firmware/seabios not found.
  21:06:41: WARNING: Could not create Seabios firmware links.
  21:06:41: INFO: Removing symbol_level = 2 from gn args, use --gn-extra-args to specify a non default value.
  21:06:41: INFO: Staging flags taken from USE in environment: highdpi
  21:06:43: WARNING: app-emulation/qemu not found.
  21:06:43: WARNING: sys-firmware/seabios not found.
  21:06:43: WARNING: Could not create Seabios firmware links.
  21:06:45: ERROR: Traceback (most recent call last):
    File "/b/swarming/w/ir/cache/cbuild/repository/chromite/cbuildbot/stages/generic_stages.py", line 723, in Run
      self.PerformStage()
    File "/b/swarming/w/ir/cache/cbuild/repository/chromite/cbuildbot/stages/chrome_stages.py", line 340, in PerformStage
      self._VMTest(sdk_cmd)
    File "/b/swarming/w/ir/cache/cbuild/repository/chromite/cbuildbot/stages/chrome_stages.py", line 309, in _VMTest
      constants.VM_IMAGE_BIN))
    File "/b/swarming/w/ir/cache/cbuild/repository/chromite/cbuildbot/commands.py", line 3134, in VMTest
      assert image_path and os.path.exists(image_path)
  AssertionError

This seems to be new in R73-11549.0.0, R73-11547.0.0 was fine. Both use the same Chrome 73.0.3664.0.
 
Cc: goog...@chromium.org achuith@chromium.org alemate@chromium.org
Labels: -Pri-3 Hotlist-CrOS-Gardener OS-Chrome Pri-1
Owner: achuith@chromium.org
https://crosland.corp.google.com/log/11547.0.0..11549.0.0

This is probably this CL:
https://chromium-review.googlesource.com/c/chromiumos/chromite/+/1381066
Summary: -release: TestSimpleChromeWorkflow - VMTest fails (was: -release: TestSimpleChromeWorkflow)
AFAICT this breaks the PFQ as well, so I'm going to revert.
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 9

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

commit 27a5a4c68a0c933a3e65f25475d2985cf0b709c8
Author: Nicolas Boichat <drinkcat@chromium.org>
Date: Wed Jan 09 07:21:11 2019

Revert "chrome_stages: Add cros_run_vm_test."

This reverts commit ad5955aeec3f9d6d396444d739910cacbfb00c61.

Reason for revert: Breaks -release and PFQ

Original change's description:
> chrome_stages: Add cros_run_vm_test.
> 
> Add cros_run_vm_test to TestSimpleChromeStage.
> 
> BUG= chromium:733394 
> TEST=amd64-generic-chromium-pfq
> 
> Change-Id: I1dac0013179ee12179c3285db8ebe1aa89133292
> Reviewed-on: https://chromium-review.googlesource.com/1381066
> Commit-Ready: Achuith Bhandarkar <achuith@chromium.org>
> Tested-by: Achuith Bhandarkar <achuith@chromium.org>
> Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
> Reviewed-by: Don Garrett <dgarrett@chromium.org>

BUG= chromium:733394 
BUG= chromium:920096 
TEST=N/A

Change-Id: I728ff2895964dea45c74f5be02a6b4dc153fb574
Reviewed-on: https://chromium-review.googlesource.com/c/1401883
Tested-by: Nicolas Boichat <drinkcat@chromium.org>
Trybot-Ready: Nicolas Boichat <drinkcat@chromium.org>
Reviewed-by: Leo Zhang <googleo@google.com>

[modify] https://crrev.com/27a5a4c68a0c933a3e65f25475d2985cf0b709c8/cbuildbot/commands.py
[modify] https://crrev.com/27a5a4c68a0c933a3e65f25475d2985cf0b709c8/cbuildbot/stages/chrome_stages.py

Status: Fixed (was: Untriaged)
Revert fixes this, I'll reland the CL with fixes. Sorry for the inconvenience!
Labels: Build-PFQ-Failures

Sign in to add a comment