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

Issue 796105 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

telemetry_perf_unittests failing on chromium.linux/Linux Tests (dbg)(1)

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, Dec 19 2017

Issue description

Filed by sheriff-o-matic@appspot.gserviceaccount.com on behalf of dullweber@google.com

telemetry_perf_unittests failing on chromium.linux/Linux Tests (dbg)(1)

Builders failed on: 
- Linux Tests (dbg)(1): 
  https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29


 
Owner: rnep...@chromium.org
rnephew@: telemetry_perf_unittests started failing. https://chromium.googlesource.com/chromium/src/+/e70c64af077f7eb1a0eede0a45d1e8e14abb0ed7 says I should contact you if there is an issue. Could you take a look?
It looks like 'Speedometer2' is failing. I think my converter script for expectations missed it because of some inheritance issues. Unimportant though, the fix is simple.

All that needs to be done is add the expectation here:
https://cs.chromium.org/chromium/src/tools/perf/expectations.config?q=expectations.config&sq=package:chromium&l=211
'
# Benchmark: Speedometer2
crbug.com/792495 [ Linux ] speedometer2/Speedometer2 [ Skip ] 
'

My desktop is packed up and being moved today, so I'm unable to land this change. I could check it out on my laptop, but that might take longer than someone else just doing it and I review the cl. Can someone help? Thanks.
Cc: charliea@chromium.org rnep...@chromium.org
Components: Speed>Telemetry
Owner: nedngu...@google.com
Status: Assigned (was: Available)
I will disable this test
Hmhh, actually the failing test is speedometer2-future

So the real bug is benchmark_smoke_unittest hasn't been hooked with expectations.config file yet
Cc: nedngu...@google.com
Owner: charliea@chromium.org
Thats an easy fix. Should be the same as this cl:
https://chromium-review.googlesource.com/c/chromium/src/+/832381
Still failing on the latest build: https://ci.chromium.org/buildbot/chromium.linux/Linux%20Tests%20%28dbg%29%281%29/69068.

Are we expecting this to recover after a catapult roll?
Labels: -Pri-2 Pri-1
CL for the fix is in the commit queue right now. The CL is Chromium-side only, so no Catapult roll should be required: https://chromium-review.googlesource.com/c/chromium/src/+/834449
Project Member

Comment 9 by bugdroid1@chromium.org, Dec 19 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/378b8d3cafcf8e3cac8e9ed98d496fc2a1ac9db3

commit 378b8d3cafcf8e3cac8e9ed98d496fc2a1ac9db3
Author: Charlie Andrews <charliea@chromium.org>
Date: Tue Dec 19 21:23:18 2017

Makes benchmark_smoke_unittest pull in expectations.config

Without this, the unit test expects benchmarks to pass that are in fact
failing.

Bug:  796105 
Change-Id: I83806d5df5e0c70677eed3ae79b74f393096c03d
Reviewed-on: https://chromium-review.googlesource.com/834449
Reviewed-by: rnephew <rnephew@chromium.org>
Commit-Queue: Charlie Andrews <charliea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#525145}
[modify] https://crrev.com/378b8d3cafcf8e3cac8e9ed98d496fc2a1ac9db3/tools/perf/benchmarks/benchmark_smoke_unittest.py

Status: Fixed (was: Assigned)
This is fixed. The build is green now!

Comment 11 by benhenry@google.com, Jan 16 (6 days ago)

Components: Test>Telemetry

Comment 12 by benhenry@google.com, Jan 16 (6 days ago)

Components: -Speed>Telemetry

Sign in to add a comment