"org.chromium.android_webview.test.PostMessageTest#testMessageChannelUsingPendingPort" is flaky |
||||||
Issue description"org.chromium.android_webview.test.PostMessageTest#testMessageChannelUsingPendingPort" is flaky. This issue was created automatically by the chromium-try-flakes app. Please find the right owner to fix the respective test/step and assign this issue to them. If the step/test is infrastructure-related, please add Infra-Troopers label and change issue status to Untriaged. When done, please remove the issue from Sheriff Bug Queue by removing the Sheriff-Chromium label. We have detected 3 recent flakes. List of all flakes can be found at https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyXwsSBUZsYWtlIlRvcmcuY2hyb21pdW0uYW5kcm9pZF93ZWJ2aWV3LnRlc3QuUG9zdE1lc3NhZ2VUZXN0I3Rlc3RNZXNzYWdlQ2hhbm5lbFVzaW5nUGVuZGluZ1BvcnQM. Flaky tests should be disabled within 30 minutes unless culprit CL is found and reverted. Please see more details here: https://sites.google.com/a/chromium.org/dev/developers/tree-sheriffs/sheriffing-bug-queues#triaging-auto-filed-flakiness-bugs
,
Apr 27 2017
I am disabling the test.
,
Apr 27 2017
The CL is https://codereview.chromium.org/2845863004/ and it is being submitted right now.
,
Apr 27 2017
Please have a look.
,
Apr 27 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c186cf10dc3367044f9143308bc3452eff0cd22c commit c186cf10dc3367044f9143308bc3452eff0cd22c Author: vitaliii <vitaliii@chromium.org> Date: Thu Apr 27 14:18:14 2017 Disable PostMessageTest#testMessageChannelUsingPendingPort because flaky Disable org.chromium.android_webview.test.PostMessageTest#testMessageCha nnelUsingPendingPort because it is flaky. TBR=sgurun@chromium.org BUG= 715960 Review-Url: https://codereview.chromium.org/2845863004 Cr-Commit-Position: refs/heads/master@{#467667} [modify] https://crrev.com/c186cf10dc3367044f9143308bc3452eff0cd22c/android_webview/javatests/src/org/chromium/android_webview/test/PostMessageTest.java
,
Apr 27 2017
,
Apr 27 2017
interesting. This is the second postmessage test that was marked as flaky, and these tests worked fine for more than a year. I wonder if some refactoring has anything to do with it. Darin's CL is the most major one I can think of.
,
Sep 15 2017
,
Sep 26 2017
looped test locally 50 times and no failures. gonna try enabling with RetryOnFailure. I guess someone check back in a couple of weeks and see if there are any flakes on the flakiness dashboard https://chromium-review.googlesource.com/c/chromium/src/+/685281
,
Sep 26 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/dacb6d43eff1948adf3f458f2a285902aac4bdc1 commit dacb6d43eff1948adf3f458f2a285902aac4bdc1 Author: Bo Liu <boliu@chromium.org> Date: Tue Sep 26 23:24:16 2017 aw: Enable testMessageChannelUsingPendingPort with RetryOnFailure Can't repro failure locally, so try again on bots. Bug: 715960 Change-Id: I9f6393799927e45c15804a8d41cdb16e0ef4da04 Reviewed-on: https://chromium-review.googlesource.com/685281 Reviewed-by: Paul Miller <paulmiller@chromium.org> Commit-Queue: Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#504517} [modify] https://crrev.com/dacb6d43eff1948adf3f458f2a285902aac4bdc1/android_webview/javatests/src/org/chromium/android_webview/test/PostMessageTest.java
,
Sep 29 2017
Looks pretty good. Not a single retry flake so far: https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webview_instrumentation_test_apk&tests=org.chromium.android_webview.test.PostMessageTest%23testMessageChannelUsingPendingPort
,
Oct 1 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/41147dd6d6eb10938fb002f546ad94bcb702b233 commit 41147dd6d6eb10938fb002f546ad94bcb702b233 Author: Bo Liu <boliu@chromium.org> Date: Sun Oct 01 03:58:14 2017 aw: Remove RetryOnFailure for testMessageChannelUsingPendingPort Looks good on cq for 3 days. Bug: 715960 Change-Id: I40905c5ddcb7cbb1797906ace7126a52d20c33a8 Reviewed-on: https://chromium-review.googlesource.com/693416 Reviewed-by: Paul Miller <paulmiller@chromium.org> Commit-Queue: Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#505497} [modify] https://crrev.com/41147dd6d6eb10938fb002f546ad94bcb702b233/android_webview/javatests/src/org/chromium/android_webview/test/PostMessageTest.java
,
Oct 1 2017
|
||||||
►
Sign in to add a comment |
||||||
Comment 1 by vitaliii@chromium.org
, Apr 27 2017