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

Issue 796743 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Dec 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 0
Type: Bug



Sign in to add a comment

Partner are unable to run CTS on build M63-10032.75.0

Project Member Reported by haddowk@chromium.org, Dec 20 2017

Issue description

Lenovo reported the issue, has been replicated by Google.

Looks like this CL https://chromium-review.googlesource.com/#/c/chromiumos/third_party/autotest/+/757128/ introduced a dependency on yaml which is not installed on moblab.

Logs from moblab:

12/20 15:41:26.751 INFO |        server_job:0214| 	FAIL	cheets_CTS_N.CtsTransitionTestCases	cheets_CTS_N.CtsTransitionTestCases	timestamp=1513813286	localtime=Dec 20 15:41:26	No module named yaml
  Traceback (most recent call last):
    File "/usr/local/autotest/server/server_job.py", line 919, in group_func
      test.runtest(self, url, tag, args, dargs)
    File "/usr/local/autotest/server/test.py", line 299, in runtest
      *logging_args)
    File "/usr/local/autotest/client/common_lib/test.py", line 869, in runtest
      exec ('import %s' % modulename, local_namespace, global_namespace)
    File "<string>", line 1, in <module>
    File "/usr/local/autotest/server/site_tests/cheets_CTS_N/cheets_CTS_N.py", line 19, in <module>
      from autotest_lib.server.cros import tradefed_test
    File "/usr/local/autotest/server/cros/tradefed_test.py", line 44, in <module>
      from autotest_lib.server.cros import cts_expected_failure_parser
    File "/usr/local/autotest/server/cros/cts_expected_failure_parser.py", line 6, in <module>
      import yaml
  ImportError: No module named yaml

 
Summary: Partner are unable to run CTS on build M63-10032.75.0 (was: Partner are unable to run CTS on build )
Cc: rohi...@chromium.org
Owner: ihf@chromium.org
Assigning to Ilja to revert M63 stable merges.

Ilja can point all the M63 merges and the TPM can create a build pulling out all those CLs.

Comment 3 by ihf@chromium.org, Dec 21 2017

Would it be possible to install yaml on moblab?

Comment 4 by ihf@chromium.org, Dec 21 2017

This is not going to be clean reverts.
Technically possible but I will have to build and push a new version of moblab and go through QA, then get all the partners to update their moblab, so probably would take a day or two.
Project Member

Comment 6 by bugdroid1@chromium.org, Dec 21 2017

Labels: merge-merged-release-R63-10032.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/ec4eaaa86705f3328aeed57cd199999b59b7588b

commit ec4eaaa86705f3328aeed57cd199999b59b7588b
Author: Ilja H. Friedel <ihf@chromium.org>
Date: Thu Dec 21 01:33:48 2017

Revert "cheets_CTS_N: Remove CtsCallLogTestCases from notest-packages."

This reverts commit b9fa058e4f4ad2931db6a7910cba53fc84845b02.

BUG= chromium:796743 

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

[modify] https://crrev.com/ec4eaaa86705f3328aeed57cd199999b59b7588b/server/site_tests/cheets_CTS_N/notest_modules/notest_packages.yaml

Project Member

Comment 7 by bugdroid1@chromium.org, Dec 21 2017

Labels: merge-merged-release-R63-10032.B
The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/autotest-cheets/+/297f0f9f1d23a759dd4db7073bbf8eb623da5915

commit 297f0f9f1d23a759dd4db7073bbf8eb623da5915
Author: Ilja H. Friedel <ihf@google.com>
Date: Thu Dec 21 01:34:17 2017

Project Member

Comment 8 by bugdroid1@chromium.org, Dec 21 2017

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

commit cfcd30157a1779ed31621295f253e7f69b17a87f
Author: Ilja H. Friedel <ihf@chromium.org>
Date: Thu Dec 21 01:34:01 2017

Revert "Add support for handling device/arch/abi specific waivers."

This reverts commit 3aa24409ea679e9968a2cc2efd6efe210997db0c.

BUG= chromium:796743 

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

[delete] https://crrev.com/ec4eaaa86705f3328aeed57cd199999b59b7588b/server/cros/cts_expected_failure_parser.py
[add] https://crrev.com/cfcd30157a1779ed31621295f253e7f69b17a87f/server/site_tests/cheets_GTS/notest_modules/notest_modules.x86
[add] https://crrev.com/cfcd30157a1779ed31621295f253e7f69b17a87f/server/site_tests/cheets_CTS_N/notest_modules/notest_packages.x86
[add] https://crrev.com/cfcd30157a1779ed31621295f253e7f69b17a87f/server/site_tests/cheets_GTS/notest_modules/notest_modules.arm
[add] https://crrev.com/cfcd30157a1779ed31621295f253e7f69b17a87f/server/site_tests/cheets_CTS_N/notest_modules/notest_packages.arm
[delete] https://crrev.com/ec4eaaa86705f3328aeed57cd199999b59b7588b/server/site_tests/cheets_GTS/notest_modules/notest_modules.yaml
[modify] https://crrev.com/cfcd30157a1779ed31621295f253e7f69b17a87f/server/cros/tradefed_test.py
[modify] https://crrev.com/cfcd30157a1779ed31621295f253e7f69b17a87f/server/site_tests/cheets_CTS_N/cheets_CTS_N.py
[delete] https://crrev.com/ec4eaaa86705f3328aeed57cd199999b59b7588b/server/site_tests/cheets_CTS_N/notest_modules/notest_packages.yaml

Comment 9 by ihf@chromium.org, Dec 21 2017

Status: Fixed (was: Assigned)
I hope it works. 
Project Member

Comment 10 by bugdroid1@chromium.org, Dec 21 2017

Labels: merge-merged-stabilize-10032.75.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/2b2be2a4c98318a256162af4133d32572f37870e

commit 2b2be2a4c98318a256162af4133d32572f37870e
Author: Ilja H. Friedel <ihf@chromium.org>
Date: Thu Dec 21 03:34:36 2017

Revert "Add support for handling device/arch/abi specific waivers."

This reverts commit 3aa24409ea679e9968a2cc2efd6efe210997db0c.

BUG= chromium:796743 

Change-Id: I7b68db4cc3f4ffcafbc0ce69d682ec480576217c
Reviewed-on: https://chromium-review.googlesource.com/838409
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>
Tested-by: Ilja H. Friedel <ihf@chromium.org>
(cherry picked from commit cfcd30157a1779ed31621295f253e7f69b17a87f)
Reviewed-on: https://chromium-review.googlesource.com/838519
Reviewed-by: Josafat Garcia <josafat@chromium.org>
Commit-Queue: Josafat Garcia <josafat@chromium.org>
Tested-by: Josafat Garcia <josafat@chromium.org>

[delete] https://crrev.com/a644fd4f66f968e4fffc7c9b405b276270b430de/server/cros/cts_expected_failure_parser.py
[add] https://crrev.com/2b2be2a4c98318a256162af4133d32572f37870e/server/site_tests/cheets_GTS/notest_modules/notest_modules.x86
[add] https://crrev.com/2b2be2a4c98318a256162af4133d32572f37870e/server/site_tests/cheets_CTS_N/notest_modules/notest_packages.x86
[add] https://crrev.com/2b2be2a4c98318a256162af4133d32572f37870e/server/site_tests/cheets_GTS/notest_modules/notest_modules.arm
[add] https://crrev.com/2b2be2a4c98318a256162af4133d32572f37870e/server/site_tests/cheets_CTS_N/notest_modules/notest_packages.arm
[delete] https://crrev.com/a644fd4f66f968e4fffc7c9b405b276270b430de/server/site_tests/cheets_GTS/notest_modules/notest_modules.yaml
[modify] https://crrev.com/2b2be2a4c98318a256162af4133d32572f37870e/server/cros/tradefed_test.py
[modify] https://crrev.com/2b2be2a4c98318a256162af4133d32572f37870e/server/site_tests/cheets_CTS_N/cheets_CTS_N.py
[delete] https://crrev.com/a644fd4f66f968e4fffc7c9b405b276270b430de/server/site_tests/cheets_CTS_N/notest_modules/notest_packages.yaml

Project Member

Comment 11 by bugdroid1@chromium.org, Dec 21 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/autotest-cheets/+/e7d4bec2951ef5b5e749b18aced10d0669af40d5

commit e7d4bec2951ef5b5e749b18aced10d0669af40d5
Author: Ilja H. Friedel <ihf@google.com>
Date: Thu Dec 21 03:39:40 2017

Project Member

Comment 13 by bugdroid1@chromium.org, Dec 23 2017

Project Member

Comment 14 by bugdroid1@chromium.org, Dec 23 2017

Labels: merge-merged-release-R64-10176.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/fa07eaff710b7e44230b4ec1dc690766525b9eec

commit fa07eaff710b7e44230b4ec1dc690766525b9eec
Author: Keith Haddow <haddowk@chromium.org>
Date: Sat Dec 23 01:15:51 2017

[moblab] Install YAML

Some tradefed code uses YAML parser now.

BUG= chromium:796743 
TEST=manual moblab testing

Change-Id: Ia0f8c0f4d3877718a8a2e93b3fd117bbe3ad60fa
Reviewed-on: https://chromium-review.googlesource.com/842242
Reviewed-by: Keith Haddow <haddowk@chromium.org>
Commit-Queue: Keith Haddow <haddowk@chromium.org>
Tested-by: Keith Haddow <haddowk@chromium.org>

[add] https://crrev.com/fa07eaff710b7e44230b4ec1dc690766525b9eec/project-moblab/chromeos-base/chromeos-bsp-moblab/chromeos-bsp-moblab-0.0.5-r75.ebuild
[modify] https://crrev.com/fa07eaff710b7e44230b4ec1dc690766525b9eec/project-moblab/chromeos-base/chromeos-bsp-moblab/chromeos-bsp-moblab-0.0.5.ebuild

Status: Archived (was: Fixed)

Sign in to add a comment