New issue
Advanced search Search tips

Issue 823357 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----



Sign in to add a comment

blink_perf.owp_storage/blob-perf-shm.html in blink_perf.owp_storage failing on chromium.perf/Android One Perf is consistently flaky

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, Mar 19 2018

Issue description

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

blink_perf.owp_storage/blob-perf-shm.html in blink_perf.owp_storage failing on chromium.perf/Android One Perf

Builders failed on: 
- Android One Perf: 
  https://build.chromium.org/p/chromium.perf/builders/Android%20One%20Perf
TimeoutException: Timed out while waiting 60s for IsJavaScriptExpressionTrue.
Console output:
(warning) http://127.0.0.1:51527/resources/runner.js:313: Timer 'blink_perf.runTest' does not exist
(debug) http://127.0.0.1:51527/resources/runner.js:313: blink_perf.runTest: 202.419921875ms
(debug) http://127.0.0.1:51527/resources/runner.js:313: blink_perf.runTest: 194.72119140625ms
(debug) http://127.0.0.1:51527/resources/runner.js:313: blink_perf.runTest: 184.830078125ms
(debug) http://127.0.0.1:51527/resources/runner.js:313: blink_perf.runTest: 186.89306640625ms
(log) http://127.0.0.1:51527/OWPStorage/resources/blob-shared.js:18: [object ProgressEvent]
(log) http://127.0.0.1:51527/OWPStorage/resources/blob-shared.js:18: Error reading blob: NotReadableError: The requested file could not be read, typically due to permission problems that have occurred after a reference to a file was acquired.
(INFO) 2018-03-18 12:33:57,813 cmd_helper._ValidateAndLogCommand:160  [host]> /b/swarming/w/ir/third_party/catapult/devil/bin/deps/linux2/x86_64/bin/adb -s AG8604420804EGC shell '( ps | grep -F org.chromium.chrome; echo "PIPESTATUS: ${PIPESTATUS[@]}" );echo %$?'
(INFO) 2018-03-18 12:33:57,914 cmd_helper._ValidateAndLogCommand:160  [host]> /b/swarming/w/ir/third_party/catapult/devil/bin/deps/linux2/x86_64/bin/adb -s AG8604420804EGC shell '( ps | grep -F zygote; echo "PIPESTATUS: ${PIPESTATUS[@]}" );echo %$?'
(INFO) 2018-03-18 12:33:58,014 cmd_helper._ValidateAndLogCommand:160  [host]> /b/swarming/w/ir/third_party/catapult/devil/bin/deps/linux2/x86_64/bin/adb -s AG8604420804EGC shell '( ps | grep -F org.chromium.chrome; echo "PIPESTATUS: ${PIPESTATUS[@]}" );echo %$?'
(INFO) 2018-03-18 12:33:58,114 browser.Close:214  Closing browser (pid=6514) ...
(INFO) 2018-03-18 12:33:58,117 cmd_helper._ValidateAndLogCommand:160  [host]> /b/swarming/w/ir/third_party/catapult/devil/bin/deps/linux2/x86_64/bin/adb -s AG8604420804EGC forward --remove tcp:37210
(INFO) 2018-03-18 12:33:58,134 cmd_helper._ValidateAndLogCommand:160  [host]> /b/swarming/w/ir/third_party/catapult/devil/bin/deps/linux2/x86_64/bin/adb -s AG8604420804EGC shell '( ps | grep -F org.chromium.chrome; echo "PIPESTATUS: ${PIPESTATUS[@]}" );echo %$?'
(INFO) 2018-03-18 12:33:58,227 cmd_helper._ValidateAndLogCommand:160  [host]> /b/swarming/w/ir/third_party/catapult/devil/bin/deps/linux2/x86_64/bin/adb -s AG8604420804EGC shell '( am force-stop org.chromium.chrome );echo %$?'
(INFO) 2018-03-18 12:33:58,936 cmd_helper._ValidateAndLogCommand:160  [host]> /b/swarming/w/ir/third_party/catapult/devil/bin/deps/linux2/x86_64/bin/adb -s AG8604420804EGC shell '( ps | grep -F org.chromium.chrome; echo "PIPESTATUS: ${PIPESTATUS[@]}" );echo %$?'
(INFO) 2018-03-18 12:33:59,036 browser.Close:228  Browser is closed.

 

Comment 1 by eyaich@chromium.org, Mar 23 2018

Cc: -dmu...@chromium.org eyaich@chromium.org
Components: Speed>Benchmarks>Waterfall
Owner: dmu...@chromium.org
disabling this test since it is still failing
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 23 2018

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

commit 6ccc199f6f18cfecb64e7fd8ff211865940591a9
Author: Emily Hanley <eyaich@google.com>
Date: Fri Mar 23 17:50:50 2018

Disabling blink_perf.owp_storage on Android one

TBR=ashleymarie@chromium.org
NOTRY=true

Bug:  823357 
Change-Id: Ia44f54405c5b6ad0dd0f3b859753de3cea7c35b4
Reviewed-on: https://chromium-review.googlesource.com/978478
Reviewed-by: Emily Hanley <eyaich@chromium.org>
Commit-Queue: Emily Hanley <eyaich@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545511}
[modify] https://crrev.com/6ccc199f6f18cfecb64e7fd8ff211865940591a9/tools/perf/expectations.config

Comment 3 by dmu...@chromium.org, Apr 11 2018

This is probably fixed now that we fixed a bug that caused blobs to never save stuff on disk.
Project Member

Comment 5 by bugdroid1@chromium.org, Apr 12 2018

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

commit d98b8ea89be482de68499bce531f6e93cdc640a5
Author: Daniel Murphy <dmurph@chromium.org>
Date: Thu Apr 12 23:43:38 2018

Revert "Disabling blink_perf.owp_storage on Android one"

This reverts commit 6ccc199f6f18cfecb64e7fd8ff211865940591a9.

Reason for revert: The blob perf tests for Android should now work due
to us fixing  https://crbug.com/829410 

Original change's description:
> Disabling blink_perf.owp_storage on Android one
> 
> TBR=ashleymarie@chromium.org
> NOTRY=true
> 
> Bug:  823357 
> Change-Id: Ia44f54405c5b6ad0dd0f3b859753de3cea7c35b4
> Reviewed-on: https://chromium-review.googlesource.com/978478
> Reviewed-by: Emily Hanley <eyaich@chromium.org>
> Commit-Queue: Emily Hanley <eyaich@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#545511}

TBR=eyaich@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug:  823357 
Change-Id: Iae4ecaab40550d8dd49c74ef74a75fdaf02926aa
Reviewed-on: https://chromium-review.googlesource.com/1008022
Reviewed-by: Daniel Murphy <dmurph@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550414}
[modify] https://crrev.com/d98b8ea89be482de68499bce531f6e93cdc640a5/tools/perf/expectations.config

Project Member

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

Labels: merge-merged-testbranch
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d98b8ea89be482de68499bce531f6e93cdc640a5

commit d98b8ea89be482de68499bce531f6e93cdc640a5
Author: Daniel Murphy <dmurph@chromium.org>
Date: Thu Apr 12 23:43:38 2018

Revert "Disabling blink_perf.owp_storage on Android one"

This reverts commit 6ccc199f6f18cfecb64e7fd8ff211865940591a9.

Reason for revert: The blob perf tests for Android should now work due
to us fixing  https://crbug.com/829410 

Original change's description:
> Disabling blink_perf.owp_storage on Android one
> 
> TBR=ashleymarie@chromium.org
> NOTRY=true
> 
> Bug:  823357 
> Change-Id: Ia44f54405c5b6ad0dd0f3b859753de3cea7c35b4
> Reviewed-on: https://chromium-review.googlesource.com/978478
> Reviewed-by: Emily Hanley <eyaich@chromium.org>
> Commit-Queue: Emily Hanley <eyaich@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#545511}

TBR=eyaich@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug:  823357 
Change-Id: Iae4ecaab40550d8dd49c74ef74a75fdaf02926aa
Reviewed-on: https://chromium-review.googlesource.com/1008022
Reviewed-by: Daniel Murphy <dmurph@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550414}
[modify] https://crrev.com/d98b8ea89be482de68499bce531f6e93cdc640a5/tools/perf/expectations.config

Comment 7 by dmu...@chromium.org, Apr 17 2018

Status: Fixed (was: Available)

Sign in to add a comment