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

Issue 862697 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 17
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: ----



Sign in to add a comment

Developer cannot schedule tests for some mis-configured DUTs.

Project Member Reported by xixuan@chromium.org, Jul 11

Issue description

Example: 

https://ubercautotest.corp.google.com/afe/#tab_id=view_job&object_id=215834749 cannot be scheduled.

It cannot be picked up by the shard due to:
1) it doesn't have any job dependency label like 'board:lulu', OR
2) the DUT chromeos15-row3-rack1-host2 cannot be recognized in shard heartbeat.

For 1), in job creating via afe interface, there's no way to pass in a job dependency label.

For 2), it's because the newly added DUT chromeos15-row3-rack1-host2 (Not sure whether there're others) doesn't have correct static labels stored in afe_static_host_labels.

The server cautotest-prod has 'respect_static_label=False', which means even a DUT is added by 'atest host create', it won't be assigned any static labels.

However, when a job is scheduled to cautotest-prod, in shardheartbeat, it uses static_label to get the job lists that assigned to a shard:
https://cs.corp.google.com/chromeos_public/src/third_party/autotest/files/frontend/afe/models.py?q=classify_label_objects&sq=package:%5Echromeos_(internal%7Cpublic)$&g=0&l=1696

Which makes the above job 215834749 cannot be picked up.

First, let's make shard-heartbeat also respect "respect_static_label" when getting job list.
 
Cc: xixuan@chromium.org harpreet@chromium.org anmarroquin@chromium.org
 Issue 862783  has been merged into this issue.
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 16

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

commit bc4e5a144ce39073927654b109072a282c41ad68
Author: Xixuan Wu <xixuan@chromium.org>
Date: Mon Jul 16 23:32:00 2018

autotest: Make 'assign_to_shard' for jobs respect config respect_static_label.

BUG= chromium:862697 
TEST=None

Change-Id: I0216d18c783a97106d018b3bf5522739d46a2ef9
Reviewed-on: https://chromium-review.googlesource.com/1132128
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Aviv Keshet <akeshet@chromium.org>

[modify] https://crrev.com/bc4e5a144ce39073927654b109072a282c41ad68/frontend/afe/models.py

Fixed?
Status: Fixed (was: Assigned)

Sign in to add a comment