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

Issue 697331 link

Starred by 1 user
Status: Archived
Owner:
davidri...@chromium.org
Closed: Apr 2017
Cc:
█ dshi@chromium.org
akes...@chromium.org
█ shuqianz@chromium.org
davidri...@chromium.org
█ jrbarnette@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Suite job logs including incorrect url for child jobs

Project Member Reported by davidri...@chromium.org, Mar 1 2017 
If the TKO parse step fails, the logs for the HWTest stage include the wrong link for child jobs since it uses the job id from the TKO database instead of the AFE database (which is missing an entry for the job since the TKO parse step failed).  To be more correct/resilient, the code should use the AFE job id if the TKO data is unavailable.

(This showed up when trying to use classifier to find TKO parse errors).

Build:
https://luci-milo.appspot.com/buildbot/chromeos/kevin-release/904

Incorrect stage logs:
  security_ChromiumOSLSM http://cautotest/tko/retrieve_logs.cgi?job=/results/103958969-chromeos-test/

Suite job logs:
https://pantheon.corp.google.com/storage/browser/chromeos-autotest-results/103958969-chromeos-test/hostless/

Correct child job logs:
https://pantheon.corp.google.com/storage/browser/chromeos-autotest-results/103959085-chromeos-test/ (see .parse.log for TKO parse errors)
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 1 2017 

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

commit b7a36ea673676fa03d83159306fcd9497bcd0d1e
Author: Dan Shi <dshi@google.com>
Date: Wed Mar 01 11:19:21 2017

[autotest] Increase tko db connection timeout

Also add more logging to tko parse failure.

BUG= chromium:697331 
TEST=None

Change-Id: Ic1fbf3479a0b156a0bda7817dd09bfca12081fa9
Reviewed-on: https://chromium-review.googlesource.com/448238
Commit-Ready: Dan Shi <dshi@google.com>
Tested-by: Dan Shi <dshi@google.com>
Reviewed-by: David Riley <davidriley@chromium.org>

[modify] https://crrev.com/b7a36ea673676fa03d83159306fcd9497bcd0d1e/tko/db_mysql.py
[modify] https://crrev.com/b7a36ea673676fa03d83159306fcd9497bcd0d1e/tko/parse.py

Comment 2 by aut...@google.com, Mar 1 2017

Owner: davidri...@chromium.org
Assigning to davidriley
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 2 2017 

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

commit dcd1a6461081a028caa2c5beb6c4c573845b6163
Author: David Riley <davidriley@chromium.org>
Date: Thu Mar 02 23:12:44 2017

[autotest] Log correct job id for jobs that fail TKO parse.

BUG= chromium:697331 
TEST=run_suite -m 103958969 --build kevin-release/R58-9325.0.0 --board kevin --suite_name bvt-inline

Change-Id: Ie1226ff37c0bc60d4fde5ee9acbcaa28da157acf
Reviewed-on: https://chromium-review.googlesource.com/448641
Commit-Ready: David Riley <davidriley@chromium.org>
Tested-by: David Riley <davidriley@chromium.org>
Reviewed-by: David Riley <davidriley@chromium.org>

[modify] https://crrev.com/dcd1a6461081a028caa2c5beb6c4c573845b6163/site_utils/run_suite.py
[modify] https://crrev.com/dcd1a6461081a028caa2c5beb6c4c573845b6163/site_utils/run_suite_unittest.py

Comment 4 by davidri...@chromium.org, Apr 25 2017

Status: Fixed (was: Untriaged)

Comment 5 by dchan@google.com, May 30 2017

Labels: VerifyIn-60

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

Labels: VerifyIn-61

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

Status: Archived (was: Fixed)
►

Sign in to add a comment