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

Issue 597474 link

Starred by 1 user
Status: Duplicate
Merged: issue 597444
Owner:
█ jrbarnette@chromium.org
Last visit > 30 days ago
Closed: Mar 2016
Cc:
█ kevcheng@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

verify and repair fail for: AttributeError: 'module' object has no attribute 'AutotestHostVerifyError'

Project Member Reported by kevcheng@chromium.org, Mar 24 2016 
Looking at chromeos2-row4-rack7-host3 verify and repair flow, it looks like there'a bug in the flow since the dut seems to be ok but still fails verify/repair.

verify: https://pantheon.corp.google.com/storage/browser/chromeos-autotest-results/hosts/chromeos2-row4-rack7-host3/52711116-verify/debug/?debugUI=DEVELOPERS

repair: https://pantheon.corp.google.com/storage/browser/chromeos-autotest-results/hosts/chromeos2-row4-rack7-host3/52710132-repair/debug/?debugUI=DEVELOPERS

Pertinent snippet:
03/23 17:54:02.795 ERROR|            repair:0284| Failed: The host's TPM is available and working
Traceback (most recent call last):
  File "/usr/local/autotest/client/common_lib/hosts/repair.py", line 281, in _verify_host
    self.verify(host)
  File "/usr/local/autotest/server/hosts/cros_repair.py", line 110, in verify
    raise hosts.AutotestHostVerifyError(
AttributeError: 'module' object has no attribute 'AutotestHostVerifyError'

Comment 1 by jrbarnette@chromium.org, Mar 24 2016

Mergedinto: 597444
Status: Duplicate (was: Untriaged)
The DUT's not actually OK; the verifier tried to raise a failure exception.
However the error message is obscuring the real cause of failure.

Comment 2 by kevcheng@chromium.org, Mar 24 2016 

Is it possible the exception needs to get update the exception raised?

From: https://cs.corp.google.com/chromeos_public/src/third_party/autotest/files/server/hosts/cros_repair.py

for AutotestHostVerifyError -> AutoservVerifyError

Comment 3 by kevcheng@chromium.org, Mar 24 2016 

blargh, bad grammar, let me try again:

Is it possible we need to update the exception being raised from AutotestHostVerifyError to AutoservVerifyError.

Comment 4 by kevcheng@chromium.org, Mar 24 2016 

ah, just saw the duped bug, looks like Xixuan has it.

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

Components: Infra>Client>ChromeOS
Labels: -Infra-ChromeOS
►

Sign in to add a comment