New issue
Advanced search Search tips

Issue 823438 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

vm_CrosVmStart: pass test and move to bvt-cq

Project Member Reported by ihf@chromium.org, Mar 19 2018

Issue description

Right now it is failing, so I am going to move it to bvt-perbuild.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 19 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/f4213f4f24aa792a821bedec59423f01bfdbf483

commit f4213f4f24aa792a821bedec59423f01bfdbf483
Author: Ilja H. Friedel <ihf@chromium.org>
Date: Mon Mar 19 20:39:32 2018

vm_CrosVmStart: move to bvt-perbuild.

The test is currently not passing.

BUG=chromium:772430, chromium:823438
TEST=https://stainless.corp.google.com/search?test=%5Evm_&exclude_non_release=true&exclude_cts=true&col=build&row=test&view=matrix&first_date=2018-03-13&last_date=2018-03-19

Change-Id: I9fcf30df1ab368f6512460a69414442ae6a48ea3
Reviewed-on: https://chromium-review.googlesource.com/969295
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>
Tested-by: Ilja H. Friedel <ihf@chromium.org>

[modify] https://crrev.com/f4213f4f24aa792a821bedec59423f01bfdbf483/client/site_tests/vm_CrosVmStart/control

Comment 2 by ihf@chromium.org, Mar 19 2018

Status: Assigned (was: Untriaged)
If there is a firmware issue, you will need to work with infra team (probably Richard) to get it updated across all lab devices.
> If there is a firmware issue, you will need to work with infra team [ ... ]

The procedure for dealing with firmware issues is outlined at go/cros-lab-firmware.
That procedure doesn't involve contacting any individual team member.

Project Member

Comment 4 by bugdroid1@chromium.org, Mar 21 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/2161c609e44ff3c4511bd509480bd38489c87d58

commit 2161c609e44ff3c4511bd509480bd38489c87d58
Author: Dylan Reid <dgreid@chromium.org>
Date: Wed Mar 21 05:21:22 2018

vm_CrosVMStart: Fix stopping VM

Use the correct stop command and read from stdin of the start command to
ensure the VM starts before trying to stop it.

BUG=823438
TEST=run this test on eve.

Change-Id: Ib82501971a8b8a6adf99bf90ee9ea57e83be3544
Signed-off-by: Dylan Reid <dgreid@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/972143
Reviewed-by: Stephen Barber <smbarber@chromium.org>

[modify] https://crrev.com/2161c609e44ff3c4511bd509480bd38489c87d58/client/site_tests/vm_CrosVmStart/vm_CrosVmStart.py

Comment 6 by dgreid@chromium.org, Mar 28 2018

66 is expected to be broken. That's OK, the feature is only enabled in 67.

Sign in to add a comment