New issue
Advanced search Search tips

Issue 787251 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: ----



Sign in to add a comment

staging test does not cover run_suite / dynamic suite cases that CQ uses

Project Member Reported by xixuan@chromium.org, Nov 21 2017

Issue description

CQ is broken by suite changes twice:  Issue 787233  & 784564 

It doesn't get tested in test_push.

 
Cc: ayatane@chromium.org
I don't think provision suite is tested in test push at all, that would be a good idea.

auron_paine is experimental, so it shouldn't break the CQ.  It's the only place provision suite is enabled currently.
Labels: -Pri-2 -Chase-Pending Chase Pri-1
Owner: ayatane@chromium.org
Status: Assigned (was: Untriaged)
Summary: staging test does not cover run_suite / dynamic suite cases that CQ uses (was: provision suite is not fully tested in test push.)
 Issue 787233  reveals a gap in staging test.
There are two separate issues

The other one is https://bugs.chromium.org/p/chromium/issues/detail?id=784564

For https://bugs.chromium.org/p/chromium/issues/detail?id=787233, it would be caught by adding provision suite to test push.  That is not needed now since provision suite is not deployed yet.

For https://bugs.chromium.org/p/chromium/issues/detail?id=784564 staging test does not exercise the "check for minimum DUTs" logic.
Project Member

Comment 4 by bugdroid1@chromium.org, Nov 30 2017

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

commit 64edf06917ba24f1ec7c31ceaa467fa7c81b4772
Author: Allen Li <ayatane@chromium.org>
Date: Thu Nov 30 15:09:45 2017

[autotest] Test minimum DUTs logic in test push

BUG= chromium:787251 
TEST=None

Change-Id: Ibb928d74c6111143a92667fd9a9feb174d1ea886
Reviewed-on: https://chromium-review.googlesource.com/792290
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Shuqian Zhao <shuqianz@chromium.org>

[modify] https://crrev.com/64edf06917ba24f1ec7c31ceaa467fa7c81b4772/site_utils/test_push.py

Status: Fixed (was: Assigned)
Status: Archived (was: Fixed)

Sign in to add a comment