Issue metadata
Sign in to add a comment
|
Many autotests are failing with "Unhandled WebSocketConnectionClosedException: socket is already closed" |
||||||||||||||||||||||||
Issue descriptionMany autotests have been failing due to this since 8/27 https://stainless.corp.google.com/search?view=list&first_date=2018-08-25&last_date=2018-08-30&test=%5Echeets%5C_&status=FAIL&status=ERROR&status=ABORT&exclude_cts=true&exclude_not_run=false&exclude_non_release=true&exclude_au=true&exclude_acts=true&exclude_retried=true&exclude_non_production=false Error log: https://paste.googleplex.com/6053467722350592?raw Failed with WebSocketConnectionClosedException while starting the browser backend. Unhandled WebSocketConnectionClosedException: socket is already closed.
,
Aug 30
The Chrome OS Infra Team doesn't have the expertise or the charter to triage mere test failures; it needs to be done by product team engineers. These are cheets test failures, so the triage needs to start with the ARC++ team. When all else fails, ARC++ failures start with the ARC++ constable.
,
Aug 30
started working on this... couldn't reproduce it locally :( niwa@ Assigning back to you... if you can't fix it today, assign it back to me and I'll continue working on this on Friday.
,
Aug 31
The failure is not limited to ARC tests: https://stainless.corp.google.com/search?view=list&first_date=2018-08-29&last_date=2018-08-31&reason=Unhandled+WebSocketConnectionClosedException&exclude_cts=false&exclude_not_run=false&exclude_non_release=false&exclude_au=false&exclude_acts=false&exclude_retried=false&exclude_non_production=false (login_GaiaLogin, platform_LogoutPerf etc are also failing) on the other hand I don't see any cheets_StartAndroid or cheets_CTS failures even though they should have run tens of thousands of times during. My gut guess is that tests doing gaia login is affected?
,
Aug 31
I coudldn't repro it locally, either. I ran login_GaiaLogin and several ARC tests on my caroline-arcnext DUT but they all passed (or failed with other errors.) Assigning back to ricardoq@, but maybe we should find someone familiar with GAIA login on autotest, as kinaba@ pointed out.
,
Sep 4
,
Sep 4
cc this week constables. This sounds very important.
,
Sep 4
Correcting courtneygo@ to edcourtney@
,
Sep 4
it seems this problem started since R70-11005.1.0
,
Sep 4
(assigning to new constable)
,
Sep 4
If kinaba@ is right this may be a dup of crbug.com/879353 , still reading logs but this seems similar at first glance.
,
Sep 5
Re #11: good finding. Looks like a dup to me, too. +achuith@
,
Sep 5
It is similar, but don't think that fix is going to work for the other autotest. I'll take a look.
,
Sep 5
Actually, this is a dupe of 879353, since the cheets tests actually use gaia login. This ought to be fixed by https://chromium-review.googlesource.com/c/catapult/+/1197749.
,
Sep 5
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by niwa@chromium.org
, Aug 30