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

Issue 782055 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocking:
issue 779267



Sign in to add a comment

betty-release VMTestStages should continue with remaining vm tests on error

Project Member Reported by pprabhu@chromium.org, Nov 7 2017

Issue description

Currently, if any of the VMTestStage stages fail, all subsequent vm tests are abandoned.
The VMTests are not related, so we should attempt them all anyway and eventually raise a CompoundError containing all the errors encountered.
 
Blocking: 779267

Comment 2 by derat@chromium.org, Nov 7 2017

Cc: derat@chromium.org pprabhu@chromium.org davidjames@chromium.org dgarr...@chromium.org
 Issue 782058  has been merged into this issue.

Comment 3 by derat@chromium.org, Nov 7 2017

Components: Infra>Client>ChromeOS
Labels: -OS-Mac OS-Chrome
Owner: derat@chromium.org
Status: Started (was: Unconfirmed)
Project Member

Comment 4 by bugdroid1@chromium.org, Nov 8 2017

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

commit 09123e9b5afe6424aa965c6c6369a74447e41e2f
Author: Daniel Erat <derat@chromium.org>
Date: Wed Nov 08 11:12:36 2017

chromite: Ensure TastVMTest runs after VMTest failure.

Update SimpleBuilder to run TastVMTestStage even if
VMTestStage fails beforehand.

BUG= chromium:782055 
TEST=added unit tests. also ran tryjobs to verify that both
     stages run and that results are reported correctly:
     http://uberchromegw/i/chromiumos.tryserver/builders/release/builds/17021
     http://uberchromegw/i/chromiumos.tryserver/builders/release/builds/17024

Change-Id: Ibfd73329044d710b5c6c13e1ecceae84155fb4b0
Reviewed-on: https://chromium-review.googlesource.com/757020
Commit-Ready: Dan Erat <derat@chromium.org>
Tested-by: Dan Erat <derat@chromium.org>
Reviewed-by: Dan Erat <derat@chromium.org>

[modify] https://crrev.com/09123e9b5afe6424aa965c6c6369a74447e41e2f/cbuildbot/builders/simple_builders_unittest.py
[modify] https://crrev.com/09123e9b5afe6424aa965c6c6369a74447e41e2f/cbuildbot/builders/simple_builders.py

Comment 5 by derat@chromium.org, Nov 8 2017

Status: Verified (was: Started)

Sign in to add a comment