New issue
Advanced search Search tips

Issue 909961 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Jan 18
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocked on:
issue 906540



Sign in to add a comment

rendering.mobile data missing from dashboard

Project Member Reported by sadrul@chromium.org, Nov 29

Issue description

It looks like the rendering.mobile data is missing from the perf dashboards. Example: https://chromeperf.appspot.com/report?sid=1957ad5fdf606ae4475d8cfc55aa80a834abf96d65f443d87fc1ac495b348e6f Notice the latest data for rendering.desktop is recent (Nov 28), whereas for rendering.mobile, the latest data is ~2 weeks old (Nov 16).

Ned pointed out that the tests are running: https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=performance_test_suite&builder=chromium.perf%3Aandroid-nexus5x-perf&sortOrder=forward&sortColumn=test

So perhaps there are some issues with the dashboard not getting the data?
 
Cc: eyaich@chromium.org crouleau@chromium.org
Blockedon: 906540
Cc: benjhayden@chromium.org
Owner: sadrul@chromium.org
I see a few OOM errors in the logs for rendering.mobile. Are there any metrics or histograms or sample diagnostics that can be disabled to shrink the result set?
Cc: chiniforooshan@chromium.org
Owner: benjhayden@chromium.org
+chiniforooshan@

We have removed the TBMv1 metrics, and now report only the TBMv2 metrics. I don't believe there are any redundant metrics we can remove at the moment.

Is there any more details about the OOMs (e.g. what tools, what it is trying to do etc.)? If we were able to reduce the size of pageset, would that help?
"Exceeded soft memory limit of 2048 MB with 2462 MB after servicing 190 requests total".

I'm not sure whether that means that reducing the number of stories by 20% would fix it, but I doubt it.

There's a temporary hack in the dashboard to remove all sample diagnostics. You could do that before uploading, but it still might not help, and it would make it harder to remove the hack later.

One relatively easy fix that would definitely fix the OOMs would be to upload results for one story at a time, plus another upload for the summary timeseries.
I'd be happy to add a flag to add_reserved_diagnostics to write multiple files if you can change the recipes to upload multiple files separately. WDYT?

Comment 6 by seanmccullough@chromium.org, Jan 16 (6 days ago)

Owner: simonhatch@chromium.org

Comment 7 by nedngu...@google.com, Jan 16 (6 days ago)

Cc: -nedngu...@google.com

Comment 8 by benjhayden@google.com, Jan 17 (6 days ago)

See more recent thinking about batching histograms in https://bugs.chromium.org/p/chromium/issues/detail?id=921236

Comment 9 by sadrul@chromium.org, Jan 18 (4 days ago)

Cc: tdres...@chromium.org

Comment 10 by simonhatch@chromium.org, Jan 18 (4 days ago)

Labels: -Pri-1 Pri-2
This seems to be working fine, is there anything to do here?

Comment 11 by crouleau@chromium.org, Jan 18 (4 days ago)

Status: WontFix (was: Assigned)
Looks like there is new data there now, so I guess it got fixed on its own.

Sign in to add a comment