New issue
Advanced search Search tips

Issue 809681 link

Starred by 0 users

Issue metadata

Status: Verified
Owner:
Closed: Feb 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Make AU test control files naming scheme consistent

Project Member Reported by dhadd...@chromium.org, Feb 6 2018

Issue description

Currently we have 

autoupdate_P2P: Runs a delta
autoupdate_ForcedOOBEUpdate: Runs a full payload
autoupdate_ForcedOOBEUpdate.delta: Runs a delta 

And soon to be:
autoupdate_Cellular.full
autoupdate_Cellular.delta 
autoupdate_ForcedOOBEUpdate.cellular: Runs a full 

This will be a mess so I will be make these all consistent 
 
Project Member

Comment 1 by bugdroid1@chromium.org, Feb 9 2018

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

commit 620ccb3ee7127557ad61e060819264541443870a
Author: David Haddock <dhaddock@chromium.org>
Date: Fri Feb 09 07:42:29 2018

Skip autoupdate_P2P when scheduler gives us DUTs from different labs.

Also adding in copyright header and renaming control file to be more
consistent.

BUG= chromium:809681 
BUG=chromium:807495
TEST=autoupdate_P2P.local

Change-Id: Id67580664331b1d9a6f64eb8600ecbb0b8a3af85
Reviewed-on: https://chromium-review.googlesource.com/905808
Commit-Ready: David Haddock <dhaddock@chromium.org>
Tested-by: David Haddock <dhaddock@chromium.org>
Reviewed-by: Amin Hassani <ahassani@chromium.org>

[modify] https://crrev.com/620ccb3ee7127557ad61e060819264541443870a/server/site_tests/autoupdate_P2P/autoupdate_P2P.py
[rename] https://crrev.com/620ccb3ee7127557ad61e060819264541443870a/server/site_tests/autoupdate_P2P/control.delta

Project Member

Comment 2 by bugdroid1@chromium.org, Feb 14 2018

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

commit ac21089415e799768a73a8d31186b6aeb58c8158
Author: David Haddock <dhaddock@chromium.org>
Date: Wed Feb 14 12:07:14 2018

Forced OOBE Update over cellular.

We already have a forced OOBE update test but we want to have a test for
cellular as a few devices will be shipping with LTE.

Working with cellular means the test can no longer use a devserver in the lab.
It cannot verify the update worked successfully by checking the hostlog.

To make this work it requires enabling updates over cellular in the
server test. Then making the payload public so it can be reached over
cellular. Then checking the update engine log for cellular entries and update
successful entries.

The client test will need to switch to cellular, start an omaha instance
and watch the update until just before reboot.

BUG= chromium:809719 
BUG= chromium:809681 
TEST=autoupdate_ForcedOOBEUpdate.cellular.{full|delta} passes
TEST=autoupdate_Cellular still passes
TEST=autoupdate_ForcedOOBEUpdate still passes
Change-Id: Iad1b1aadd6f437aebc8026bb3ce612466fa78fb4
Reviewed-on: https://chromium-review.googlesource.com/911605
Commit-Ready: David Haddock <dhaddock@chromium.org>
Tested-by: David Haddock <dhaddock@chromium.org>
Reviewed-by: Amin Hassani <ahassani@chromium.org>
Reviewed-by: Harpreet Grewal <harpreet@chromium.org>

[modify] https://crrev.com/ac21089415e799768a73a8d31186b6aeb58c8158/server/cros/update_engine/update_engine_test.py
[modify] https://crrev.com/ac21089415e799768a73a8d31186b6aeb58c8158/server/site_tests/autoupdate_Cellular/autoupdate_Cellular.py
[rename] https://crrev.com/ac21089415e799768a73a8d31186b6aeb58c8158/server/site_tests/autoupdate_ForcedOOBEUpdate/control.full
[add] https://crrev.com/ac21089415e799768a73a8d31186b6aeb58c8158/server/site_tests/autoupdate_ForcedOOBEUpdate/control.cellular.delta
[add] https://crrev.com/ac21089415e799768a73a8d31186b6aeb58c8158/server/site_tests/autoupdate_ForcedOOBEUpdate/control.cellular.full
[modify] https://crrev.com/ac21089415e799768a73a8d31186b6aeb58c8158/client/site_tests/autoupdate_StartOOBEUpdate/autoupdate_StartOOBEUpdate.py
[modify] https://crrev.com/ac21089415e799768a73a8d31186b6aeb58c8158/server/site_tests/autoupdate_ForcedOOBEUpdate/autoupdate_ForcedOOBEUpdate.py

Status: Verified (was: Assigned)

Sign in to add a comment