New issue
Advanced search Search tips

Issue 630701 link

Starred by 3 users

Issue metadata

Status: Duplicate
Merged: issue 631015
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression

Blocked on:
issue 631298



Sign in to add a comment

Fix and reenable smoothness.tough_pinch_zoom_cases failure on chromium.perf

Project Member Reported by eyaich@chromium.org, Jul 22 2016

Issue description

Revision range first seen:

405973 - 405981


Buildbot Status page (Step 52): https://uberchromegw.corp.google.com/i/chromium.perf/builders/Android%20Nexus5X%20Perf%20%282%29/builds/1104


Link to failing step log:

https://uberchromegw.corp.google.com/i/chromium.perf/builders/Android%20Nexus5X%20Perf%20%282%29/builds/1104/steps/smoothness.tough_pinch_zoom_cases/logs/stdio


Seems to be a chrome crash from logs.  

Traceback (most recent call last):
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/telemetry/internal/story_runner.py", line 85, in _RunStoryAndProcessErrorIfNeeded
    state.RunStory(results)
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/telemetry/page/shared_page_state.py", line 319, in RunStory
    self._current_page.Run(self)
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/telemetry/page/__init__.py", line 99, in Run
    self.RunPageInteractions(action_runner)
  File "/b/rr/tmpXmh5AU/w/src/tools/perf/page_sets/tough_pinch_zoom_cases.py", line 36, in RunPageInteractions
    self.RunPinchGesture(action_runner, scale_factor=1/2.0)
  File "/b/rr/tmpXmh5AU/w/src/tools/perf/page_sets/tough_pinch_zoom_cases.py", line 27, in RunPinchGesture
    speed_in_pixels_per_second=speed_in_pixels_per_second)
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/telemetry/internal/actions/action_runner.py", line 294, in PinchPage
    speed_in_pixels_per_second=speed_in_pixels_per_second))
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/telemetry/internal/actions/action_runner.py", line 45, in _RunAction
    action.RunAction(self._tab)
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/telemetry/internal/actions/pinch.py", line 72, in RunAction
    tab.WaitForJavaScriptExpression('window.__pinchActionDone', 60)
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/telemetry/internal/browser/web_contents.py", line 123, in WaitForJavaScriptExpression
    util.WaitFor(IsJavaScriptExpressionTrue, timeout)
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/telemetry/core/util.py", line 86, in WaitFor
    res = condition()
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/telemetry/internal/browser/web_contents.py", line 116, in IsJavaScriptExpressionTrue
    return bool(self.EvaluateJavaScript(expr))
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/telemetry/internal/browser/web_contents.py", line 187, in EvaluateJavaScript
    expr, context_id=None, timeout=timeout)
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/telemetry/internal/browser/web_contents.py", line 215, in EvaluateJavaScriptInContext
    expr, context_id=context_id, timeout=timeout)
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/telemetry/internal/backends/chrome_inspector/inspector_backend.py", line 35, in inner
    inspector_backend._ConvertExceptionFromInspectorWebsocket(e)
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/telemetry/internal/backends/chrome_inspector/inspector_backend.py", line 32, in inner
    return func(inspector_backend, *args, **kwargs)
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/telemetry/internal/backends/chrome_inspector/inspector_backend.py", line 203, in EvaluateJavaScript
    return self._runtime.Evaluate(expr, context_id, timeout)
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/telemetry/internal/backends/chrome_inspector/inspector_runtime.py", line 45, in Evaluate
    res = self._inspector_websocket.SyncRequest(request, timeout)
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/telemetry/internal/backends/chrome_inspector/inspector_websocket.py", line 110, in SyncRequest
    res = self._Receive(timeout)
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/telemetry/internal/backends/chrome_inspector/inspector_websocket.py", line 149, in _Receive
    data = self._socket.recv()
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/third_party/websocket-client/websocket.py", line 596, in recv
    opcode, data = self.recv_data()
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/third_party/websocket-client/websocket.py", line 606, in recv_data
    frame = self.recv_frame()
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/third_party/websocket-client/websocket.py", line 637, in recv_frame
    self._frame_header = self._recv_strict(2)
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/third_party/websocket-client/websocket.py", line 746, in _recv_strict
    bytes = self._recv(shortage)
  File "/b/rr/tmpXmh5AU/w/src/third_party/catapult/telemetry/third_party/websocket-client/websocket.py", line 739, in _recv
    raise WebSocketConnectionClosedException()
DevtoolsTargetCrashException: Devtools target crashed



If the test is disabled, please downgrade to Pri-2.

Will disable the test as a next step.

 
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 23 2016

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

commit 0dd36fd482f3a39bf2d546ceadb5e7378cc137ba
Author: eyaich <eyaich@google.com>
Date: Sat Jul 23 03:39:39 2016

Disabling smoothness.tough_pinch_zoom_cases for android Nexus 5X

BUG= 630701 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq

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

[modify] https://crrev.com/0dd36fd482f3a39bf2d546ceadb5e7378cc137ba/tools/perf/benchmarks/smoothness.py

Owner: dtu@chromium.org
Assigning to dtu, bug triage oncall for this week: Dave, the bisect failed due to build failure, anything we can do to bisect around the failures?

Comment 4 by picksi@chromium.org, Jul 25 2016

Perf sheriff says... I'm disabling this test (smoothness.tough_pinch_zoom_cases) as it is turning a number of bots red.

see: https://codereview.chromium.org/2177203003 for the patch.

Also turning Android Nexus6 Perf (2) red: https://bugs.chromium.org/p/chromium/issues/detail?id=631015
 

Comment 5 by dtu@chromium.org, Jul 26 2016

Blockedon: 631298

Comment 6 by dtu@chromium.org, Jul 26 2016

Was not a build failure, but a bug in bisect. Filed  issue 631298 . Please try again after https://codereview.chromium.org/2183693002/ lands!
Project Member

Comment 7 by bugdroid1@chromium.org, Jul 26 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build.git/+/9810d80e493f8ddef253bbb58f41e5caae2d9805

commit 9810d80e493f8ddef253bbb58f41e5caae2d9805
Author: dtu <dtu@chromium.org>
Date: Tue Jul 26 23:14:41 2016

Fix bisect depot url for catapult.

Also remove ".git" from the v8 url, to be consistent with the rest (although both forms are acceptable).

BUG= 630701 ,  631298 

Review-Url: https://codereview.chromium.org/2183693002

[modify] https://crrev.com/9810d80e493f8ddef253bbb58f41e5caae2d9805/scripts/slave/recipe_modules/auto_bisect/depot_config.py
[modify] https://crrev.com/9810d80e493f8ddef253bbb58f41e5caae2d9805/scripts/slave/recipe_modules/auto_bisect/example.expected/android_arm64_bisector.json
[modify] https://crrev.com/9810d80e493f8ddef253bbb58f41e5caae2d9805/scripts/slave/recipe_modules/auto_bisect/example.expected/android_bisector.json
[modify] https://crrev.com/9810d80e493f8ddef253bbb58f41e5caae2d9805/scripts/slave/recipe_modules/auto_bisect/example.expected/basic.json
[modify] https://crrev.com/9810d80e493f8ddef253bbb58f41e5caae2d9805/scripts/slave/recipe_modules/auto_bisect/example.expected/basic_bisect_script.json
[modify] https://crrev.com/9810d80e493f8ddef253bbb58f41e5caae2d9805/scripts/slave/recipe_modules/auto_bisect/example.expected/mac_bisector.json
[modify] https://crrev.com/9810d80e493f8ddef253bbb58f41e5caae2d9805/scripts/slave/recipe_modules/auto_bisect/example.expected/reversed_basic.json
[modify] https://crrev.com/9810d80e493f8ddef253bbb58f41e5caae2d9805/scripts/slave/recipe_modules/auto_bisect/example.expected/windows_bisector.json
[modify] https://crrev.com/9810d80e493f8ddef253bbb58f41e5caae2d9805/scripts/slave/recipe_modules/auto_bisect/example.expected/windows_x64_bisector.json

Project Member

Comment 8 by bugdroid1@chromium.org, Jul 26 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build.git/+/9810d80e493f8ddef253bbb58f41e5caae2d9805

commit 9810d80e493f8ddef253bbb58f41e5caae2d9805
Author: dtu <dtu@chromium.org>
Date: Tue Jul 26 23:14:41 2016

Fix bisect depot url for catapult.

Also remove ".git" from the v8 url, to be consistent with the rest (although both forms are acceptable).

BUG= 630701 ,  631298 

Review-Url: https://codereview.chromium.org/2183693002

[modify] https://crrev.com/9810d80e493f8ddef253bbb58f41e5caae2d9805/scripts/slave/recipe_modules/auto_bisect/depot_config.py
[modify] https://crrev.com/9810d80e493f8ddef253bbb58f41e5caae2d9805/scripts/slave/recipe_modules/auto_bisect/example.expected/android_arm64_bisector.json
[modify] https://crrev.com/9810d80e493f8ddef253bbb58f41e5caae2d9805/scripts/slave/recipe_modules/auto_bisect/example.expected/android_bisector.json
[modify] https://crrev.com/9810d80e493f8ddef253bbb58f41e5caae2d9805/scripts/slave/recipe_modules/auto_bisect/example.expected/basic.json
[modify] https://crrev.com/9810d80e493f8ddef253bbb58f41e5caae2d9805/scripts/slave/recipe_modules/auto_bisect/example.expected/basic_bisect_script.json
[modify] https://crrev.com/9810d80e493f8ddef253bbb58f41e5caae2d9805/scripts/slave/recipe_modules/auto_bisect/example.expected/mac_bisector.json
[modify] https://crrev.com/9810d80e493f8ddef253bbb58f41e5caae2d9805/scripts/slave/recipe_modules/auto_bisect/example.expected/reversed_basic.json
[modify] https://crrev.com/9810d80e493f8ddef253bbb58f41e5caae2d9805/scripts/slave/recipe_modules/auto_bisect/example.expected/windows_bisector.json
[modify] https://crrev.com/9810d80e493f8ddef253bbb58f41e5caae2d9805/scripts/slave/recipe_modules/auto_bisect/example.expected/windows_x64_bisector.json

Project Member

Comment 9 by bugdroid1@chromium.org, Jul 27 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra.git/+/3aeffe4b0054fcd5281a37675ffc2301c6119cbd

commit 3aeffe4b0054fcd5281a37675ffc2301c6119cbd
Author: recipe-roller <recipe-roller@chromium.org>
Date: Wed Jul 27 00:46:54 2016

Roll recipe dependencies (trivial).

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/bc11eedbd74022297d9a64be416ad6baef1f4ef8 Refactor chromium.perf recipe configs. (dtu@chromium.org)
  https://crrev.com/9810d80e493f8ddef253bbb58f41e5caae2d9805 Fix bisect depot url for catapult. (dtu@chromium.org)
  https://crrev.com/01ffce92ae86a69cf8df252b0f29ea2c9b6c4d30 Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/761a85597e4e990fbf964f5688cf444481eca846 Add config for Linux layout test try bot with --enable-slimming-paint-v2 (qyearsley@chromium.org)
  https://crrev.com/aef80ea24343ad84eac0aa070f43c39e66626e74 Add builder config for blink try bots (mac10.10, mac10.11, trusty, win10). (qyearsley@chromium.org)
  https://crrev.com/12f907139d62df04c66b9545abe28ce7d97cc3fb Support Swarming for iOS (smut@google.com)
depot_tools:
  https://crrev.com/e3144ea35900f9a1183327f07b82918d2ba9f3fa Remove git-try from depot_tools (agable@chromium.org)

R=mmoss@chromium.org,smut@google.com,dtu@chromium.org,recipe-roller@chromium.org,martiniss@chromium.org,phajdan.jr@chromium.org,agable@chromium.org,qyearsley@chromium.org
BUG= 590036 , 611484 ,609893, 630701 , 631298 ,622865,598405

TBR=martiniss@chromium.org,phajdan.jr@chromium.org

Review-Url: https://codereview.chromium.org/2182443004

[modify] https://crrev.com/3aeffe4b0054fcd5281a37675ffc2301c6119cbd/infra/config/recipes.cfg

Project Member

Comment 10 by bugdroid1@chromium.org, Jul 27 2016

Project Member

Comment 12 by 42576172...@developer.gserviceaccount.com, Jul 29 2016

Bisect failed: Unknown
Failure reason: the build has failed due to infrastructure failure.

Project Member

Comment 13 by 42576172...@developer.gserviceaccount.com, Jul 29 2016


===== BISECT JOB RESULTS =====
Status: failed


===== TESTED REVISIONS =====
Revision         Exit Code  Std Dev  N  Good?
chromium@405973  0          N/A      1  good
chromium@405977  0          N/A      1  good
chromium@405979  0          N/A      1  good
chromium@405981  1          N/A      1  bad

Bisect job ran on: android_nexus5X_perf_bisect
Bug ID: 630701

Test Command: src/tools/perf/run_benchmark -v --browser=android-chromium --output-format=chartjson --upload-results --also-run-disabled-tests smoothness.tough_pinch_zoom_cases
Test Metric: frame_lengths/Blogger
Relative Change: Zero to non-zero
Score: 0

Buildbot stdio: http://build.chromium.org/p/tryserver.chromium.perf/builders/android_nexus5X_perf_bisect/builds/360
Job details: https://chromeperf.appspot.com/buildbucket_job_status/9006411881563786864


Not what you expected? We'll investigate and get back to you!
  https://chromeperf.appspot.com/bad_bisect?try_job_id=6419139124527104

| O O | Visit http://www.chromium.org/developers/speed-infra/perf-bug-faq
|  X  | for more information addressing perf regression bugs. For feedback,
| / \ | file a bug with component Tests>AutoBisect.  Thank you!
Project Member

Comment 16 by 42576172...@developer.gserviceaccount.com, Jul 29 2016

Bisect failed: Unknown
Failure reason: the build has failed due to infrastructure failure.

Project Member

Comment 17 by 42576172...@developer.gserviceaccount.com, Jul 29 2016


===== BISECT JOB RESULTS =====
Status: failed


===== TESTED REVISIONS =====
Revision         Exit Code  Std Dev  N  Good?
chromium@405973  0          N/A      1  good
chromium@405977  0          N/A      1  good
chromium@405979  0          N/A      1  good
chromium@405981  1          N/A      1  bad

Bisect job ran on: android_nexus5X_perf_bisect
Bug ID: 630701

Test Command: src/tools/perf/run_benchmark -v --browser=android-chromium --output-format=chartjson --upload-results --also-run-disabled-tests smoothness.tough_pinch_zoom_cases
Test Metric: frame_lengths/Blogger
Relative Change: Zero to non-zero
Score: 0

Buildbot stdio: http://build.chromium.org/p/tryserver.chromium.perf/builders/android_nexus5X_perf_bisect/builds/360
Job details: https://chromeperf.appspot.com/buildbucket_job_status/9006411881563786864


Not what you expected? We'll investigate and get back to you!
  https://chromeperf.appspot.com/bad_bisect?try_job_id=6419139124527104

| O O | Visit http://www.chromium.org/developers/speed-infra/perf-bug-faq
|  X  | for more information addressing perf regression bugs. For feedback,
| / \ | file a bug with component Tests>AutoBisect.  Thank you!
Friendly sheriff ping

(According to the Perf Bot Sheriffing guide, Pri-1 build bot bugs should be pinged daily. Please close the issue if it's already been fixed.)
Cc: -bokan@chromium.org dtu@chromium.org
Labels: -Pri-1 Pri-2
Owner: bokan@chromium.org
Summary: Fix and reenable smoothness.tough_pinch_zoom_cases failure on chromium.perf (was: smoothness.tough_pinch_zoom_cases failure on chromium.perf )
Mergedinto: 631015
Status: Duplicate (was: Untriaged)

Sign in to add a comment