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

Issue 609956 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: May 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Brillo/Android test result parsing is wrong

Project Member Reported by dshi@chromium.org, May 6 2016

Issue description

From the tko record for job id 62443612:

mysql> select * from tko_jobs where afe_job_id=62443612\G; 
*************************** 1. row ***************************
          job_idx: 58342156
              tag: 62443612-chromeos-test/brillo-row1-legorack3-a4db2310
            label: git_mnc-brillo-dev/dragonboard-eng/2842888/brillo-bvt/brillo_Minijail
         username: chromeos-test
      machine_idx: 6398
      queued_time: 2016-05-06 10:11:56
     started_time: 2016-05-06 10:19:27
    finished_time: 2016-05-06 10:20:03
       afe_job_id: 62443612
afe_parent_job_id: 62443597
            build: git_mnc-brillo-dev/dragonboard-eng
    build_version: dragonboard-eng
            suite: 2842888
            board: NULL

The build value is wrong, build id is missing
The suite value is wrong, the value is actually build id
The board value is missing.

It seems there is a bug in parser that fails to parse job name for Brillo tests correctly. This may also lead to wrong data in wmatrix.
 

Comment 1 by sbasi@chromium.org, May 6 2016

Cc: drinkcat@chromium.org

Comment 2 by autumn@chromium.org, May 11 2016

Owner: dshi@chromium.org
Project Member

Comment 3 by bugdroid1@chromium.org, May 16 2016

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

commit ef31f03d7347b40c431754a6280b5ff9eea2903f
Author: Dan Shi <dshi@google.com>
Date: Fri May 13 22:51:39 2016

[autotest] Fix test name parsing logic for Launch Control builds

BUG= chromium:609956 
TEST=unittest

Change-Id: I2707715c2d1c4e34846c713e018527410d28b5b0
Reviewed-on: https://chromium-review.googlesource.com/344752
Commit-Ready: Dan Shi <dshi@google.com>
Tested-by: Dan Shi <dshi@google.com>
Reviewed-by: Kevin Cheng <kevcheng@chromium.org>

[modify] https://crrev.com/ef31f03d7347b40c431754a6280b5ff9eea2903f/server/site_utils_unittest.py
[modify] https://crrev.com/ef31f03d7347b40c431754a6280b5ff9eea2903f/server/site_utils.py

Comment 4 by dshi@chromium.org, May 18 2016

Status: Fixed (was: Untriaged)
Status: Verified (was: Fixed)
Closing. please reopen if its not fixed.

Sign in to add a comment