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

Issue 811290 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jan 3
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

139.8%-305% regression in blink_perf.owp_storage at 534517:534678

Project Member Reported by nedngu...@google.com, Feb 12 2018

Issue description

This suite goes from 2 minutes to 11 minutes, which is way too much
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Feb 12 2018

All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=811290

(For debugging:) Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?sid=ca7a2e5031a435c14e2b8b64fe520ee3e54f259d9e8c518a7132ccf47bedba8a


Bot(s) for this bug's original alert(s):

android-nexus5
android-nexus5X
android-nexus6
android-nexus7v2
android-one
android-webview-nexus6
Owner: dmu...@chromium.org
Status: Assigned (was: Untriaged)
Daniel: can you look into the benchmark runtime regression?

Comment 4 by dmu...@chromium.org, Feb 12 2018

Status: WontFix (was: Assigned)
These are the new benchmarks being added.
Status: Assigned (was: WontFix)
Daniel: how many tests did you add? Our device budget is very constraint, so it would be great if you can look into reducing the runtime

Comment 6 by dmu...@chromium.org, Feb 12 2018

Ok, I'll reduce the test count from 12 to 4 by removing some of our strategies, and it will definitely bring this time down. When do you need this done by?
This is affecting how soon people can get perf results, so the sooner you can address this the better. Fixing this in 2 weeks is reasonable to me.
Project Member

Comment 8 by 42576172...@developer.gserviceaccount.com, Feb 12 2018

Cc: pwnall@chromium.org dmu...@chromium.org
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/14b3085d840000

[BlobStorage] Adding size configurations to perftests by dmurph@chromium.org
https://chromium.googlesource.com/chromium/src/+/1131391c410a6d01c584c4449accc7e1b1258b06

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions

Comment 9 by pwnall@chromium.org, Feb 12 2018

Status: WontFix (was: Assigned)
I think this is WAI, given that we changed the tests.
Status: Assigned (was: WontFix)
#9: please see #5 & #6

Comment 11 by cos...@gmail.com, Feb 13 2018

Sorry I didn't read the whole thread carefully and thank you for fixing my mistake!
Cc: alexclarke@chromium.org
 Issue 814250  has been merged into this issue.
Status: Fixed (was: Assigned)
Reduced the tests, just didn't put this bug in the cl number. Now we have only 4, and they are pretty small.

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

Components: Test>Telemetry

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

Components: -Speed>Telemetry

Sign in to add a comment