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

Issue 754893 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

[CFM Test Automation] CFM automated tests started failing on M61

Project Member Reported by harpreet@chromium.org, Aug 11 2017

Issue description

After various experiments were enabled for the test domain being used by CFM automated tests, these tests started failing because of devtool crash which is a result of app re-launching in the middle of the test.


08/11 03:02:28.496 DEBUG|              test:0390| Test failed due to RPC error: cfm.wait_for_oobe_start_page
Traceback (most recent call last):
  File "./multimedia_xmlrpc_server.py", line 89, in _dispatch
    return func(*params)
  File "/usr/local/autotest/cros/multimedia/cfm_facade_native.py", line 150, in wait_for_oobe_start_page
    self.cfmApi.wait_for_oobe_start_page()
  File "/usr/local/autotest/cros/multimedia/cfm_facade_native.py", line 108, in cfmApi
    if self._webview_context.EvaluateJavaScript(
  File "/usr/local/autotest/cros/multimedia/cfm_facade_native.py", line 102, in _webview_context
    self._resource._browser, self._EXT_ID)
  File "/usr/local/autotest/common_lib/cros/kiosk_utils.py", line 27, in get_webview_context
    "document.querySelector('webview') ? 'WEBVIEW' : 'NOWEBVIEW'")
  File "/usr/local/telemetry/src/third_party/catapult/common/py_trace_event/py_trace_event/trace_event_impl/decorators.py", line 52, in traced_function
    return func(*args, **kwargs)
  File "/usr/local/telemetry/src/third_party/catapult/telemetry/telemetry/internal/browser/web_contents.py", line 167, in EvaluateJavaScript
    return self._inspector_backend.EvaluateJavaScript(*args, **kwargs)
  File "/usr/local/telemetry/src/third_party/catapult/common/py_trace_event/py_trace_event/trace_event_impl/decorators.py", line 52, in traced_function
    return func(*args, **kwargs)
  File "/usr/local/telemetry/src/third_party/catapult/telemetry/telemetry/internal/backends/chrome_inspector/inspector_backend.py", line 237, in EvaluateJavaScript
    return self._EvaluateJavaScript(expression, context_id, timeout)
  File "/usr/local/telemetry/src/third_party/catapult/common/py_trace_event/py_trace_event/trace_event_impl/decorators.py", line 52, in traced_function
    return func(*args, **kwargs)
  File "/usr/local/telemetry/src/third_party/catapult/telemetry/telemetry/internal/backends/chrome_inspector/inspector_backend.py", line 39, in inner
    inspector_backend._ConvertExceptionFromInspectorWebsocket(e)
  File "/usr/local/telemetry/src/third_party/catapult/common/py_trace_event/py_trace_event/trace_event_impl/decorators.py", line 52, in traced_function
    return func(*args, **kwargs)
  File "/usr/local/telemetry/src/third_party/catapult/telemetry/telemetry/internal/backends/chrome_inspector/inspector_backend.py", line 36, in inner
    return func(inspector_backend, *args, **kwargs)
  File "/usr/local/telemetry/src/third_party/catapult/telemetry/telemetry/internal/backends/chrome_inspector/inspector_backend.py", line 501, in _EvaluateJavaScript
    return self._runtime.Evaluate(expression, context_id, timeout)
  File "/usr/local/telemetry/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 "/usr/local/telemetry/src/third_party/catapult/telemetry/telemetry/internal/backends/chrome_inspector/inspector_websocket.py", line 110, in SyncRequest
    res = self._Receive(timeout)
  File "/usr/local/telemetry/src/third_party/catapult/telemetry/telemetry/internal/backends/chrome_inspector/inspector_websocket.py", line 149, in _Receive
    data = self._socket.recv()
  File "/usr/local/telemetry/src/third_party/catapult/telemetry/third_party/websocket-client/websocket/_core.py", line 293, in recv
    opcode, data = self.recv_data()
  File "/usr/local/telemetry/src/third_party/catapult/telemetry/third_party/websocket-client/websocket/_core.py", line 310, in recv_data
    opcode, frame = self.recv_data_frame(control_frame)
  File "/usr/local/telemetry/src/third_party/catapult/telemetry/third_party/websocket-client/websocket/_core.py", line 323, in recv_data_frame
    frame = self.recv_frame()
  File "/usr/local/telemetry/src/third_party/catapult/telemetry/third_party/websocket-client/websocket/_core.py", line 357, in recv_frame
    return self.frame_buffer.recv_frame()
  File "/usr/local/telemetry/src/third_party/catapult/telemetry/third_party/websocket-client/websocket/_abnf.py", line 336, in recv_frame
    self.recv_header()
  File "/usr/local/telemetry/src/third_party/catapult/telemetry/third_party/websocket-client/websocket/_abnf.py", line 286, in recv_header
    header = self.recv_strict(2)
  File "/usr/local/telemetry/src/third_party/catapult/telemetry/third_party/websocket-client/websocket/_abnf.py", line 371, in recv_strict
    bytes_ = self.recv(min(16384, shortage))
  File "/usr/local/telemetry/src/third_party/catapult/telemetry/third_party/websocket-client/websocket/_core.py", line 427, in _recv
    return recv(self.sock, bufsize)
  File "/usr/local/telemetry/src/third_party/catapult/telemetry/third_party/websocket-client/websocket/_socket.py", line 93, in recv
    "Connection is already closed.")
DevtoolsTargetCrashException: Devtools target crashed
********************************************************************************
(/usr/local/telemetry/src/third_party/catapult/telemetry/telemetry/internal/backends/chrome_inspector/inspector_backend.py:475 _ConvertExceptionFromInspectorWebsocket) Original exception:
Connection is already closed.
********************************************************************************
(/usr/local/telemetry/src/third_party/catapult/telemetry/telemetry/internal/backends/chrome_inspector/inspector_backend.py:496 _AddDebuggingInformation) Received a socket error in the browser connection and the tab no longer exists. The tab probably crashed.
********************************************************************************
(/usr/local/telemetry/src/third_party/catapult/telemetry/telemetry/internal/backends/chrome_inspector/inspector_backend.py:497 _AddDebuggingInformation) Debugger url: ws://127.0.0.1:60784/devtools/page/15117888-0e66-4b9d-bc8b-60a7d376a9a1
Found Minidump: False
Stack Trace:
********************************************************************************
	Cannot get stack trace on CrOS
********************************************************************************
Standard output:
********************************************************************************
	Cannot get standard output on CrOS
********************************************************************************
System log:
(Not implemented)
. Exception log follows the after_iteration_hooks.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 12 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/57db30b57ef4b6d7cf1d9890aa19ec6a9254a1bc

commit 57db30b57ef4b6d7cf1d9890aa19ec6a9254a1bc
Author: harpreet <harpreet@google.com>
Date: Sat Aug 12 08:26:27 2017

Fix CFM enrollment and app launch

Fix issue with app re-launching during the test run causing devtool
failure resulting in test failure.

BUG= chromium:754893 
TEST=Tested on a local setup.

Change-Id: Iad9c7536667e4d16d56e52a8e803d156f0d8972e
Reviewed-on: https://chromium-review.googlesource.com/612656
Commit-Ready: Harpreet Grewal <harpreet@chromium.org>
Tested-by: Harpreet Grewal <harpreet@chromium.org>
Reviewed-by: danny chan <dchan@chromium.org>

[modify] https://crrev.com/57db30b57ef4b6d7cf1d9890aa19ec6a9254a1bc/client/cros/multimedia/cfm_facade_native.py
[modify] https://crrev.com/57db30b57ef4b6d7cf1d9890aa19ec6a9254a1bc/server/site_tests/enterprise_CFM_SessionStress/enterprise_CFM_SessionStress.py
[modify] https://crrev.com/57db30b57ef4b6d7cf1d9890aa19ec6a9254a1bc/server/cros/multimedia/cfm_facade_adapter.py
[modify] https://crrev.com/57db30b57ef4b6d7cf1d9890aa19ec6a9254a1bc/server/site_tests/enterprise_CFM_USBPeripheralHotplugStress/enterprise_CFM_USBPeripheralHotplugStress.py
[modify] https://crrev.com/57db30b57ef4b6d7cf1d9890aa19ec6a9254a1bc/server/site_tests/enterprise_CFM_USBPeripheralRebootStress/enterprise_CFM_USBPeripheralRebootStress.py
[modify] https://crrev.com/57db30b57ef4b6d7cf1d9890aa19ec6a9254a1bc/server/site_tests/enterprise_CFM_MeetAppSanity/enterprise_CFM_MeetAppSanity.py
[modify] https://crrev.com/57db30b57ef4b6d7cf1d9890aa19ec6a9254a1bc/server/site_tests/enterprise_CFM_Perf/enterprise_CFM_Perf.py
[modify] https://crrev.com/57db30b57ef4b6d7cf1d9890aa19ec6a9254a1bc/server/site_tests/enterprise_CFM_Sanity/enterprise_CFM_Sanity.py
[modify] https://crrev.com/57db30b57ef4b6d7cf1d9890aa19ec6a9254a1bc/client/site_tests/enterprise_RemoraRequisition/enterprise_RemoraRequisition.py
[modify] https://crrev.com/57db30b57ef4b6d7cf1d9890aa19ec6a9254a1bc/server/site_tests/enterprise_CFM_USBPeripheralHotplugDetect/enterprise_CFM_USBPeripheralHotplugDetect.py
[modify] https://crrev.com/57db30b57ef4b6d7cf1d9890aa19ec6a9254a1bc/server/site_tests/enterprise_CFM_VolumeChange/enterprise_CFM_VolumeChange.py

Labels: M-61 Merge-Request-61
Project Member

Comment 3 by sheriffbot@chromium.org, Aug 14 2017

Labels: -Merge-Request-61 Merge-Review-61 Hotlist-Merge-Review
This bug requires manual review: M61 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), ketakid@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 4 by ketakid@google.com, Aug 14 2017

Labels: -Merge-Review-61 Merge-Approved-61
Approving merge for M61 Chrome OS.
Project Member

Comment 5 by bugdroid1@chromium.org, Aug 14 2017

Labels: merge-merged-release-R61-9765.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/e1bf26f5f5f80a3075f2f28f8398e5e7d16ce87b

commit e1bf26f5f5f80a3075f2f28f8398e5e7d16ce87b
Author: harpreet <harpreet@google.com>
Date: Mon Aug 14 23:07:12 2017

Fix CFM enrollment and app launch

Fix issue with app re-launching during the test run causing devtool
failure resulting in test failure.

BUG= chromium:754893 
TEST=Tested on a local setup.

Change-Id: Iad9c7536667e4d16d56e52a8e803d156f0d8972e
Reviewed-on: https://chromium-review.googlesource.com/612656
Commit-Ready: Harpreet Grewal <harpreet@chromium.org>
Tested-by: Harpreet Grewal <harpreet@chromium.org>
Reviewed-by: danny chan <dchan@chromium.org>
(cherry picked from commit 57db30b57ef4b6d7cf1d9890aa19ec6a9254a1bc)
Reviewed-on: https://chromium-review.googlesource.com/614740
Reviewed-by: Harpreet Grewal <harpreet@chromium.org>
Commit-Queue: Harpreet Grewal <harpreet@chromium.org>
Trybot-Ready: Harpreet Grewal <harpreet@chromium.org>

[modify] https://crrev.com/e1bf26f5f5f80a3075f2f28f8398e5e7d16ce87b/client/cros/multimedia/cfm_facade_native.py
[modify] https://crrev.com/e1bf26f5f5f80a3075f2f28f8398e5e7d16ce87b/server/site_tests/enterprise_CFM_SessionStress/enterprise_CFM_SessionStress.py
[modify] https://crrev.com/e1bf26f5f5f80a3075f2f28f8398e5e7d16ce87b/server/cros/multimedia/cfm_facade_adapter.py
[modify] https://crrev.com/e1bf26f5f5f80a3075f2f28f8398e5e7d16ce87b/server/site_tests/enterprise_CFM_USBPeripheralHotplugStress/enterprise_CFM_USBPeripheralHotplugStress.py
[modify] https://crrev.com/e1bf26f5f5f80a3075f2f28f8398e5e7d16ce87b/server/site_tests/enterprise_CFM_USBPeripheralRebootStress/enterprise_CFM_USBPeripheralRebootStress.py
[modify] https://crrev.com/e1bf26f5f5f80a3075f2f28f8398e5e7d16ce87b/server/site_tests/enterprise_CFM_MeetAppSanity/enterprise_CFM_MeetAppSanity.py
[modify] https://crrev.com/e1bf26f5f5f80a3075f2f28f8398e5e7d16ce87b/server/site_tests/enterprise_CFM_Perf/enterprise_CFM_Perf.py
[modify] https://crrev.com/e1bf26f5f5f80a3075f2f28f8398e5e7d16ce87b/server/site_tests/enterprise_CFM_Sanity/enterprise_CFM_Sanity.py
[modify] https://crrev.com/e1bf26f5f5f80a3075f2f28f8398e5e7d16ce87b/client/site_tests/enterprise_RemoraRequisition/enterprise_RemoraRequisition.py
[modify] https://crrev.com/e1bf26f5f5f80a3075f2f28f8398e5e7d16ce87b/server/site_tests/enterprise_CFM_USBPeripheralHotplugDetect/enterprise_CFM_USBPeripheralHotplugDetect.py
[modify] https://crrev.com/e1bf26f5f5f80a3075f2f28f8398e5e7d16ce87b/server/site_tests/enterprise_CFM_VolumeChange/enterprise_CFM_VolumeChange.py

Project Member

Comment 6 by bugdroid1@chromium.org, Aug 15 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/18e5dc16be238089173c9bfb2dd65319d78e5d7f

commit 18e5dc16be238089173c9bfb2dd65319d78e5d7f
Author: harpreet <harpreet@google.com>
Date: Tue Aug 15 03:37:52 2017

Part 2 of fixing CFM enrollment and app launch

The plan is to merge the other CL back to M61 but because the tests in
this CL are not in M61, I created a separate CL for these tests.

BUG= chromium:754893 
TEST=Tested on a local setup.

Change-Id: Iab87ef8a1deee1bff4c4c6eb07632cd71292d55e
Reviewed-on: https://chromium-review.googlesource.com/612701
Commit-Ready: Harpreet Grewal <harpreet@chromium.org>
Tested-by: Harpreet Grewal <harpreet@chromium.org>
Reviewed-by: danny chan <dchan@chromium.org>

[modify] https://crrev.com/18e5dc16be238089173c9bfb2dd65319d78e5d7f/server/site_tests/enterprise_CFM_AutoZoomSanity/enterprise_CFM_AutoZoomSanity.py
[modify] https://crrev.com/18e5dc16be238089173c9bfb2dd65319d78e5d7f/server/site_tests/enterprise_CFM_RebootStress/enterprise_CFM_RebootStress.py
[modify] https://crrev.com/18e5dc16be238089173c9bfb2dd65319d78e5d7f/server/site_tests/enterprise_CFM_AutoZoomSanity/control

Labels: meeting-rooms-sheriff
Status: Fixed (was: Untriaged)
Cc: dtosic@chromium.org mzhuo@chromium.org
Project Member

Comment 9 by sheriffbot@chromium.org, Aug 18 2017

Cc: ketakid@google.com
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: Merge-Merged
Project Member

Comment 11 by sheriffbot@chromium.org, Aug 21 2017

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Approved-61

Comment 13 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment