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

Issue 681839 link

Starred by 1 user
Status: Duplicate
Merged: issue 679768
Owner: ----
Closed: Jan 2017
Cc:
█ nednguyen@chromium.org
charliea@chromium.org
█ eakuefner@chromium.org
sullivan@chromium.org
primiano@chromium.org
u...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

long_running:tools:gmail-foreground is producing too much trace data

Project Member Reported by skyos...@chromium.org, Jan 17 2017 
Example failure:

https://chromium-swarm.appspot.com/user/task/33c2c947684a3010

INFO:root:Trace sizes in bytes: {TraceDataPart("traceEvents"): 310432045, TraceDataPart("telemetry"): 125422, TraceDataPart("tabIds"): 40}
[ RUN      ] /tmp/tmppclwT5.html
TraceImportError: Invalid string length
    at Array.join (native)
    at Object.f [as string] (/usr/local/google/home/skyostil/tmp/foo/third_party/catapult/tracing/third_party/jszip/jszip.min.js:12:20494)
    at Object.c.transformTo (/usr/local/google/home/skyostil/tmp/foo/third_party/catapult/tracing/third_party/jszip/jszip.min.js:12:22247)
    at Object.c.transformTo (/usr/local/google/home/skyostil/tmp/foo/third_party/catapult/tracing/third_party/jszip/jszip.min.js:12:6414)
    at Function.GzipImporter.transformToString (/tracing/extras/importer/gzip_importer.html:141:26)
    at Function.GzipImporter.inflateGzipData_ (/tracing/extras/importer/gzip_importer.html:116:31)
    at GzipImporter.extractSubtraces (/tracing/extras/importer/gzip_importer.html:177:36)
    at Import.createImports (/tracing/importer/import.html:138:40)
    at Task.run (/tracing/base/task.html:71:21)
    at Function.Task.RunSynchronously (/tracing/base/task.html:152:25)
[  FAILED  ] /tmp/tmppclwT5.html (16289 ms)
Traceback (most recent call last):
  File "/usr/local/google/home/skyostil/tmp/foo/third_party/catapult/telemetry/telemetry/value/failure.py", line 41, in _GetExcInfoFromMessage
    raise Exception(message)
Exception: TraceImportError: Invalid string length
    at Array.join (native)
    at Object.f [as string] (/usr/local/google/home/skyostil/tmp/foo/third_party/catapult/tracing/third_party/jszip/jszip.min.js:12:20494)
    at Object.c.transformTo (/usr/local/google/home/skyostil/tmp/foo/third_party/catapult/tracing/third_party/jszip/jszip.min.js:12:22247)
    at Object.c.transformTo (/usr/local/google/home/skyostil/tmp/foo/third_party/catapult/tracing/third_party/jszip/jszip.min.js:12:6414)
    at Function.GzipImporter.transformToString (/tracing/extras/importer/gzip_importer.html:141:26)
    at Function.GzipImporter.inflateGzipData_ (/tracing/extras/importer/gzip_importer.html:116:31)
    at GzipImporter.extractSubtraces (/tracing/extras/importer/gzip_importer.html:177:36)
    at Import.createImports (/tracing/importer/import.html:138:40)
    at Task.run (/tracing/base/task.html:71:21)
    at Function.Task.RunSynchronously (/tracing/base/task.html:152:25)

Comment 1 by perezju@chromium.org, Jan 17 2017

Cc: eakuefner@chromium.org sullivan@chromium.org
+Ethan, +Annie.

Comment 2 by perezju@chromium.org, Jan 17 2017

Cc: u...@chromium.org
+Ulan as benchmark owner.

Comment 3 by sullivan@chromium.org, Jan 17 2017

Cc: nednguyen@chromium.org charliea@chromium.org
Components: Tests>Telemetry
Project Member

Comment 4 by bugdroid1@chromium.org, Jan 17 2017 

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

commit 33818797373a6eff577677cebd2f659a1328d053
Author: skyostil <skyostil@chromium.org>
Date: Tue Jan 17 16:39:38 2017

perf: Disable long_running:tools:gmail-foreground

This benchmark is producing too much trace data, causing results
processing to fail.

BUG= 681839 

Review-Url: https://codereview.chromium.org/2635983004
Cr-Commit-Position: refs/heads/master@{#444069}

[modify] https://crrev.com/33818797373a6eff577677cebd2f659a1328d053/tools/perf/page_sets/system_health/long_running_stories.py

Comment 5 by eostroukhov@chromium.org, Jan 30 2017

Components: -Platform>DevTools>Memory

Comment 6 by nedngu...@google.com, Jan 30 2017

Components: Internals>Tracing

Comment 7 by nedngu...@google.com, Jan 30 2017

Mergedinto: 679768
Status: Duplicate (was: Untriaged)
Project Member

Comment 8 by bugdroid1@chromium.org, May 22 2017 

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

commit 961a33a21ebca4823dd6e2578bdcb70013b6e4fc
Author: Charlie Andrews <charliea@chromium.org>
Date: Mon May 22 19:49:56 2017

Reenable stories disabled due to chrome:tracing trace size limitations

These issues should be resolved by
https://codereview.chromium.org/2864743002.

NOTRY=true
TBR=perezju@chromium.org
BUG= 704197 , 657433,  681839 

Change-Id: I8cbd766867ce43ca8c4e5cfb3d4a1856e01b93b5
Reviewed-on: https://chromium-review.googlesource.com/508231
Reviewed-by: Charlie Andrews <charliea@chromium.org>
Reviewed-by: Juan Antonio Navarro Pérez <perezju@chromium.org>
Commit-Queue: Charlie Andrews <charliea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#473661}
[modify] https://crrev.com/961a33a21ebca4823dd6e2578bdcb70013b6e4fc/testing/buildbot/chromium.perf.fyi.json
[modify] https://crrev.com/961a33a21ebca4823dd6e2578bdcb70013b6e4fc/testing/buildbot/chromium.perf.json
[modify] https://crrev.com/961a33a21ebca4823dd6e2578bdcb70013b6e4fc/tools/perf/benchmarks/v8_browsing.py
[modify] https://crrev.com/961a33a21ebca4823dd6e2578bdcb70013b6e4fc/tools/perf/page_sets/system_health/browsing_stories.py
[modify] https://crrev.com/961a33a21ebca4823dd6e2578bdcb70013b6e4fc/tools/perf/page_sets/system_health/long_running_stories.py
[modify] https://crrev.com/961a33a21ebca4823dd6e2578bdcb70013b6e4fc/tools/perf/page_sets/system_health/multi_tab_stories.py
►

Sign in to add a comment