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

Issue 732621 link

Starred by 1 user

Issue metadata

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

Blocked on:
issue 730296



Sign in to add a comment

No coverage for cheets_MediaPlayerVideoHWDecodeUsed in the Chrome PFQ and Android PFQ

Reported by jrbarnette@chromium.org, Jun 13 2017

Issue description

The test "cheets_MediaPlayerVideoHWDecodeUsed" recently broke
due to a code change in Chrome, see bug 732192.  The bug passed
the PFQ, and proceeded to break the CQ.

The test is in suite "bvt-cq", but apparently no board that
might have found the bug runs "bvt-cq".

There need to be changes to add appropriate coverage for the
test in the PFQ.  If this can't be done, the test must be
removed from bvt-cq, so that Chrome bugs of this sort won't
break CQ runs.

 

Comment 1 by owenlin@google.com, Jun 13 2017

This bug should be catched from any board with ARC++. I saw many boards that I am sure should be with ARC++ in PFQ, such as, cyan, peach-pit, ....

But when I take a look at the log, e.g., 
https://uberchromegw.corp.google.com/i/chromeos/builders/cyan-chrome-pfq/builds/1251

I don't see any tests running. Will bvt-cq be executed in PFQ at all?

Comment 2 by ihf@chromium.org, Jun 13 2017

Blockedon: 730296
Cc: davidri...@chromium.org owenlin@chromium.org davidjames@chromium.org marc...@chromium.org
Components: Infra>Client>ChromeOS
Labels: OS-Chrome
Owner: akes...@chromium.org
All the veyrons are on the pfq, just like paladins cover them

_paladin_hwtest_assignments = frozenset([
    #bvt-inline        bvt-cq            arc-bvt-cq        family
    ('veyron_mighty',  'veyron_speedy',  'veyron_minnie'), # pinky (N)

Don't know if veyron run hwtests on pfq (actually hard to find in a reasonable amount of time):

  _chrome_pfq_important_boards = frozenset([
      'veyron_minnie',
      'veyron_rialto',

Caroline runs on pfq, but it still is experimental and doesn't run hwtest yet. I would say this is a variant of issue 730296, which means we have a set of fancy builders and hardware pools for different PFQ/CQ/suites, but we don't know if they play well together.

Comment 3 by ihf@chromium.org, Jun 13 2017

I think I found the veyron hwtests. For now more coverage on Chrome PFQ might be obtained by landing
https://chromium-review.googlesource.com/#/c/532594/
https://chromium-review.googlesource.com/#/c/532653/

But this is all very complicated and hard to track.
Owner: ihf@chromium.org
sgtm. assigning to ihf since he's actually got the CLs for this
Cc: akes...@chromium.org
Labels: akeshet-pending-downgrade
ChromeOS Infra P1 Bugscrub.

P1 Bugs in this component should be important enough to get weekly status updates.

Is this already fixed?  -> Fixed
Is this no longer relevant? -> Archived or WontFix
Is this not a P1, based on go/chromeos-infra-bug-slo rubric? -> lower priority.
Is this a Feature Request rather than a bug? Type -> Feature
Is this missing important information or scope needed to decide how to proceed? -> Ask question on bug, possibly reassign.
Does this bug have the wrong owner? -> reassign.

Bugs that remain in this state next week will be downgraded to P2.
Labels: -akeshet-pending-downgrade Pri-2
ChromeOS Infra P1 Bugscrub.

Issue untouched in a week after previous message. Downgrading to P2.
Labels: -Pri-2 Pri-1
Summary: No coverage for cheets_MediaPlayerVideoHWDecodeUsed in the Chrome PFQ and Android PFQ (was: No coverage for cheets_MediaPlayerVideoHWDecodeUsed in the Chrome PFQ)
I think it's better to keep this discussion on one bug rather than two, please split off if you think otherwise.
We had another ~.75 working day outage because the Android PFQ let in bad android this time that broke this test on the CQ: issue 797314
Labels: Chase-Pending
Chase-Pending to decide if we can have a quick CL to add android PFQ coverage.

Comment 10 by ihf@chromium.org, Dec 23 2017

Cc: pprabhu@chromium.org
Sorry, just move it to bvt-arc, it runs in all three places. I won't be able to do this till January. If that is not enough (due to SOC specific feature), we will have some puzzling to do.
Labels: -Chase-Pending Chase
Owner: akes...@chromium.org
Status: Fixed (was: Started)
Project Member

Comment 14 by bugdroid1@chromium.org, Feb 21 2018

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/autotest-cheets/+/675d228933b01440ac4985f6ce262a6f23df0cd0

commit 675d228933b01440ac4985f6ce262a6f23df0cd0
Author: Aviv Keshet <akeshet@chromium.org>
Date: Wed Feb 21 16:44:33 2018

Sign in to add a comment