New issue
Advanced search Search tips

Issue 874511 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 30
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Stainless does not show "Wifi_Matfunc" suite test results starting 08/11/2018 for coral board

Project Member Reported by dsunk...@chromium.org, Aug 15

Issue description

Summary: Stainless does not show "Wifi_Matfunc" suite test results starting 08/11/2018 for coral board (was: Stainless does not show "Wifi_Matfunc" suite test results starting 08/11/2018)
Owner: xixuan@chromium.org
same is the case with board: nami.
board: fizz has results displayed in stainless with models kench & teemo
There's a bug in suite-scheduler:

https://cs.corp.google.com/chromeos_public/infra/suite_scheduler/cloud_sql_client.py?q=cloud_sql_&sq=package:%5Echromeos_(internal%7Cpublic)$&g=0&l=136

When it tries to get builds which only pass 'hwtest [sanity]', it specifies 'hwtest [sanity]' in column bs.name, but for coral, the stage name includes the model name:

| HWTest [sanity] [blacktip]                   | pass    |
| HWTest [sanity] [blacktip360]                | pass    |
| HWTest [sanity] [bruce]                      | pass    |
| HWTest [sanity] [epaulette]                  | pass    |
| HWTest [sanity] [lava]                       | pass    |
| HWTest [sanity] [nasher]                     | pass    |
| HWTest [sanity] [nasher360]                  | pass    |
| HWTest [sanity] [robo]                       | pass    |
| HWTest [sanity] [robo360]                    | pass    |
| HWTest [sanity] [santa]                      | pass    |
| HWTest [sanity] [whitetip]                   | pass    |

Which makes some builds which passes sanity stage (https://cros-goldeneye.corp.google.com/chromeos/healthmonitoring/buildDetails?buildbucketId=8938254668875464624) are not scheduled.
Status: Assigned (was: Available)
sorry, no results for coral yet.
Status: Started (was: Assigned)
Project Member

Comment 9 by bugdroid1@chromium.org, Aug 29

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/infra/suite_scheduler/+/384f91339e439b3fc970532cf86c5ab4eacef048

commit 384f91339e439b3fc970532cf86c5ab4eacef048
Author: Xixuan Wu <xixuan@chromium.org>
Date: Tue Aug 28 22:48:41 2018

autotest: Reduce the duplicated rows from cidb for relaxed builds.

The previous sql used for fetching relaxed builds can give us about 20k
rows, most of which are duplicated. This CL is to reduce it.

BUG= chromium:874511 
TEST=None

Change-Id: I9b35b096f6b0ed4760509571f0f05b87c83622f1

[modify] https://crrev.com/384f91339e439b3fc970532cf86c5ab4eacef048/cloud_sql_client.py
[modify] https://crrev.com/384f91339e439b3fc970532cf86c5ab4eacef048/build_lib.py

Project Member

Comment 10 by bugdroid1@chromium.org, Aug 29

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/infra/suite_scheduler/+/8d2f2864f0161feec1c248521bd57994d97cee0b

commit 8d2f2864f0161feec1c248521bd57994d97cee0b
Author: Xixuan Wu <xixuan@chromium.org>
Date: Wed Aug 29 16:29:23 2018

Involve model information in build info.

This is to verify whether a hwtest sanity stage for a specific
model is passed. If so, only kick off suites on this model.

BUG= chromium:874511 
TEST=None

Change-Id: I8b0a0bef877ed2588533ddf3a732672af03c8f8c

[modify] https://crrev.com/8d2f2864f0161feec1c248521bd57994d97cee0b/cloud_sql_client.py
[modify] https://crrev.com/8d2f2864f0161feec1c248521bd57994d97cee0b/task_unittest.py
[modify] https://crrev.com/8d2f2864f0161feec1c248521bd57994d97cee0b/trigger_receiver_unittest.py
[modify] https://crrev.com/8d2f2864f0161feec1c248521bd57994d97cee0b/task.py
[modify] https://crrev.com/8d2f2864f0161feec1c248521bd57994d97cee0b/build_lib.py

Verified suite ArcGtsDevNightly in coral:

[ArcGtsDevNightly]
owner: chromeos-test-team@google.com
run_on: nightly
hour: 10
suite: arc-gts
pool: cts
# Only these boards have DUTs assigned to pool:cts
boards: asuka, auron_paine, auron_yuna, banon, bob, caroline, cave, celes, chell, coral, cyan, edgar, elm, enguarde, eve, fizz, gandof, gnawty, grunt, hana, kefka, kevin, lars, lulu, nami, nautilus, pyro, reef, reks, relm, samus, sand, scarlet, sentry, setzer, soraka, terra, ultima, veyron_fievel, veyron_jerry, veyron_mighty, veyron_minnie, veyron_speedy, veyron_tiger, wizpig
branch_specs: ==tot
priority: 24
timeout: 72
force: True
only_hwtest_sanity_required: True

Here's the kicked off suites for each model:

https://chromeos-proxy.appspot.com/tasklist?c=name&c=state&c=created_ts&c=duration&c=pending_time&c=pool&c=bot&et=1535566920000&f=pool%3Asuite-scheduler&f=build-tag%3Acoral-release%2FR70-11015.0.0&l=50&n=true&s=created_ts%3Adesc&st=1535480520000




Status: Fixed (was: Started)
https://chromeos-proxy.appspot.com/tasklist?c=name&c=state&c=created_ts&c=duration&c=pending_time&c=pool&c=bot&et=1535664300000&f=pool%3Asuite-scheduler&f=board-tag%3Acoral&f=suite-tag%3Awifi_matfunc&l=50&n=true&s=created_ts%3Adesc&st=1535577900000

Mark it as fixed.

There's a problem in AFE that the same suites for different models are using the same name. But it belongs to another bug.

Sign in to add a comment