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

Issue 601861 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Apr 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

Updating both FW_RW and FW_RO failed with "provision_FirmwareUpdate already exists"

Project Member Reported by dshi@chromium.org, Apr 8 2016

Issue description

Both provisions use the same test, provision_FirmwareUpdate, therefore, in the provision special task, they both try to save results to folder provision_FirmwareUpdate. This conflicts with the autoserv run.

We need to force the provision_FirmwareUpdate test to use different result folder name. Or in the worse case, create a new provision_FirmwareROUpdate test for RO update.

 
Making them use different tag arguments of job.run_test() in the control files will save the results to different folders.
As it is not called though the control files, it should be the 'tag' in the extra_kwargs argument of the actionables.TestActionable().
Project Member

Comment 3 by bugdroid1@chromium.org, Apr 12 2016

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

commit 61e407ce852ac90b9b9962347e1d566f9d724e1c
Author: Dan Shi <dshi@google.com>
Date: Fri Apr 08 21:21:07 2016

[autotest] Add tag rw_only to provision_FirmwareUpdate test for RW provision job

This is to make sure provision_FirmwareUpdate for RW only update saves result
to a different folder other than provision_FirmwareUpdate. So a single provision
job can update RO and RW to different build.

BUG= chromium:601861 
TEST=local test

Change-Id: Ief8824c394a4d8d62929a5e42534a6c4887ba5b2
Reviewed-on: https://chromium-review.googlesource.com/338040
Commit-Ready: Dan Shi <dshi@google.com>
Tested-by: Dan Shi <dshi@google.com>
Reviewed-by: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: Dan Shi <dshi@google.com>

[modify] https://crrev.com/61e407ce852ac90b9b9962347e1d566f9d724e1c/server/cros/provision.py

Test to know.

Comment 5 by dshi@chromium.org, Apr 14 2016

Status: Fixed (was: Assigned)

Comment 6 by benhenry@google.com, Apr 27 2016

Components: Infra>Client>ChromeOS
Labels: -Infra-ChromeOS
Status: Verified (was: Fixed)
Closing. please reopen if its not fixed.

Sign in to add a comment