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

Issue 710923 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

suite_enumerator.py is broken, no 'stable_tests'

Project Member Reported by waihong@chromium.org, Apr 12 2017

Issue description

It failed like:

$ ./site_utils/suite_enumerator.py faft_lv1
ERROR:root:Could not import tzlocal.
Traceback (most recent call last):
  File "./site_utils/suite_enumerator.py", line 91, in <module>
    sys.exit(main())
  File "./site_utils/suite_enumerator.py", line 76, in main
    PRETEST_LIST, suite.stable_tests):
AttributeError: 'Suite' object has no attribute 'stable_tests'

Probably caused by the recent work on server/cros/dynamic_suite/suite.py.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 29 2017

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

commit 00280b9fd4e1f94290b75d2a5c1ea7d17c284ea5
Author: Mike Mason <michael.w.mason@intel.com>
Date: Sat Apr 29 05:25:43 2017

[autotest] Adapt suite_enumerator.py to changes in suite.py

Changes in server/cros/dynamic_suite/suite.py broke the feature
in suite_enumerator.py that prints the tests in a suite. This patch
fixes that, calls create_fs_getter and list_all_suites outside of
the Suite class (calling in the class is deprecated), and removes
checks for experimental tests, which are no longer supported.

BUG=b/36296378,chromium:710923
TEST=./suite_enumerator.py -a ../ graphics

Change-Id: Ic11249d4b52bafb3a7656f2d51d1921908d5062d
Signed-off-by: Mike Mason <michael.w.mason@intel.com>
Reviewed-on: https://chromium-review.googlesource.com/486219
Commit-Ready: Richard Barnette <jrbarnette@chromium.org>
Tested-by: Michael W Mason <michael.w.mason@intel.com>
Reviewed-by: Richard Barnette <jrbarnette@chromium.org>
Reviewed-by: Allen Li <ayatane@chromium.org>

[modify] https://crrev.com/00280b9fd4e1f94290b75d2a5c1ea7d17c284ea5/site_utils/suite_enumerator.py

Status: Fixed (was: Untriaged)

Comment 3 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

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

Status: Archived (was: Fixed)

Sign in to add a comment