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

Issue 741230 link

Starred by 2 users

Issue metadata

Status: Archived
Owner:
Closed: Jul 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Partners are confused by WARNING: Passed CTS test message

Project Member Reported by haddowk@chromium.org, Jul 12 2017

Issue description


If a test passes but took more than a single execution of tradefed it issues a warning message, this is confusing to partners as there is nothing for them to do the tests passed

Example warning:

cheets_CTS_N.CtsMediaTestCases	reef-release/R60-9592.52.0/cts_N_Pre/cheets_CTS_N.arm.CtsMediaTestCasesPreconditions	2	Warning	Passed: after 1 retries passing 1395 tests, waived=0. run(t=1395, p=1394, f=1, ne=0, w=0) retry(t=1, p=1, f=0, ne=0, w=0)

Change the CTS tests to provide the option to skip this warning and turn that option on for partner CTS.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 14 2017

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

commit c01a32fdad903f1f2b68d3932c497c52feb252d8
Author: Keith Haddow <haddowk@chromium.org>
Date: Fri Jul 14 02:46:12 2017

[autotest] Remove test warning on tradefed retry.

CPCon and partners do not care about this warning, the test passed.

Reduce clutter and confusion by adding a flag so the partner control
files can switch off this warning.

BUG= chromium:741230 
TEST=local moblab testing

Change-Id: I69ef12be579f7364f964ce6ceb09434675655c68
Reviewed-on: https://chromium-review.googlesource.com/567770
Commit-Ready: Keith Haddow <haddowk@chromium.org>
Tested-by: Keith Haddow <haddowk@chromium.org>
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>
Reviewed-by: Keith Haddow <haddowk@chromium.org>

[modify] https://crrev.com/c01a32fdad903f1f2b68d3932c497c52feb252d8/server/site_tests/cheets_CTS_N/cheets_CTS_N.py

Project Member

Comment 2 by bugdroid1@chromium.org, Jul 14 2017

Labels: merge-merged-release-R60-9592.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/135e4658d5cf3562a88dfa80b11249b65fec8fe7

commit 135e4658d5cf3562a88dfa80b11249b65fec8fe7
Author: Keith Haddow <haddowk@chromium.org>
Date: Fri Jul 14 16:35:13 2017

[autotest] Remove test warning on tradefed retry.

CPCon and partners do not care about this warning, the test passed.

Reduce clutter and confusion by adding a flag so the partner control
files can switch off this warning.

BUG= chromium:741230 
TEST=local moblab testing

Change-Id: I69ef12be579f7364f964ce6ceb09434675655c68
Reviewed-on: https://chromium-review.googlesource.com/567770
Commit-Ready: Keith Haddow <haddowk@chromium.org>
Tested-by: Keith Haddow <haddowk@chromium.org>
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>
Reviewed-by: Keith Haddow <haddowk@chromium.org>
(cherry picked from commit c01a32fdad903f1f2b68d3932c497c52feb252d8)
Reviewed-on: https://chromium-review.googlesource.com/571414
Reviewed-by: Michael Tang <ntang@chromium.org>
Commit-Queue: Keith Haddow <haddowk@chromium.org>

[modify] https://crrev.com/135e4658d5cf3562a88dfa80b11249b65fec8fe7/server/site_tests/cheets_CTS_N/cheets_CTS_N.py

Status: Fixed (was: Assigned)

Comment 4 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment