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

Issue 789866 link

Starred by 3 users

Issue metadata

Status: Verified
Owner:
Closed: Jul 2
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Run autotest with static file configuration

Project Member Reported by hiroh@chromium.org, Nov 30 2017

Issue description

This issue is a successive issue of crbug.com/663285.

Currently, the label used in autotest is decided by avtest_label_detect.
However, there are some problems in the method.
1. A device is not powerful enough to decode even though the driver stats the resolution is supported.
2. Camera isn't detected on some DUTs. ( crbug.com/677247 )
3. The supported codec depends on the CPU type even if the device is same. (crbug.com/663285#c33)

The design docs is https://docs.google.com/document/d/1ILLTN9MBG9-6WgzuVmm0FH8US188tjQ52_WACAR-2WU

I collected the decoding/encoding performance data in VDAPerf, PlaybackPerf and VEAPerf tests.
The truly and detailed spec of each driver based on these data is https://docs.google.com/spreadsheets/d/1eT28U8fRUsy4-rl1xS_X5Jv5CuzyNy0IJqxfVlRU6Iw
 

Comment 1 by hiroh@chromium.org, Dec 5 2017

The CL for default labels is crosreview.com/802596 and crosreview.com/805636
I think the below are directories for chipsets.
SundyBridge: overlay-stumpy
Haswell: chipset-hsw
BayTrail(Rabmbi): chipset-byt
Broadwell: chipset-bdw 
Braswell: chipset-bsw
Skylake: chipset-skl
ApolloLake: chipset-apl
KabyLake: chipset-kbl
CannonLake: chipset-cnl
Exynos5250: overlay-daisy
Exynos5420:overlay-peach
Tegra K1: overlay-nyan
RK3288: overlay-veyron
RK3399: chipset-rk3399
MTK8173: chipset-mt8173

What is a directory for IvyBridge?

Comment 2 by hiroh@chromium.org, Dec 8 2017

I uploaded some CLs this week.
autotest-capability: this package installs capabilities /etc/autotest-capability on DUT
crosreview.com/805636
crosreview.com/802596

video_Capability: test for verifying the static label solution.
crosreview.com/814254

I confirm video_Capability run on peach_pit.
In the current implementation, the test outputs results/capability.log.
The output result on peach_pit is the below
dec_h264_720_60
dec_h264_1080_30
dec_h264_1080_60
dec_h264_2160_30
dec_vp8_720_60
dec_vp8_1080_30
dec_vp8_1080_60
dec_jpeg
enc_h264_1080_30
enc_vp8_1080_30
webcam

I am going to address more detailed cases, for instance, SKEW problem and kepler next week.
I am also implementing a monitor program. It acquires video_Capability results on autotest lab and alert if it is changed.

Comment 3 by hiroh@chromium.org, Dec 12 2017

I will add the change for IvyBridge in crosreview.com/805636, and add one for kepler in crosreview.com/821391.

Additionally, I will update my python script in crosreview.com/814254 following up a current yaml grammar.

Comment 4 by hiroh@chromium.org, Dec 12 2017

I and owenlin@ will sync up on Thursday. After that and approved the CLs, I plan to submit these CLs.

Comment 5 by hiroh@chromium.org, Dec 13 2017

I write a small script to visualize a tree structure in overlays. 
The created graphs is attached. (Sorry for awful visual and I'd say it is possibly wrong.)
Some boards have no person, for example, sumo.
The parent of sumo should be chipset-byt.
I also found the parent of parrot-ivb is parrot. parrot-ivb is IvyBridge and parrot is SandyBridge.
In the case, we couldn't probably set the parent of parrot to "chipset-sb."
Of course, as you know, there is no "chipset-sb" now.
overlays.pdf
47.5 KB Download

Comment 6 by hiroh@chromium.org, Dec 13 2017

Blockedon: 794415
Hmm, as I said, it seems I need to sort out these relationship and dependency.
I created the issue tracker for that. crbug.com/794415

Comment 7 by hiroh@chromium.org, Dec 14 2017

We don't and won't have chipset directories for SandyBridge and IvyBridge.
So it is necessary to put chipset and baseboard yaml files in each overlay directory.

SandyBridge:
butterfly, lumpy, parrot, stumpy

IvyBrdige:
link, parrot-ivb, stout


Comment 8 by hiroh@chromium.org, Dec 14 2017

I updated the overlay CLs. crosreview.com/821391, crosreview.com/805636
The current CLs are expected to install the appropriate files to DUT.
I am going to review CLs carefully by myself and test some cases tomorrow.
It is better they will be reviewed by others after I would do.

Comment 9 by hiroh@chromium.org, Dec 14 2017

Self review was done.
The CLs are ready for review.

Comment 10 by hiroh@chromium.org, Dec 15 2017

I noticed IvyBridge supports JPEG decoding, and h264 hw encoding test run on lumpy, stumpy, link and stout.
For link and stout, IvyBridge and SandyBridge supports hw h264 encoding.
I changed as parrot-ivb doesn't support h264 encoding.
As for SandyBridge devices, supporting hw h264 encoding probably depends on SKU.
We actually know lumpy supports hw h264 encoding if its CPU type is intel i3.
Therefore, it would depend on SKU in the case of stumpy as well.
However, I checked stumpy on lab. All the stumpy support hw h264 encoding.
So I deicded to regard stumpy always supports hw h264 encoding.

Please see the following changes for them.
https://chromium-review.googlesource.com/c/chromiumos/overlays/board-overlays/+/805636/10..11
https://chromium-review.googlesource.com/c/chromiumos/overlays/board-overlays/+/821391/5..6

Comment 11 by hiroh@chromium.org, Dec 19 2017

Blockedon: -794415
Components: OS>Kernel>Video

Comment 12 by hiroh@chromium.org, Dec 20 2017

I classified boards in crosreview.com/821391 by baseboard.
[auron]
buddy (samus, auron_yuna, gandof, etc.)
[jecht]
guado, rikku (tidus)
[beltino]
panther, tricky, zako (monroe, mccloud)
[slippy]
peppy, wolf (falco, leon, falco-li)
[?]
stumpy, variant-parrot-ivb, lumpy

The devices in () are ones whose baseboard is the same but don't have the capabilities. 

If I put autotest-capability file in baseboard directory, the capabilities will be installed to all the same baseboard devices.
Thus, I can do so only for the case of baseboard-jecht.

Comment 13 by hiroh@chromium.org, Dec 22 2017

I arranged the capabilty for chipset as following go/crosvideocodec.
Concretely, add hw_enc_h264_1080_30 for SandyBridge and Haswell.
https://chromium-review.googlesource.com/c/chromiumos/overlays/board-overlays/+/805636/14..15
So far, we set hw_enc_h264_1080_30 to Haswell and SandyBridge and IvyBridge devices.
Since I set available by default for them, I needed to change their overlay directories as well in order to keep correspondence.
https://chromium-review.googlesource.com/c/chromiumos/overlays/board-overlays/+/821391/10..11

Apart from that, detectors for kepler and cpu type are moved to overlay directories.
https://chromium-review.googlesource.com/c/chromiumos/overlays/board-overlays/+/842582
https://chromium-review.googlesource.com/c/chromiumos/overlays/chromiumos-overlay/+/802596




Comment 14 by hiroh@chromium.org, Dec 22 2017

It is worth noting that the status of HW H264 encoding in lab on Haswell, IvyBridge and SandyBridge devices on lab.

SandyBridge
[butterfly] x
[lumpy] i5:o, celeron:x
[parrot] x
[stumpy] o

IvyBridge
[link] o
[parrot-ivb] x
[stout] celeron 877: o, celeron 1007U: x

Haswell
[falco] x
[leon] x
[mccloud] o
[monroe] x
[panther] i3, i7: o, celeron: x
[peppy] i3: o, celeron: x
[tricky] i3, i7: o, celeron: x
[wolf] i3, i5: o, celeron: x
[zako] i7: o, celeron: x

Comment 15 by hiroh@chromium.org, Jan 16 2018

I list up CPUs in SandyBridge and IvyBridge and Haswell as many as I got by script.
The result is attached.
cpulist.csv
1.9 KB View Download
Project Member

Comment 16 by bugdroid1@chromium.org, Feb 6 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2

commit f69bcdd731cb1a76ce766b98e9b856608c7ebeb2
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Tue Feb 06 03:08:17 2018

autotest-capability: Add initial change for chipset capabilities

autotest-capability is used to manage the autotest labels applied to the DUTs.
These labels define the capability of the DUTs so that each test can decide
if the test should run
This CL adds the chipset specific capabilities.

BUG= chromium:789866 
TEST=emerge daisy_spring, kevin, cyan, rikku and eve
CQ-DEPEND=CL:802596

Change-Id: I63578a495bbd16a78f4d47045da50b97fcaf7ef2
Reviewed-on: https://chromium-review.googlesource.com/805636
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>

[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-kbl/virtual/autotest-capability/autotest-capability-1.5.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-bsw/chromeos-base/autotest-capability-chipset-bsw/autotest-capability-chipset-bsw-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-byt/virtual/autotest-capability/autotest-capability-1.5.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-variant-parrot-ivb/chromeos-base/autotest-capability-parrot-ivb/files/15-chipset-ivb-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-skl/chromeos-base/autotest-capability-chipset-skl/files/15-chipset-skl-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-peach/chromeos-base/autotest-capability-peach/autotest-capability-peach-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-link/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-stout/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-mt8173/chromeos-base/autotest-capability-chipset-mt8173/autotest-capability-chipset-mt8173-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-stumpy/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-link/chromeos-base/autotest-capability-link/files/15-chipset-ivb-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-link/chromeos-base/autotest-capability-link/autotest-capability-link-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-peach/chromeos-base/autotest-capability-peach/files/15-exynos_5420-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-butterfly/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-bdw/chromeos-base/autotest-capability-chipset-bdw/autotest-capability-chipset-bdw-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-variant-parrot-ivb/chromeos-base/autotest-capability-parrot-ivb/autotest-capability-parrot-ivb-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-butterfly/chromeos-base/autotest-capability-butterfly/files/15-chipset-sdb-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-veyron/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-skl/chromeos-base/autotest-capability-chipset-skl/autotest-capability-chipset-skl-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-hsw/chromeos-base/autotest-capability-chipset-hsw/autotest-capability-chipset-hsw-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-daisy/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-cnl/chromeos-base/autotest-capability-chipset-cnl/autotest-capability-chipset-cnl-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-nyan/chromeos-base/autotest-capability-nyan/autotest-capability-nyan-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-cnl/chromeos-base/autotest-capability-chipset-cnl/files/15-chipset-cnl-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-bdw/virtual/autotest-capability/autotest-capability-1.5.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-variant-parrot-ivb/virtual/autotest-capability/autotest-capability-2.5.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-skl/virtual/autotest-capability/autotest-capability-1.5.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-bsw/chromeos-base/autotest-capability-chipset-bsw/files/15-chipset-bsw-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-apl/virtual/autotest-capability/autotest-capability-1.5.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-bsw/virtual/autotest-capability/autotest-capability-1.5.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-kbl/chromeos-base/autotest-capability-chipset-kbl/files/15-chipset-kbl-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-daisy/chromeos-base/autotest-capability-daisy/files/15-exynos_5250-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-rk3399/chromeos-base/autotest-capability-chipset-rk3399/autotest-capability-chipset-rk3399-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-butterfly/chromeos-base/autotest-capability-butterfly/autotest-capability-butterfly-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-lumpy/chromeos-base/autotest-capability-lumpy/files/15-chipset-sdb-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-kbl/chromeos-base/autotest-capability-chipset-kbl/autotest-capability-chipset-kbl-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-stumpy/chromeos-base/autotest-capability-stumpy/autotest-capability-stumpy-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-lumpy/chromeos-base/autotest-capability-lumpy/autotest-capability-lumpy-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-nyan/chromeos-base/autotest-capability-nyan/files/15-tegra_k1-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-stout/chromeos-base/autotest-capability-stout/autotest-capability-stout-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-byt/chromeos-base/autotest-capability-chipset-byt/autotest-capability-chipset-byt-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-hsw/virtual/autotest-capability/autotest-capability-1.5.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-nyan/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-byt/chromeos-base/autotest-capability-chipset-byt/files/15-chipset-byt-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-cnl/virtual/autotest-capability/autotest-capability-1.5.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-veyron/chromeos-base/autotest-capability-veyron/files/15-rk3288-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-parrot/chromeos-base/autotest-capability-parrot/files/15-chipset-sdb-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-daisy/chromeos-base/autotest-capability-daisy/autotest-capability-daisy-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-parrot/chromeos-base/autotest-capability-parrot/autotest-capability-parrot-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-apl/chromeos-base/autotest-capability-chipset-apl/files/15-chipset-apl-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-apl/chromeos-base/autotest-capability-chipset-apl/autotest-capability-chipset-apl-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-parrot/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-peach/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-stout/chromeos-base/autotest-capability-stout/files/15-chipset-ivb-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-veyron/chromeos-base/autotest-capability-veyron/autotest-capability-veyron-1.0.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-mt8173/virtual/autotest-capability/autotest-capability-1.5.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-lumpy/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-hsw/chromeos-base/autotest-capability-chipset-hsw/files/15-chipset-hsw-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-rk3399/virtual/autotest-capability/autotest-capability-1.5.ebuild
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-mt8173/chromeos-base/autotest-capability-chipset-mt8173/files/15-chipset-mt8173-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-rk3399/chromeos-base/autotest-capability-chipset-rk3399/files/15-chipset-rk3399-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/overlay-stumpy/chromeos-base/autotest-capability-stumpy/files/15-chipset-sdb-capabilities.yaml
[add] https://crrev.com/f69bcdd731cb1a76ce766b98e9b856608c7ebeb2/chipset-bdw/chromeos-base/autotest-capability-chipset-bdw/files/15-chipset-bdw-capabilities.yaml

Project Member

Comment 17 by bugdroid1@chromium.org, Feb 6 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/896fa7a565c25f36531c28bb50b864ca23320637

commit 896fa7a565c25f36531c28bb50b864ca23320637
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Tue Feb 06 03:08:43 2018

autotest-capability: Add initial change for baseboard and board capabilities.

autotest-capability is used to manage the autotest labels applied to the DUTs.
These labels define the capability of the DUTs so that each test can decide
if the test should run.
This CL adds the baseboard and each board specific capabilities.

BUG= chromium:789866 
TEST=emerge daisy_spring, kevin, cyan, rikku and eve

Change-Id: I19fd840e5f3e0e16392744f8ef3d656880ba72c0
Reviewed-on: https://chromium-review.googlesource.com/821391
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>

[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-monroe/chromeos-base/autotest-capability-monroe/autotest-capability-monroe-1.0.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-buddy/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-variant-parrot-ivb/chromeos-base/autotest-capability-parrot-ivb/files/20-parrot-ivb-capabilities.yaml
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-buddy/chromeos-base/autotest-capability-buddy/autotest-capability-buddy-1.0.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-wolf/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-lumpy/chromeos-base/autotest-capability-lumpy/files/20-lumpy-capabilities.yaml
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-guado/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-panther/chromeos-base/autotest-capability-panther/autotest-capability-panther-1.0.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-peppy/chromeos-base/autotest-capability-peppy/files/20-peppy-capabilities.yaml
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-stout/chromeos-base/autotest-capability-stout/files/20-stout-capabilities.yaml
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-tricky/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-panther/chromeos-base/autotest-capability-panther/files/20-panther-capabilities.yaml
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-peppy/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-rikku/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-zako/chromeos-base/autotest-capability-zako/files/20-zako-capabilities.yaml
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-buddy/chromeos-base/autotest-capability-buddy/files/20-buddy-capabilities.yaml
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-rikku/chromeos-base/autotest-capability-rikku/autotest-capability-rikku-1.0.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-guado/chromeos-base/autotest-capability-guado/autotest-capability-guado-1.0.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-parrot/chromeos-base/autotest-capability-parrot/files/20-parrot-capabilities.yaml
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-peppy/chromeos-base/autotest-capability-peppy/autotest-capability-peppy-1.0.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-leon/chromeos-base/autotest-capability-leon/autotest-capability-leon-1.0.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-tricky/chromeos-base/autotest-capability-tricky/autotest-capability-tricky-1.0.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-wolf/chromeos-base/autotest-capability-wolf/files/20-wolf-capabilities.yaml
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-butterfly/chromeos-base/autotest-capability-butterfly/files/20-butterfly-capabilities.yaml
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/baseboard-jecht/chromeos-base/autotest-capability-baseboard-jecht/files/18-baseboard-jecht-capabilities.yaml
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/baseboard-jecht/virtual/autotest-capability/autotest-capability-1.8.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-rikku/chromeos-base/autotest-capability-rikku/files/20-rikku-capabilities.yaml
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-panther/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/baseboard-jecht/chromeos-base/autotest-capability-baseboard-jecht/autotest-capability-baseboard-jecht-1.0.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-tricky/chromeos-base/autotest-capability-tricky/files/20-tricky-capabilities.yaml
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-monroe/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-wolf/chromeos-base/autotest-capability-wolf/autotest-capability-wolf-1.0.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-falco/chromeos-base/autotest-capability-falco/files/20-falco-capabilities.yaml
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-zako/chromeos-base/autotest-capability-zako/autotest-capability-zako-1.0.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-falco/chromeos-base/autotest-capability-falco/autotest-capability-falco-1.0.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-zako/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-leon/chromeos-base/autotest-capability-leon/files/20-leon-capabilities.yaml
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-guado/chromeos-base/autotest-capability-guado/files/20-guado-capabilities.yaml
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-falco/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-monroe/chromeos-base/autotest-capability-monroe/files/20-monroe-capabilities.yaml
[add] https://crrev.com/896fa7a565c25f36531c28bb50b864ca23320637/overlay-leon/virtual/autotest-capability/autotest-capability-2.0.ebuild

Project Member

Comment 18 by bugdroid1@chromium.org, Feb 6 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/903bd9783f2827ddf9f6bda2d76ee8038518013f

commit 903bd9783f2827ddf9f6bda2d76ee8038518013f
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Tue Feb 06 03:08:16 2018

autotest-capability: The initial change to add the project

autotest-capability is used to define the autotest capabilities applied to
the DUTs. Each test can decide should run or skip based on these capabilities.

BUG= chromium:789866 
TEST=emerge daisy_spring, kevin, cyan, rikku and eve

Change-Id: Ica5223e701c9b7ebe1ead7946af5e49e499dd29a
Reviewed-on: https://chromium-review.googlesource.com/802596
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>

[add] https://crrev.com/903bd9783f2827ddf9f6bda2d76ee8038518013f/chromeos-base/autotest-capability-default/files/managed-capabilities.yaml
[add] https://crrev.com/903bd9783f2827ddf9f6bda2d76ee8038518013f/chromeos-base/autotest-capability-default/autotest-capability-default-0.0.1.ebuild
[add] https://crrev.com/903bd9783f2827ddf9f6bda2d76ee8038518013f/virtual/autotest-capability/autotest-capability-1.ebuild
[add] https://crrev.com/903bd9783f2827ddf9f6bda2d76ee8038518013f/chromeos-base/autotest-capability-default/README.md

The above three CLs are a package to install configuration for autotest cappability (i.e. yaml files).
Project Member

Comment 20 by bugdroid1@chromium.org, Feb 8 2018

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

commit 9ab2c288825d4bfbcf4afcc31fd69b88eaee8e05
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Thu Feb 08 09:23:54 2018

Add detectors used in autotest-capability

autotest-capability is used to manage the autotest labels applied to the DUTs.
These labels define the capability of the DUTs so that each test can decide
if the test should run.
The detectors dynamically detects a capability on DUT in autotest-capability,
for example, kepler and CPU type.

BUG= chromium:789866 
TEST=emerge daisy_spring, kevin, cyan, rikku and eve

Change-Id: I1cd3bb12d434959b8e5a61cf9db6f47279e020b0
Reviewed-on: https://chromium-review.googlesource.com/869150
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Kuang-che Wu <kcwu@chromium.org>
Reviewed-by: Owen Lin <owenlin@chromium.org>

[add] https://crrev.com/9ab2c288825d4bfbcf4afcc31fd69b88eaee8e05/client/cros/video/detectors/__init__.py
[add] https://crrev.com/9ab2c288825d4bfbcf4afcc31fd69b88eaee8e05/client/cros/video/detectors/kepler.py
[add] https://crrev.com/9ab2c288825d4bfbcf4afcc31fd69b88eaee8e05/client/cros/video/detectors/intel_cpu.py

Project Member

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

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

commit 0591a9f898942554586d899c62ec6ffc7bd42cda
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Thu Feb 08 17:24:09 2018

video_VideoCapability: add a test for verifying autotest-capablity settings for video features

We are going to use static capabilities in /etc/autotest-capability so that
each autotest decides if the test should be run on the DUT. If the device doesn't
satisfies a capability required by the test, the test is skipped and returns TEST_NA.
video_VideoCapability is a test for verifying autotest-capability settings.
Additionanly, it will be used to monitor capabilities on each DUT in the future.

BUG= chromium:789866 
TEST=video_VideoCapability on stout

Change-Id: Ie6a0e6e3206d53c060a0a4b87c1b648c611e04b2
Reviewed-on: https://chromium-review.googlesource.com/814254
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Kuang-che Wu <kcwu@chromium.org>

[add] https://crrev.com/0591a9f898942554586d899c62ec6ffc7bd42cda/client/cros/video/device_capability.py
[add] https://crrev.com/0591a9f898942554586d899c62ec6ffc7bd42cda/client/site_tests/video_VideoCapability/video_VideoCapability.py
[add] https://crrev.com/0591a9f898942554586d899c62ec6ffc7bd42cda/client/site_tests/video_VideoCapability/control

Autotest to verify static capability configuration (video_VideoCapabilty) was merged.
The dashboard is here. I wait and see the test results for a week or so.
https://cros-goldeneye.corp.google.com/chromeos/healthmonitoring/testDetails?testName=video_VideoCapability
I am going to apply static capability to the actual test in two weeks from now.
The first test candidate is video_VDAPerf.
Project Member

Comment 24 by bugdroid1@chromium.org, Feb 15 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/9769a53ea129f11e18b8c94911637a4722babe86

commit 9769a53ea129f11e18b8c94911637a4722babe86
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Thu Feb 15 10:33:11 2018

autotest-capability: Fixed typo in configuration file

TBR=owenlin@chromium.org

TEST=emerge-guado virtual/autotest-capability
BUG= 789866 

Change-Id: I34c2474b92a9d46b12145f4b35dd9e7884dc3af5
Reviewed-on: https://chromium-review.googlesource.com/920122
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>

[modify] https://crrev.com/9769a53ea129f11e18b8c94911637a4722babe86/overlay-rikku/virtual/autotest-capability/autotest-capability-2.0.ebuild
[modify] https://crrev.com/9769a53ea129f11e18b8c94911637a4722babe86/overlay-guado/virtual/autotest-capability/autotest-capability-2.0.ebuild

Project Member

Comment 25 by bugdroid1@chromium.org, Feb 15 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/0bb9553c91f1815c06f321a97f56e06d1460fc22

commit 0bb9553c91f1815c06f321a97f56e06d1460fc22
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Thu Feb 15 13:42:35 2018

target-chromium-os-{dev/test}: Add autotest-capability ebuild

autotest-capability packages install configurations for media capability
to /etc/autotest-capability.
This CL changes to install them by default to dev/test images.

BUG= chromium:789866 
TEST=emerge-stout virtual/autotest-capability

Change-Id: I2ef3bfb3faf152688cab26c496cd1fa916a30f5c
Reviewed-on: https://chromium-review.googlesource.com/914985
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Owen Lin <owenlin@chromium.org>

[rename] https://crrev.com/0bb9553c91f1815c06f321a97f56e06d1460fc22/virtual/target-chromium-os-dev/target-chromium-os-dev-1-r29.ebuild
[rename] https://crrev.com/0bb9553c91f1815c06f321a97f56e06d1460fc22/virtual/target-chromium-os-test/target-chromium-os-test-1-r62.ebuild
[modify] https://crrev.com/0bb9553c91f1815c06f321a97f56e06d1460fc22/virtual/target-chromium-os-test/target-chromium-os-test-1.ebuild
[modify] https://crrev.com/0bb9553c91f1815c06f321a97f56e06d1460fc22/virtual/target-chromium-os-dev/target-chromium-os-dev-1.ebuild

Project Member

Comment 26 by bugdroid1@chromium.org, Feb 16 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/018d2e73910acf4434c745b04355d733f06cf9bd

commit 018d2e73910acf4434c745b04355d733f06cf9bd
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Fri Feb 16 15:46:13 2018

autotest-tests: Add video_VideoCapability test

Add video_VideoCapability to client autotests so that the test runs by autotest
scheduler.

BUG= chromium:789866 
TEST=emerge-stout chromeos-base/autotest-tests &&
     test_that -b stout ${DUT_IP} video_VideoCapability
CQ-DEPEND=CL:914985

Change-Id: I10aad46b3da0abf4f6e93dea15c5201a296338ff
Reviewed-on: https://chromium-review.googlesource.com/914212
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Owen Lin <owenlin@chromium.org>

[modify] https://crrev.com/018d2e73910acf4434c745b04355d733f06cf9bd/chromeos-base/autotest-tests/autotest-tests-9999.ebuild

Project Member

Comment 27 by bugdroid1@chromium.org, Feb 16 2018

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

commit 24c394c99d0b8aa709941ca97e13c458c0b4d48d
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Fri Feb 16 19:09:32 2018

autotest-capablity: Correct autotest-capability directory

autotest-capability is installed to /usr/local/etc/autotest-capability, by
target-chromium-os-test.

TEST=video_VideoCapability
BUG= chromium:789866 

Change-Id: I7a2ec3624132a59e27f116809f47e05b6cb3f80e
Reviewed-on: https://chromium-review.googlesource.com/923463
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Tomasz Figa <tfiga@chromium.org>

[modify] https://crrev.com/24c394c99d0b8aa709941ca97e13c458c0b4d48d/client/cros/video/device_capability.py

Comment 28 by hiroh@chromium.org, Feb 17 2018

Cc: vsu...@chromium.org
Thanks to the above CLs, video_VideoCapability test runs by autotest scheduler.
The right test results will be shown from 10411.0.0, on which the fixing path CL (comment 28) will be landed.

I uploaded CLs; this autotest-capability is actually applied to our autotests.
I plan the first tests are video_VDAPerf an video_VEAPerf.
The CLs are crosreview.com/923609 and crosreview.com/923610
I plan to merge the CLs after I sufficiently verify autotest capability configuration settings by video_VideoCapability.
Project Member

Comment 29 by bugdroid1@chromium.org, Feb 19 2018

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

commit 2d1f510d51f173d437c450484cc470077ffa5bd3
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Mon Feb 19 05:25:35 2018

video_VideoCapabilty: Fix error message variable and Add test failure message

TEST=None
BUG= chromium:789866 

Change-Id: Ia4c5b70e826bb49ce8f29dff207e09717dcb5fd4
Reviewed-on: https://chromium-review.googlesource.com/925121
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>

[modify] https://crrev.com/2d1f510d51f173d437c450484cc470077ffa5bd3/client/site_tests/video_VideoCapability/video_VideoCapability.py

Comment 30 by hiroh@chromium.org, Feb 20 2018

I summarize the current failure reason.
[bob, kevin]
HW jpeg decoding is not available by avtest_label_detect

[veyron_mickey]
webcam is not detected by avtest_label_detect

[beltino devices except monroe (mccloud, panther, tricky, zako)]
webcam is not detected by avtest_label_detect

[Skylake? (carolline, cave, chell, asuka, sentry)]
hw_enc_jpeg is detected by avtest_label_detect

[guado, rikku]
webcam is not detected by avtest_label_detect
(because I didn't set their parent to jecht.)

[ninja]
webcam is not detected by avtest_label_detect

[Apollolake? (pyro, sand, snappy)]
hw_enc_jpeg is detected by avtest_label_detect

[Braswell? (strago devices)]
hw_enc_jpeg is detected by avtest_label_detect

Comment 31 by hiroh@chromium.org, Feb 20 2018

The CL for fixing the errors in comment 30 is crosreview.com//925864.

Comment 32 by hiroh@chromium.org, Feb 20 2018

I split crosreview.com/925864 to five CLs (crosreview.com/925877 -- crosreview.com/925873) so that reviewers can easily review them.
Project Member

Comment 33 by bugdroid1@chromium.org, Feb 20 2018

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

commit 639fa946020ad03eddaf76d7d5a339de7e37e753
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Tue Feb 20 09:52:14 2018

autotest-capability: Fix regular expression for label in DeviceCapability

Fix regurar expression to go along with '-' contained labels (e.g. hw_dec_vp9-2_1080_30)

BUG= chromium:789866 
TEST=video_VideoCapability on eve

Change-Id: I79b997839a41e3ad75300d9497a57acdd51733ac
Reviewed-on: https://chromium-review.googlesource.com/925868
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Tomasz Figa <tfiga@chromium.org>

[modify] https://crrev.com/639fa946020ad03eddaf76d7d5a339de7e37e753/client/cros/video/device_capability.py

Project Member

Comment 34 by bugdroid1@chromium.org, Feb 20 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/e0170b64de77a48e8f56421107063b1eb2770384

commit e0170b64de77a48e8f56421107063b1eb2770384
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Tue Feb 20 14:45:40 2018

autotest-capability: Modify hw jpeg decoding capability for rk3399

In Rockchip3399 devices, HW jpeg decoding isn't enabled yet.

BUG= chromium:789866 
TEST=emerge-kevin virtual/autotest-capability

Change-Id: I021e7cfc3f8ac1bb300006bf3bd54bceb759be70
Reviewed-on: https://chromium-review.googlesource.com/925874
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Tomasz Figa <tfiga@chromium.org>

[modify] https://crrev.com/e0170b64de77a48e8f56421107063b1eb2770384/chipset-rk3399/chromeos-base/autotest-capability-chipset-rk3399/files/15-chipset-rk3399-capabilities.yaml

Project Member

Comment 35 by bugdroid1@chromium.org, Feb 20 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/4391b30ed47d39ea883e0f66747f8c77ec0ed633

commit 4391b30ed47d39ea883e0f66747f8c77ec0ed633
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Tue Feb 20 14:45:41 2018

autotest-capability: Set guado and rikku's dependency to baseboard-jecht

Because guado and rikku's baseboard is jecht, autotest-capability-jecht
should run in virtual/autotest-capability for guado and riiku.
Otherwise, "no webcam" in autotest-capability-jecht would be lacked.

BUG= chromium:789866 
TEST=emerge-guado virtual/autotest-capability

Change-Id: I0ba2254b3eb08a9163d368b291187bbaba727feb
Reviewed-on: https://chromium-review.googlesource.com/925875
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Tomasz Figa <tfiga@chromium.org>

[modify] https://crrev.com/4391b30ed47d39ea883e0f66747f8c77ec0ed633/overlay-guado/chromeos-base/autotest-capability-guado/autotest-capability-guado-1.0.0.ebuild
[modify] https://crrev.com/4391b30ed47d39ea883e0f66747f8c77ec0ed633/overlay-rikku/chromeos-base/autotest-capability-rikku/autotest-capability-rikku-1.0.0.ebuild

Project Member

Comment 36 by bugdroid1@chromium.org, Feb 20 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/667d2a0d79f74f0949a1401c7bb1c8c678d8c01c

commit 667d2a0d79f74f0949a1401c7bb1c8c678d8c01c
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Tue Feb 20 17:58:30 2018

autotest-capability: beltino devices except monroe don't have camera

BUG= chromium:789866 
TEST=emerge-mccloud virtual/autotest-capability

Change-Id: I63346775974ff1af8e48bcad1bfd6dbd8ca625fc
Reviewed-on: https://chromium-review.googlesource.com/925876
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Tomasz Figa <tfiga@chromium.org>

[add] https://crrev.com/667d2a0d79f74f0949a1401c7bb1c8c678d8c01c/overlay-mccloud/chromeos-base/autotest-capability-mccloud/autotest-capability-mccloud-1.0.0.ebuild
[add] https://crrev.com/667d2a0d79f74f0949a1401c7bb1c8c678d8c01c/overlay-mccloud/virtual/autotest-capability/autotest-capability-2.0.ebuild
[modify] https://crrev.com/667d2a0d79f74f0949a1401c7bb1c8c678d8c01c/overlay-panther/chromeos-base/autotest-capability-panther/files/20-panther-capabilities.yaml
[modify] https://crrev.com/667d2a0d79f74f0949a1401c7bb1c8c678d8c01c/overlay-tricky/chromeos-base/autotest-capability-tricky/files/20-tricky-capabilities.yaml
[add] https://crrev.com/667d2a0d79f74f0949a1401c7bb1c8c678d8c01c/overlay-mccloud/chromeos-base/autotest-capability-mccloud/files/20-mccloud-capabilities.yaml
[modify] https://crrev.com/667d2a0d79f74f0949a1401c7bb1c8c678d8c01c/overlay-zako/chromeos-base/autotest-capability-zako/files/20-zako-capabilities.yaml

Project Member

Comment 37 by bugdroid1@chromium.org, Feb 20 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/2b834daf23992c85db5dc2ecd1d0eff622471e3c

commit 2b834daf23992c85db5dc2ecd1d0eff622471e3c
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Tue Feb 20 17:58:31 2018

autotest-capability: ninja and veryon_mickey don't have camera

BUG= chromium:789866 
TEST=emerge-{veyron_mickey, ninja} virtual/autotest-capability

Change-Id: I2aa02b21aa8d94d943efa30c83afef3de8b5a78c
Reviewed-on: https://chromium-review.googlesource.com/925877
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Tomasz Figa <tfiga@chromium.org>

[add] https://crrev.com/2b834daf23992c85db5dc2ecd1d0eff622471e3c/overlay-ninja/chromeos-base/autotest-capability-ninja/files/20-ninja-capabilities.yaml
[add] https://crrev.com/2b834daf23992c85db5dc2ecd1d0eff622471e3c/overlay-ninja/chromeos-base/autotest-capability-ninja/autotest-capability-ninja-1.0.0.ebuild
[add] https://crrev.com/2b834daf23992c85db5dc2ecd1d0eff622471e3c/overlay-variant-veyron-mickey/chromeos-base/autotest-capability-veyron-mickey/files/20-veyron-mickey-capabilities.yaml
[add] https://crrev.com/2b834daf23992c85db5dc2ecd1d0eff622471e3c/overlay-variant-veyron-mickey/virtual/autotest-capability/autotest-capability-2.5.ebuild
[add] https://crrev.com/2b834daf23992c85db5dc2ecd1d0eff622471e3c/overlay-variant-veyron-mickey/chromeos-base/autotest-capability-veyron-mickey/autotest-capability-veyron-mickey-1.0.0.ebuild
[add] https://crrev.com/2b834daf23992c85db5dc2ecd1d0eff622471e3c/overlay-ninja/virtual/autotest-capability/autotest-capability-2.0.ebuild

Project Member

Comment 38 by bugdroid1@chromium.org, Feb 20 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/90af93018435053ad5b13825b12ab9b1fc9f73db

commit 90af93018435053ad5b13825b12ab9b1fc9f73db
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Tue Feb 20 21:14:29 2018

autotest-capability: Set HW jpeg encoding for Braswell, Apollolake, Skylake, KabyLake, CannonLake

avtets_label_detect currently detects "hw_jpeg_enc_acc" on the devices.
video_VideoCapability fails due to the mismatch.
Even though HW jpeg encoding is not used for now in Chrome(OS), video_JpegEncodeAccelerator and
video_JEAPerf will run on above devices soon.
Adding capability will not affect on the tests right now, since autotest-capability is not applied
to the tests yet.

BUG= chromium:789866 
TEST=emerge-{banon, pyro, asuka, eve} virtual/autotest-capability

Change-Id: I7b676268541b587cf09e8d9a11a05e18db40469b
Reviewed-on: https://chromium-review.googlesource.com/925873
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Tomasz Figa <tfiga@chromium.org>

[modify] https://crrev.com/90af93018435053ad5b13825b12ab9b1fc9f73db/chipset-skl/chromeos-base/autotest-capability-chipset-skl/files/15-chipset-skl-capabilities.yaml
[modify] https://crrev.com/90af93018435053ad5b13825b12ab9b1fc9f73db/chipset-cnl/chromeos-base/autotest-capability-chipset-cnl/files/15-chipset-cnl-capabilities.yaml
[modify] https://crrev.com/90af93018435053ad5b13825b12ab9b1fc9f73db/chipset-apl/chromeos-base/autotest-capability-chipset-apl/files/15-chipset-apl-capabilities.yaml
[modify] https://crrev.com/90af93018435053ad5b13825b12ab9b1fc9f73db/chipset-bsw/chromeos-base/autotest-capability-chipset-bsw/files/15-chipset-bsw-capabilities.yaml
[modify] https://crrev.com/90af93018435053ad5b13825b12ab9b1fc9f73db/chipset-kbl/chromeos-base/autotest-capability-chipset-kbl/files/15-chipset-kbl-capabilities.yaml

Comment 39 Deleted

Comment 40 by hiroh@chromium.org, Feb 21 2018

video_VideoCapability on veyron_jaq failed because of webcam capability mismatch.
avtest_label_detect doesn't detect 'webcam' on chromeos2-row6-rack8-host11.
The HWID and spec is https://www.google.com/chromeos/partner/fe/#hwid:q=JAQ%20D25-Q6R-C4A-A6A-A6K.
All the other veyron_jaq on lab has 'webcam' label.
Even the device whose hw id is the same has 'webcam'. (e.g. chromeos2-row6-rack8-host8)

Is something wrong in avtest_label_detect, driver or the device?
Project Member

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

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

commit cf8789d38fd1f63e52fa423460d69ffe5558be20
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Wed Feb 21 22:19:57 2018

autotest-capability: Remove wrong '|' in regular expression for labels

We don't expect label conatins '|'. Remove '|' from regular expression for
labels in DeviceCapability.

BUG= chromium:789866 
TEST=None

Change-Id: I1364821998a66c1162b8e7ce5493e44cfa9a58ed
Reviewed-on: https://chromium-review.googlesource.com/928059
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Tomasz Figa <tfiga@chromium.org>

[modify] https://crrev.com/cf8789d38fd1f63e52fa423460d69ffe5558be20/client/cros/video/device_capability.py

Comment 42 by hiroh@chromium.org, Feb 22 2018

I will rename the capability for USB camera, from 'webcam' to 'usb_camera'.
The new one is obviously clearer.
Project Member

Comment 43 by bugdroid1@chromium.org, Feb 22 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/ed1c3a61690295e54687fb7583d936662cd2c3d2

commit ed1c3a61690295e54687fb7583d936662cd2c3d2
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Thu Feb 22 12:13:57 2018

autotest-capability: Set no webcam to baseboard-poppy and scarlet

Poppy devices and scarlet don't have USB camera.

BUG= chromium:789866 
TEST=emerge-{poppy, scarlet} virtual/autotest-capability

Change-Id: Ide7ac925726a7758e610f19a1e6c626db16683e1
Reviewed-on: https://chromium-review.googlesource.com/930604
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>

[add] https://crrev.com/ed1c3a61690295e54687fb7583d936662cd2c3d2/baseboard-poppy/chromeos-base/autotest-capability-baseboard-poppy/files/18-baseboard-poppy-capabilities.yaml
[add] https://crrev.com/ed1c3a61690295e54687fb7583d936662cd2c3d2/baseboard-poppy/virtual/autotest-capability/autotest-capability-1.8.ebuild
[add] https://crrev.com/ed1c3a61690295e54687fb7583d936662cd2c3d2/baseboard-fizz/chromeos-base/autotest-capability-baseboard-fizz/files/18-baseboard-fizz-capabilities.yaml
[add] https://crrev.com/ed1c3a61690295e54687fb7583d936662cd2c3d2/overlay-scarlet/chromeos-base/autotest-capability-scarlet/files/20-scarlet-capabilities.yaml
[add] https://crrev.com/ed1c3a61690295e54687fb7583d936662cd2c3d2/overlay-scarlet/virtual/autotest-capability/autotest-capability-2.0.ebuild
[add] https://crrev.com/ed1c3a61690295e54687fb7583d936662cd2c3d2/overlay-scarlet/chromeos-base/autotest-capability-scarlet/autotest-capability-scarlet-1.0.0.ebuild
[add] https://crrev.com/ed1c3a61690295e54687fb7583d936662cd2c3d2/baseboard-fizz/chromeos-base/autotest-capability-baseboard-fizz/autotest-capability-baseboard-fizz-1.0.0.ebuild
[add] https://crrev.com/ed1c3a61690295e54687fb7583d936662cd2c3d2/baseboard-poppy/chromeos-base/autotest-capability-baseboard-poppy/autotest-capability-baseboard-poppy-1.0.0.ebuild
[add] https://crrev.com/ed1c3a61690295e54687fb7583d936662cd2c3d2/baseboard-fizz/virtual/autotest-capability/autotest-capability-1.8.ebuild

Comment 44 by hiroh@chromium.org, Feb 26 2018

Only chromeos6-row2-rack5-host13 (stout) doesn't have 'hw_dec_jpeg'.
I will look into this more.
Project Member

Comment 45 by bugdroid1@chromium.org, Feb 26 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/f01ac6fe6f6958a55c54144c95402f8a92b7050b

commit f01ac6fe6f6958a55c54144c95402f8a92b7050b
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Mon Feb 26 17:32:48 2018

autotest-capability: Rename USB camera capability, from 'webcam' to 'usb_camera'

'usb_camera' is clearer name to stand for USB camera capability.

BUG= chromium:789866 
TEST=emerge-hana virtual/autotest-capability && video_VideoCapability on hana
CQ-DEPEND=CL:930443

Change-Id: If0488d3c14cb1a351f62ae347bce329142f61ed3
Reviewed-on: https://chromium-review.googlesource.com/930424
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Tomasz Figa <tfiga@chromium.org>

[rename] https://crrev.com/f01ac6fe6f6958a55c54144c95402f8a92b7050b/chromeos-base/autotest-capability-default/autotest-capability-default-0.0.2.ebuild
[modify] https://crrev.com/f01ac6fe6f6958a55c54144c95402f8a92b7050b/chromeos-base/autotest-capability-default/files/managed-capabilities.yaml
[modify] https://crrev.com/f01ac6fe6f6958a55c54144c95402f8a92b7050b/chromeos-base/autotest-capability-default/README.md

Project Member

Comment 46 by bugdroid1@chromium.org, Feb 26 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/572b38c9e498eaef4c903aad00d15bbd8b567011

commit 572b38c9e498eaef4c903aad00d15bbd8b567011
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Mon Feb 26 17:32:49 2018

autotest-capability: Rename USB camera capability, from 'webcam' to 'usb_camera'

'usb_camera' is clearer name to stand for USB camera capability.

BUG= chromium:789866 
TEST=emerge-hana virtual/autotest-capability && video_VideoCapability on hana
CQ-DEPEND=CL:930443

Change-Id: Iaf15c11c0f38d46aba5b86e510bb622ca2989df0
Reviewed-on: https://chromium-review.googlesource.com/930643
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Tomasz Figa <tfiga@chromium.org>

[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-bsw/chromeos-base/autotest-capability-chipset-bsw/autotest-capability-chipset-bsw-1.0.1.ebuild
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-panther/chromeos-base/autotest-capability-panther/autotest-capability-panther-1.0.1.ebuild
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-kbl/chromeos-base/autotest-capability-chipset-kbl/autotest-capability-chipset-kbl-1.0.1.ebuild
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-parrot/chromeos-base/autotest-capability-parrot/autotest-capability-parrot-1.0.1.ebuild
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-skl/chromeos-base/autotest-capability-chipset-skl/files/15-chipset-skl-capabilities.yaml
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-lumpy/chromeos-base/autotest-capability-lumpy/autotest-capability-lumpy-1.0.1.ebuild
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-link/chromeos-base/autotest-capability-link/autotest-capability-link-1.0.1.ebuild
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-hsw/chromeos-base/autotest-capability-chipset-hsw/autotest-capability-chipset-hsw-1.0.1.ebuild
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-byt/chromeos-base/autotest-capability-chipset-byt/autotest-capability-chipset-byt-1.0.1.ebuild
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-peach/chromeos-base/autotest-capability-peach/files/15-exynos_5420-capabilities.yaml
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-variant-parrot-ivb/chromeos-base/autotest-capability-parrot-ivb/files/15-chipset-ivb-capabilities.yaml
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-tricky/chromeos-base/autotest-capability-tricky/autotest-capability-tricky-1.0.1.ebuild
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-stumpy/chromeos-base/autotest-capability-stumpy/autotest-capability-stumpy-1.0.1.ebuild
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-butterfly/chromeos-base/autotest-capability-butterfly/files/15-chipset-sdb-capabilities.yaml
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-variant-veyron-mickey/chromeos-base/autotest-capability-veyron-mickey/files/20-veyron-mickey-capabilities.yaml
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-panther/chromeos-base/autotest-capability-panther/files/20-panther-capabilities.yaml
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/baseboard-jecht/chromeos-base/autotest-capability-baseboard-jecht/autotest-capability-baseboard-jecht-1.0.1.ebuild
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-cnl/chromeos-base/autotest-capability-chipset-cnl/files/15-chipset-cnl-capabilities.yaml
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-veyron/chromeos-base/autotest-capability-veyron/autotest-capability-veyron-1.0.1.ebuild
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-butterfly/chromeos-base/autotest-capability-butterfly/autotest-capability-butterfly-1.0.1.ebuild
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-rk3399/chromeos-base/autotest-capability-chipset-rk3399/autotest-capability-chipset-rk3399-1.0.1.ebuild
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-zako/chromeos-base/autotest-capability-zako/autotest-capability-zako-1.0.1.ebuild
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-bsw/chromeos-base/autotest-capability-chipset-bsw/files/15-chipset-bsw-capabilities.yaml
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-apl/chromeos-base/autotest-capability-chipset-apl/autotest-capability-chipset-apl-1.0.1.ebuild
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-ninja/chromeos-base/autotest-capability-ninja/files/20-ninja-capabilities.yaml
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-zako/chromeos-base/autotest-capability-zako/files/20-zako-capabilities.yaml
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-apl/chromeos-base/autotest-capability-chipset-apl/files/15-chipset-apl-capabilities.yaml
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-mt8173/chromeos-base/autotest-capability-chipset-mt8173/autotest-capability-chipset-mt8173-1.0.1.ebuild
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-kbl/chromeos-base/autotest-capability-chipset-kbl/files/15-chipset-kbl-capabilities.yaml
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/baseboard-poppy/chromeos-base/autotest-capability-baseboard-poppy/files/18-baseboard-poppy-capabilities.yaml
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-daisy/chromeos-base/autotest-capability-daisy/files/15-exynos_5250-capabilities.yaml
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-daisy/chromeos-base/autotest-capability-daisy/autotest-capability-daisy-1.0.1.ebuild
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-lumpy/chromeos-base/autotest-capability-lumpy/files/15-chipset-sdb-capabilities.yaml
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-peach/chromeos-base/autotest-capability-peach/autotest-capability-peach-1.0.1.ebuild
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-nyan/chromeos-base/autotest-capability-nyan/files/15-tegra_k1-capabilities.yaml
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-skl/chromeos-base/autotest-capability-chipset-skl/autotest-capability-chipset-skl-1.0.1.ebuild
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-ninja/chromeos-base/autotest-capability-ninja/autotest-capability-ninja-1.0.1.ebuild
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-mccloud/chromeos-base/autotest-capability-mccloud/files/20-mccloud-capabilities.yaml
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-stumpy/chromeos-base/autotest-capability-stumpy/files/15-chipset-sdb-capabilities.yaml
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/baseboard-jecht/chromeos-base/autotest-capability-baseboard-jecht/files/18-baseboard-jecht-capabilities.yaml
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-byt/chromeos-base/autotest-capability-chipset-byt/files/15-chipset-byt-capabilities.yaml
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-veyron/chromeos-base/autotest-capability-veyron/files/15-rk3288-capabilities.yaml
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-variant-parrot-ivb/chromeos-base/autotest-capability-parrot-ivb/autotest-capability-parrot-ivb-1.0.1.ebuild
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-parrot/chromeos-base/autotest-capability-parrot/files/15-chipset-sdb-capabilities.yaml
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-mt8173/chromeos-base/autotest-capability-chipset-mt8173/files/15-chipset-mt8173-capabilities.yaml
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/baseboard-fizz/chromeos-base/autotest-capability-baseboard-fizz/files/18-baseboard-fizz-capabilities.yaml
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-tricky/chromeos-base/autotest-capability-tricky/files/20-tricky-capabilities.yaml
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-nyan/chromeos-base/autotest-capability-nyan/autotest-capability-nyan-1.0.1.ebuild
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-stout/chromeos-base/autotest-capability-stout/files/15-chipset-ivb-capabilities.yaml
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-variant-veyron-mickey/chromeos-base/autotest-capability-veyron-mickey/autotest-capability-veyron-mickey-1.0.1.ebuild
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-link/chromeos-base/autotest-capability-link/files/15-chipset-ivb-capabilities.yaml
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-cnl/chromeos-base/autotest-capability-chipset-cnl/autotest-capability-chipset-cnl-1.0.1.ebuild
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-scarlet/chromeos-base/autotest-capability-scarlet/files/20-scarlet-capabilities.yaml
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-mccloud/chromeos-base/autotest-capability-mccloud/autotest-capability-mccloud-1.0.1.ebuild
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-hsw/chromeos-base/autotest-capability-chipset-hsw/files/15-chipset-hsw-capabilities.yaml
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/baseboard-poppy/chromeos-base/autotest-capability-baseboard-poppy/autotest-capability-baseboard-poppy-1.0.1.ebuild
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/baseboard-fizz/chromeos-base/autotest-capability-baseboard-fizz/autotest-capability-baseboard-fizz-1.0.1.ebuild
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-scarlet/chromeos-base/autotest-capability-scarlet/autotest-capability-scarlet-1.0.1.ebuild
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-bdw/chromeos-base/autotest-capability-chipset-bdw/autotest-capability-chipset-bdw-1.0.1.ebuild
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-rk3399/chromeos-base/autotest-capability-chipset-rk3399/files/15-chipset-rk3399-capabilities.yaml
[rename] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/overlay-stout/chromeos-base/autotest-capability-stout/autotest-capability-stout-1.0.1.ebuild
[modify] https://crrev.com/572b38c9e498eaef4c903aad00d15bbd8b567011/chipset-bdw/chromeos-base/autotest-capability-chipset-bdw/files/15-chipset-bdw-capabilities.yaml

Project Member

Comment 47 by bugdroid1@chromium.org, Feb 26 2018

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

commit eaa9c52b415f3eda73626b9e04fcfd9f1125f49f
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Mon Feb 26 17:32:50 2018

autotest-capability: Rename USB camera capability, from 'webcam' to 'usb_camera'

'usb_camera' is clearer name to stand for USB camera capability.

BUG= chromium:789866 
TEST=video_VideoCapability on hana
CQ-DEPEND=CL:930424, CL:930643

Change-Id: Ie4047c15875bc51211d83867e0ffd0f41b7a91a3
Reviewed-on: https://chromium-review.googlesource.com/930443
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Tomasz Figa <tfiga@chromium.org>

[modify] https://crrev.com/eaa9c52b415f3eda73626b9e04fcfd9f1125f49f/client/site_tests/video_VideoCapability/video_VideoCapability.py

Comment 48 by hiroh@chromium.org, Feb 27 2018

I still need to investigate and perhaps correct webcam availability on some devices, from video_VideoCapability results.
However, it seems the capability for HW video decoding/encoding are correct on devices.
I plan to apply autotest-capability to video_VDAPerf and video_VEAPerf this week, maybe from Thursday.


Project Member

Comment 49 by bugdroid1@chromium.org, Feb 27 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/2e83737271a144b1c13c951e5bab40a53c84fcb8

commit 2e83737271a144b1c13c951e5bab40a53c84fcb8
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Tue Feb 27 13:14:30 2018

autotest-capability: hw jpeg decoding is unavailable on stout whose cpu type is intel celeron 877

BUG= chromium:789866 
TEST=video_VideoCapability

Change-Id: I013e17b8ab2523b03a77f27137b560fa86cf07d6
Reviewed-on: https://chromium-review.googlesource.com/938863
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Tomasz Figa <tfiga@chromium.org>

[modify] https://crrev.com/2e83737271a144b1c13c951e5bab40a53c84fcb8/overlay-stout/chromeos-base/autotest-capability-stout/files/20-stout-capabilities.yaml
[rename] https://crrev.com/2e83737271a144b1c13c951e5bab40a53c84fcb8/overlay-stout/chromeos-base/autotest-capability-stout/autotest-capability-stout-1.0.2.ebuild

Project Member

Comment 50 by bugdroid1@chromium.org, Mar 1 2018

Project Member

Comment 51 by bugdroid1@chromium.org, Mar 3 2018

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

commit 110939e8da81b262da9fe224ada5f4b9c01e01f1
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Sat Mar 03 06:22:32 2018

device_capability: Add func that returns whether a specified capability is available on a device

ensure_capability() throws TestNAError if a specified capability isn't available.
When iterating test cases each of which requires a different capability from
another one, we couldn't run the successive tests once test is broken up by
the exception with ensure_capability().
have_capability() just returns whether a specified capability is avaialbe on the
device. The caller can decide the behavior when the capablity is unavailable.

BUG= chromium:789866 
TEST=video_VDAPerf on veyron_minnie

Change-Id: I88f72e1c532a0f3e9e961822afca24994d482327
Reviewed-on: https://chromium-review.googlesource.com/942764
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Kuang-che Wu <kcwu@chromium.org>
Reviewed-by: Owen Lin <owenlin@chromium.org>

[modify] https://crrev.com/110939e8da81b262da9fe224ada5f4b9c01e01f1/client/cros/video/device_capability.py

Project Member

Comment 52 by bugdroid1@chromium.org, Mar 5 2018

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

commit 4b07d98ad041caa901d0c75169891de2cac51605
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Mon Mar 05 12:56:59 2018

Apply autotest capability to video_VDAPerf

video_VDAPerf is one of the first tests that run with static file configuration.
Because video_VDAPerf suite is crosbolt_perf_perbuild, its failure will not
block CQ, PFQ and canary build.
I set the corrsponding static capability to each test case.
720@50fps : hw_dec_XXX_1080_30
1080@50fps: hw_dec_XXX_1080_60
2160@60fps: hw_dec_XXX_2160_60

In addition to the above, I unify control files to "control".
After this change, "video_VDAPerf" runs autotest following static capability
on DUT.

BUG= chromium:789866 
TEST=video_VDAPerf on veyron_minnie and eve

Change-Id: Ib5d6b0cd1074a5ffbeb1d7595bd71fdecba13235
Reviewed-on: https://chromium-review.googlesource.com/923609
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Kuang-che Wu <kcwu@chromium.org>

[add] https://crrev.com/4b07d98ad041caa901d0c75169891de2cac51605/client/site_tests/video_VDAPerf/control
[modify] https://crrev.com/4b07d98ad041caa901d0c75169891de2cac51605/client/site_tests/video_VDAPerf/video_VDAPerf.py
[delete] https://crrev.com/dfe7697ba0721a9f288949d141c678ad5e0ac046/client/site_tests/video_VDAPerf/control.vp9
[delete] https://crrev.com/dfe7697ba0721a9f288949d141c678ad5e0ac046/client/site_tests/video_VDAPerf/control.vp8
[delete] https://crrev.com/dfe7697ba0721a9f288949d141c678ad5e0ac046/client/site_tests/video_VDAPerf/control.h264
[delete] https://crrev.com/dfe7697ba0721a9f288949d141c678ad5e0ac046/client/site_tests/video_VDAPerf/control.h264.4k
[delete] https://crrev.com/dfe7697ba0721a9f288949d141c678ad5e0ac046/client/site_tests/video_VDAPerf/control.vp8.4k
[delete] https://crrev.com/dfe7697ba0721a9f288949d141c678ad5e0ac046/client/site_tests/video_VDAPerf/control.vp9.4k

I plan the order of autotests to apply autotest-capability.
I list up all the autotests which run with dynamically detected label currently.  
https://docs.google.com/spreadsheets/d/1E8SMBhLyEmnOmYzROX7A98CzDrCW_ao335cp820xiqY

I am going to apply in the following order.
video_VDAPerf (done)
video_VEAPerf
video_PlaybackPerf
video_MediaRecorderPerf
video_VideoSeek
video_HangoutHardwarePerf
video_ChromeVidResChangeHWDecode
video_VDASanity
video_MediaRecorderHWEncodeUsed
video_VideoDecodeAccelerator
video_VideoEncodeAccelerator
----Video Decoding/Encoding Test Migration Done----
video_JDAPerf
video_JEAPerf
video_JpegDecodeAccelerator
video_JpegEncodeAccelerator
----Jpeg Decoding/Encoding Test Migration Done----
video_WebRtcPeerConnectionWithCamera
video_WebRtcCamera
power_CameraSuspend
camera_V4L2
----Camera Test Migration Done----
video_ChromeHWDecodeUsed
video_ChromeRTCHWDecodeUsed
video_ChromeRTCHWEncodeUsed
cheets_MediaPlayerVideoHWDecodeUsed
cheets_CameraOrientation
----CQ Test Migration Done----

Project Member

Comment 54 by bugdroid1@chromium.org, Mar 13 2018

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

commit db685479d7c9d5aa6bfcc235040b92e3e672cc1e
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Tue Mar 13 14:36:54 2018

autotest-capability: Add suites to video_VideoCapability

This changes video_VideoCapability runs in various suites on which media tests run,
except bvt-cq and bvt-arc on which a failure fails CQ and PFQ.

BUG= chromium:789866 
TEST=None

Change-Id: Ie98619d16489a9c1619cd65f8617429de202cc40
Reviewed-on: https://chromium-review.googlesource.com/955866
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>

[modify] https://crrev.com/db685479d7c9d5aa6bfcc235040b92e3e672cc1e/client/site_tests/video_VideoCapability/control

Comment 55 by ihf@chromium.org, Mar 20 2018

Cc: ihf@chromium.org
Labels: OS-Chrome
Project Member

Comment 56 by bugdroid1@chromium.org, Mar 26 2018

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

commit d2234c0718a19067a8d19ab5d102279d7ad4ca76
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Mon Mar 26 11:17:46 2018

Add video_VDAPerf to perf_dashboard_config.json

video_VDAPerf.* are got together to one control file "video_VDAPerf."
There is a need to add video_VDAPerf to perf_dashboard_config.json so as to
show the measured values in chromeperf.

BUG= chromium:789866 
TEST=None

Change-Id: I015e3354555369bcafddf118549440aad73dae39
Reviewed-on: https://chromium-review.googlesource.com/979391
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Chung-yih Wang <cywang@google.com>

[modify] https://crrev.com/d2234c0718a19067a8d19ab5d102279d7ad4ca76/tko/perf_upload/perf_dashboard_config.json

Comment 57 by hiroh@chromium.org, Mar 27 2018

Labels: -Restrict-View-Google
Project Member

Comment 59 by bugdroid1@chromium.org, Apr 18 2018

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

commit 4b831df237548792b79923ef974026d98961f383
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Wed Apr 18 16:30:32 2018

video_VideoCapability: Include mismatched capabilities in error message.

Include mismatched capabilities in error message to easily find what is
mismatched.

BUG= chromium:789866 
TEST=video_VideoCapabiltiy on eve

Change-Id: Ib7cb9aa65822143c64fa5e75d7ececf82569e941
Reviewed-on: https://chromium-review.googlesource.com/1016449
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Tomasz Figa <tfiga@chromium.org>

[modify] https://crrev.com/4b831df237548792b79923ef974026d98961f383/client/site_tests/video_VideoCapability/video_VideoCapability.py

Project Member

Comment 60 by bugdroid1@chromium.org, Apr 23 2018

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

commit fb6b72ef09d3f1ebaf25945bff37ae2091d76fd5
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Mon Apr 23 20:07:13 2018

autotest-capability: Apply to video_VEAPerf

video_VEAPerf is one of the first tests that run with static file configuration.
Because video_VEAPerf suite is crosbolt_perf_perbuild or bvt-perbuild,
its failure will not block CQ, PFQ and canary build.

BUG= chromium:789866 
TEST=video_VEAPerf on veyron_minnie and eve

Change-Id: Id99fe570f111ea74275e043cc940d5b4d5821d2e
Reviewed-on: https://chromium-review.googlesource.com/923610
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Chih-Yu Huang <akahuang@chromium.org>

[modify] https://crrev.com/fb6b72ef09d3f1ebaf25945bff37ae2091d76fd5/client/site_tests/video_VEAPerf/video_VEAPerf.py
[modify] https://crrev.com/fb6b72ef09d3f1ebaf25945bff37ae2091d76fd5/client/site_tests/video_VEAPerf/control.vp8.bvt
[modify] https://crrev.com/fb6b72ef09d3f1ebaf25945bff37ae2091d76fd5/client/site_tests/video_VEAPerf/control.h264
[modify] https://crrev.com/fb6b72ef09d3f1ebaf25945bff37ae2091d76fd5/client/site_tests/video_VEAPerf/control.vp8
[modify] https://crrev.com/fb6b72ef09d3f1ebaf25945bff37ae2091d76fd5/client/site_tests/video_VEAPerf/control.h264.bvt

Project Member

Comment 61 by bugdroid1@chromium.org, Apr 23 2018

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

commit 4014825a063fee7c142c3e9da5e1f13bf9f36904
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Mon Apr 23 20:07:14 2018

autotest-capability: Apply to video_MediaRecorderPerf

BUG= chromium:789866 
TEST=test_that video_MediaRecorderPerf on hana

Change-Id: I93227fc2c5fee6f1968f64bf67f8913eb490bbf8
Reviewed-on: https://chromium-review.googlesource.com/974682
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Sheng-hao Tsao <shenghao@google.com>

[modify] https://crrev.com/4014825a063fee7c142c3e9da5e1f13bf9f36904/client/site_tests/video_MediaRecorderPerf/video_MediaRecorderPerf.py
[modify] https://crrev.com/4014825a063fee7c142c3e9da5e1f13bf9f36904/client/site_tests/video_MediaRecorderPerf/control.h264

Project Member

Comment 65 by bugdroid1@chromium.org, Apr 24 2018

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

commit 307193846acc5a1312e91e164e2ef9c6a40b9d79
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Tue Apr 24 15:39:34 2018

autotest-capability: Apply to video_VDASanity

BUG= chromium:789866 
TEST=test_that video_VDASanity on hana

Change-Id: I76a59dafea52d0f54b69253c9d6483a898431ffc
Reviewed-on: https://chromium-review.googlesource.com/974686
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Kuang-che Wu <kcwu@chromium.org>

[modify] https://crrev.com/307193846acc5a1312e91e164e2ef9c6a40b9d79/client/site_tests/video_VDASanity/control.vp9
[modify] https://crrev.com/307193846acc5a1312e91e164e2ef9c6a40b9d79/client/site_tests/video_VDASanity/video_VDASanity.py

Project Member

Comment 68 by bugdroid1@chromium.org, Apr 25 2018

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

commit c4afab9372b6a48098304c80f7c7bf5b52dcad2c
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Wed Apr 25 01:53:20 2018

Add missing argument to run_once in video_MediaRecorderPerf

Fix a failure caused by CL:974682

BUG= chromium:789866 
TEST=video_MediaRecorderPerf

Change-Id: I9e5077257789f23146cec4b939421d147c9dfca4
Reviewed-on: https://chromium-review.googlesource.com/1025852
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Pin-chih Lin <johnylin@chromium.org>

[modify] https://crrev.com/c4afab9372b6a48098304c80f7c7bf5b52dcad2c/client/site_tests/video_MediaRecorderPerf/video_MediaRecorderPerf.py

So far, I applied autotest-capability to 
video_VDAPerf
video_VEAPerf
video_MediaRecorderPerf
video_VideoSeek
video_HangoutHardwarePerf
video_ChromeVidResChangeHWDecode
video_VDASanity
video_MediaRecorderHWEncodeUsed
video_VideoDecodeAccelerator
video_VideoEncodeAccelerator

All of them are running properly suite correct times.
Project Member

Comment 72 by bugdroid1@chromium.org, May 11 2018

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

commit 471e270f518305ce030e99909d8cc46923f426c1
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Fri May 11 06:18:01 2018

autotest-capability: Apply to video_JEAPerf

BUG= chromium:789866 
TEST=None, since this test is not executable yet in autotest framework

Change-Id: If09fac72b0c9bfc8fa02dfd4f33cd0458ba99098
Reviewed-on: https://chromium-review.googlesource.com/974691
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Sheng-hao Tsao <shenghao@google.com>

[modify] https://crrev.com/471e270f518305ce030e99909d8cc46923f426c1/client/site_tests/video_JEAPerf/video_JEAPerf.py
[modify] https://crrev.com/471e270f518305ce030e99909d8cc46923f426c1/client/site_tests/video_JEAPerf/control

Project Member

Comment 73 by bugdroid1@chromium.org, May 11 2018

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

commit ee79e034ce61611e033587b1675e3b7935b7f4b3
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Fri May 11 06:18:02 2018

autotest-capability: Apply to video_JpegDecodeAccelerator

BUG= chromium:789866 
TEST=test_that video_JpegDecodeAccelerator

Change-Id: I58bf0af3bdbf23d26b1464648a6d776f2b05dc0e
Reviewed-on: https://chromium-review.googlesource.com/974692
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Heng-ruey Hsu <henryhsu@chromium.org>

[modify] https://crrev.com/ee79e034ce61611e033587b1675e3b7935b7f4b3/client/site_tests/video_JpegDecodeAccelerator/control
[modify] https://crrev.com/ee79e034ce61611e033587b1675e3b7935b7f4b3/client/site_tests/video_JpegDecodeAccelerator/video_JpegDecodeAccelerator.py

Project Member

Comment 74 by bugdroid1@chromium.org, May 11 2018

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

commit f20ae3d4fbefdfb5179cf935bec2914aae712787
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Fri May 11 06:18:02 2018

autotest-capability: Apply to video_JpegEncodeAccelerator

BUG= chromium:789866 
TEST=None, since this test is not executable yet in autotest framework

Change-Id: Iaa9bb4273cd14c09138eb903c1867567da126a84
Reviewed-on: https://chromium-review.googlesource.com/974693
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Sheng-hao Tsao <shenghao@google.com>

[modify] https://crrev.com/f20ae3d4fbefdfb5179cf935bec2914aae712787/client/site_tests/video_JpegEncodeAccelerator/control
[modify] https://crrev.com/f20ae3d4fbefdfb5179cf935bec2914aae712787/client/site_tests/video_JpegEncodeAccelerator/video_JpegEncodeAccelerator.py

Project Member

Comment 76 by bugdroid1@chromium.org, May 19 2018

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

commit 7db03cf140fcc5eccb1a04cb28cad5b06e62c486
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Sat May 19 03:08:12 2018

autotest-capability: Apply to video_WebRtcCamera

BUG= chromium:789866 
TEST=test_that video_WebRtcCamera

Change-Id: I50d7496ae10f27c45dbd8b8ada173292b3063881
Reviewed-on: https://chromium-review.googlesource.com/974695
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>

[modify] https://crrev.com/7db03cf140fcc5eccb1a04cb28cad5b06e62c486/client/site_tests/video_WebRtcCamera/control
[modify] https://crrev.com/7db03cf140fcc5eccb1a04cb28cad5b06e62c486/client/site_tests/video_WebRtcCamera/video_WebRtcCamera.py

Project Member

Comment 77 by bugdroid1@chromium.org, May 19 2018

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

commit e70cda94914f75199b0d0eb3b90143ed85cdc15c
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Sat May 19 03:08:13 2018

autotest-capability: Apply to power_CameraSuspend

BUG= chromium:789866 
TEST=test_that power_CameraSuspend

Change-Id: Ie82861cffdcdd0d8e32b277ba0138f575ea04d09
Reviewed-on: https://chromium-review.googlesource.com/974696
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Tomasz Figa <tfiga@chromium.org>

[modify] https://crrev.com/e70cda94914f75199b0d0eb3b90143ed85cdc15c/client/site_tests/power_CameraSuspend/control
[modify] https://crrev.com/e70cda94914f75199b0d0eb3b90143ed85cdc15c/client/site_tests/power_CameraSuspend/power_CameraSuspend.py

Comment 79 by hiroh@chromium.org, May 23 2018

This week, I am going to apply to all the tests in autotest.

Next, I will start applying tests in autotest-test-cheets.
* cheets_MediaPlayerVideoHWDecodeUsed
* cheets_MediaPlayerVideoHWDecodeUsed_P
* cheets_CameraOrientation
* cheets_CameraOrientation_P
* cheets_CameraSuspendResume
Project Member

Comment 80 by bugdroid1@chromium.org, May 23 2018

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

commit d2cdfc21ca2ad8aec5fe74f01a61ccc82f5e4b19
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Wed May 23 13:47:24 2018

Remove useless DEPENDENCIES from video_JDAPerf and video_JEAPerf

In https://crosreview.com/974691 and https://crosreview.com/974690, I forgot
removing DEPENDENCIES from control files.
This deletes the useless DEPENDENCIES and autotest-capability framework will be
used instead.

BUG= chromium:789866 
TEST=video_JDAPerf and video_JEAPerf

Change-Id: Ic912fd340176391de8d7d6de403c1c609a4d9adc
Reviewed-on: https://chromium-review.googlesource.com/1067561
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Sheng-hao Tsao <shenghao@google.com>

[modify] https://crrev.com/d2cdfc21ca2ad8aec5fe74f01a61ccc82f5e4b19/client/site_tests/video_JEAPerf/control
[modify] https://crrev.com/d2cdfc21ca2ad8aec5fe74f01a61ccc82f5e4b19/client/site_tests/video_JDAPerf/control.power

Project Member

Comment 81 by bugdroid1@chromium.org, May 23 2018

Project Member

Comment 82 by bugdroid1@chromium.org, May 29 2018

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

commit e7f1331dd61e58115e0ee59a508e60d3dc0aa1ab
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Tue May 29 05:46:29 2018

autotest-capability: generate capability from media_codecs.xml in android container

ArcDeviceCapability is like DeviceCapability for ARC++.
ARC++ HW capability is based on media_codecs.xml in android container.
ArcDeviceCapability reads media_codecs.xml and generates capabilities based on
that.

BUG= chromium:789866 
TEST=cheets_MediaPlayerVideoHWDecodeUsed.h264

Change-Id: Ia228dc9f1b17b18116041ead234fd7b9843b4265
Reviewed-on: https://chromium-review.googlesource.com/1071491
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Chih-Yu Huang <akahuang@chromium.org>

[add] https://crrev.com/e7f1331dd61e58115e0ee59a508e60d3dc0aa1ab/client/cros/video/arc_device_capability.py

Project Member

Comment 83 by bugdroid1@chromium.org, May 29 2018

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

commit 552861e492138a484a5af7ee54f58d0c3253a10a
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Tue May 29 16:44:49 2018

autotest-capability: Apply to video_ChromeHWDecodeUsed

BUG= chromium:789866 
TEST=test_that video_ChromeHWDecodeUsed

Change-Id: I6be0c566701cd2ddde53a16dd7e99d960fe7d10f
Reviewed-on: https://chromium-review.googlesource.com/974698
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Pin-chih Lin <johnylin@chromium.org>

[modify] https://crrev.com/552861e492138a484a5af7ee54f58d0c3253a10a/client/site_tests/video_ChromeHWDecodeUsed/control.h264
[modify] https://crrev.com/552861e492138a484a5af7ee54f58d0c3253a10a/client/site_tests/video_ChromeHWDecodeUsed/video_ChromeHWDecodeUsed.py
[modify] https://crrev.com/552861e492138a484a5af7ee54f58d0c3253a10a/client/site_tests/video_ChromeHWDecodeUsed/control.h264.mse
[modify] https://crrev.com/552861e492138a484a5af7ee54f58d0c3253a10a/client/site_tests/video_ChromeHWDecodeUsed/control.vp8
[modify] https://crrev.com/552861e492138a484a5af7ee54f58d0c3253a10a/client/site_tests/video_ChromeHWDecodeUsed/control.vp9
[modify] https://crrev.com/552861e492138a484a5af7ee54f58d0c3253a10a/client/site_tests/video_ChromeHWDecodeUsed/control.vp9.mse
[modify] https://crrev.com/552861e492138a484a5af7ee54f58d0c3253a10a/client/site_tests/video_ChromeHWDecodeUsed/control.vp8.mse

Project Member

Comment 84 by bugdroid1@chromium.org, May 30 2018

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

commit b317fae3b7e48cf4e0ddaa9af18e6af44d1e48a9
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Wed May 30 19:50:33 2018

Project Member

Comment 86 by bugdroid1@chromium.org, May 30 2018

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

commit dddfcef4fb9b817e0270299f76bab500d4cfc906
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Wed May 30 19:50:28 2018

autotest-capability: Apply to video_ChromeRTCHWEncodeUsed

BUG= chromium:789866 
TEST=test_that video_ChromeRTCHWEncodeUsed

Change-Id: I8a11e80fd6686bcbe905c52a3bc954276d6953a9
Reviewed-on: https://chromium-review.googlesource.com/974700
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Chih-Yu Huang <akahuang@chromium.org>

[modify] https://crrev.com/dddfcef4fb9b817e0270299f76bab500d4cfc906/client/site_tests/video_ChromeRTCHWEncodeUsed/control
[modify] https://crrev.com/dddfcef4fb9b817e0270299f76bab500d4cfc906/client/site_tests/video_ChromeRTCHWEncodeUsed/video_ChromeRTCHWEncodeUsed.py

Confirmed all the tests run properly so far.
Project Member

Comment 88 by bugdroid1@chromium.org, Jun 5 2018

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

commit f98b8bdba015f83d32a13b113e73162a97c774cc
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Tue Jun 05 12:42:18 2018

video_PlaybackPerf: Remove 4k_video DEPENDCIES

video_PlaybackPerf measures end-to-end Chrome video playback performance with
both software and hardware decoder.
Hardware decoder has device capability (i.e. some devices do not support 4K
video hardware decode). However, for software decoder, it has no such
limitation. Hence, all devices can play 4K video regardless performance.
Currently, the test skips 4K video test if the device does not support hardware
decoding for 4k video. However, it should still measure software decoder
performance so that we can monitor decoders' performance and see if Chrome
video stack changes improve or degrade performance.
The cost of this change is the increment of test running time. And this shall be
the cost worth to spend.

By enabling us to monitor playback performance results, we can understand
playback peroformance on Chrome OS device as of today, and improve Chrome video
stack if chance.

BUG= chromium:789866 
TEST=4k video_PlaybackPerf on minnie and eve

Change-Id: I829371d8caf13b54125a7652c099ef2f1b5b0dcf
Reviewed-on: https://chromium-review.googlesource.com/1077872
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Pin-chih Lin <johnylin@chromium.org>

[modify] https://crrev.com/f98b8bdba015f83d32a13b113e73162a97c774cc/client/site_tests/video_PlaybackPerf/control.vp8.4k
[modify] https://crrev.com/f98b8bdba015f83d32a13b113e73162a97c774cc/client/site_tests/video_PlaybackPerf/control.h264.4k
[modify] https://crrev.com/f98b8bdba015f83d32a13b113e73162a97c774cc/client/site_tests/video_PlaybackPerf/control.vp9.4k.60
[modify] https://crrev.com/f98b8bdba015f83d32a13b113e73162a97c774cc/client/site_tests/video_PlaybackPerf/control.vp9.4k

Comment 89 by hiroh@chromium.org, Jun 11 2018

At beta channel, I don't apply autotest-capability to video_ChromeRTCHWEncodeUsed.
We enable HW VP8 encoding at dev channel.
Therefore, video_ChromeRTCHWEncodeUsed fails in the following step.
(1) at dev branch, avtest_label_detect hw_enc_vp8_acc
(2) at beta branch, the test looks the label, hw_enc_vp8_acc, on DUT and judge that the test can run on the device.
(3) the test is provided and runs. 
(4) However, in the beta branch image, HW vp8 encoding is disabled and the test fails as a result.

We should fix this issue in some way.
The simplest way is to apply autotest-capability to beta branch.
As a result, even if the test is provided, it checks static-capability in beta image. There is no HW VP8 encoding capability in the list of beta image, and the test is skipped (not failed).

I created cherry-pick CL.
https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/1094934

Comment 90 by hiroh@chromium.org, Jun 11 2018

Labels: Merge-Request-68
Hi, TPM. Could you see #c89 and approve the cherry-pick CL?

Thanks.
Project Member

Comment 91 by sheriffbot@chromium.org, Jun 11 2018

Labels: -Merge-Request-68 Hotlist-Merge-Review Merge-Review-68
This bug requires manual review: M68 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: cmasso@(Android), kariahda@(iOS), bhthompson@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Hotlist-Merge-Review -Merge-Review-68 Merge-Approved-68

Comment 93 by hiroh@chromium.org, Jun 11 2018

Labels: -Merge-Approved-68
Project Member

Comment 94 by bugdroid1@chromium.org, Jun 11 2018

Labels: merge-merged-release-R68-10718.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/cbf94206ba016f3f336019eddb4db7a91bb89dc8

commit cbf94206ba016f3f336019eddb4db7a91bb89dc8
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Mon Jun 11 22:54:58 2018

autotest-capability: Apply to video_ChromeRTCHWEncodeUsed

BUG= chromium:789866 
TEST=test_that video_ChromeRTCHWEncodeUsed

Change-Id: I8a11e80fd6686bcbe905c52a3bc954276d6953a9
Reviewed-on: https://chromium-review.googlesource.com/974700
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Chih-Yu Huang <akahuang@chromium.org>
(cherry picked from commit dddfcef4fb9b817e0270299f76bab500d4cfc906)
Reviewed-on: https://chromium-review.googlesource.com/1094934
Trybot-Ready: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>
Commit-Queue: Hirokazu Honda <hiroh@chromium.org>

[modify] https://crrev.com/cbf94206ba016f3f336019eddb4db7a91bb89dc8/client/site_tests/video_ChromeRTCHWEncodeUsed/control
[modify] https://crrev.com/cbf94206ba016f3f336019eddb4db7a91bb89dc8/client/site_tests/video_ChromeRTCHWEncodeUsed/video_ChromeRTCHWEncodeUsed.py

Comment 95 by hiroh@chromium.org, Jun 13 2018

Labels: -merge-merged-release-R68-10718.B
Project Member

Comment 96 by bugdroid1@chromium.org, Jun 15 2018

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

commit 4f7cd5abeb9a035ac320af25bd65ee11ede94373
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Fri Jun 15 17:56:52 2018

Project Member

Comment 97 by bugdroid1@chromium.org, Jun 15 2018

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

commit 458d65c351d8c50f2c38fc833be1e57018fec4e3
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Fri Jun 15 17:56:53 2018

Comment 98 by hiroh@chromium.org, Jun 20 2018

Two server tests are still remained that uses the dynamic labels.
server/site_tests/video_PowerConsumption
server/site_tests/video_VDAStress
Project Member

Comment 99 by bugdroid1@chromium.org, Jun 30 2018

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

commit 7c3c321a9baacd5ee28105c6c62c3cbc9279a22d
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Sat Jun 30 00:50:23 2018

autotest-capability: Apply video_PowerConsumption

There are two contro files using label.
control.hw_hangout.vp8, the used labels are replaced by autotest-capability labels.
contro.webrtc, video_WebRtcPerf is expected to run with and without hardware
acceleration. Therefore, the label is not required at all. Remove it, not
replaced by autotest-capability labels.

BUG= chromium:789866 
TEST=video_PowerConsumption

Change-Id: I2ac7bb7aa9d7956c3e78e000b0bd7dc4da0b23d4
Reviewed-on: https://chromium-review.googlesource.com/1111765
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Pin-chih Lin <johnylin@chromium.org>

[modify] https://crrev.com/7c3c321a9baacd5ee28105c6c62c3cbc9279a22d/server/site_tests/video_PowerConsumption/control.hw_hangout.vp8
[modify] https://crrev.com/7c3c321a9baacd5ee28105c6c62c3cbc9279a22d/server/site_tests/video_PowerConsumption/control.webrtc

Status: Verified (was: Started)
video_VDAStress doesn't run on lab.
So video_PowerConsumption is the last test to run on lab using a dynamic label.
There is no remained task about autotest capability framework.

Some work is required to run video_VDAStress. It is separated from this work and then I created crbug.com/855970 for the task.

Sign in to add a comment