"org.chromium.content.browser.JavaBridgeBasicsTest#testReflectPrivateFieldRaisesException" is flaky |
||||||
Issue description"org.chromium.content.browser.JavaBridgeBasicsTest#testReflectPrivateFieldRaisesException" 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 6 recent flakes. List of all flakes can be found at https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyYwsSBUZsYWtlIlhvcmcuY2hyb21pdW0uY29udGVudC5icm93c2VyLkphdmFCcmlkZ2VCYXNpY3NUZXN0I3Rlc3RSZWZsZWN0UHJpdmF0ZUZpZWxkUmFpc2VzRXhjZXB0aW9uDA. 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
,
Dec 18 2017
Detected 3 new flakes for test/step "org.chromium.content.browser.JavaBridgeBasicsTest#testReflectPrivateFieldRaisesException". To see the actual flakes, please visit https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyYwsSBUZsYWtlIlhvcmcuY2hyb21pdW0uY29udGVudC5icm93c2VyLkphdmFCcmlkZ2VCYXNpY3NUZXN0I3Rlc3RSZWZsZWN0UHJpdmF0ZUZpZWxkUmFpc2VzRXhjZXB0aW9uDA. This message was posted automatically by the chromium-try-flakes app.
,
Dec 20 2017
torne@, michaelbai@, you are listed as OWNERS, could you please triage? Very flaky on android_n5x_swarming_rel trybot.
,
Dec 20 2017
Example failure: https://ci.chromium.org/buildbot/tryserver.chromium.android/android_n5x_swarming_rel/326202 result_details: https://00e9e64bacf9e1a7a795a96e7f0030ead9d91e85f754f88510-apidata.googleusercontent.com/download/storage/v1/b/chromium-result-details/o/html%2Fcontent_shell_test_apk_android_n5x_swarming_rel_326202_2017_12_14_T18_17_48-UTC?qk=AD5uMEuY7tR23fRrNRebAJF6VobRB3JyJV7J2uOiFU0hEahp4cwshQJBJUlWKke3S9eLxBB0egsncFTDP-AZRHN0i0h-ZzLgltwW2dX7iaQdUxwwGJzWlRrVCHDpH7TAeqAgmEr_3fAbaVBCq3Lo8-6e_1e0rXhA8y_v_7pAfxq2YSvRy6-WIollm4tawDUpAG_QLsH63n4Mv1hTlM7x7McPErsUBNsZBgQG3KeRxn2NBUj1QSqnGZDEXk_OxKOI6xIH073bxE4EMhghDdh2wm5Mv5SPLzJK8Sud-K1PwecjPODlAKwuKQoTEjCTdWNU5kwaImDI0thqExeGXXBFcEYLiFEwHPBkriNvP0Qvs8Qz6bhkMoL1roTwpyqeoAmkUfwXKDE2RFE_HBWJRqUURINzzPlyjYHvWSJ7Fhzs0Ar_MvGIEvRVdgHyQpApoTvCESX477fmVh2vts_l8IEeMRJdjs4mjZu-B15VhpqmFfPKsum5-e68rOqSsISRBH85k38bHUjX_C5qHC3nPmPZGf99LvXrOTqWolhoEad5kkt7yKNdpjL04Uw45-RT4QAd9J7TMEOKL-Xo95FBJVIw-f_UKsN-sHb83ifo5blV3vIEWnX0eF8lcY9GAEjpp_mKHGEN_TLKWTcekdOr1YQYXUHVLnBHVumpwLstQG2smBBkNiuLGi_g9iYaHMx_fTXX19M0V2VHmw9AvUg9Kbn0EhWKZVbKJLjiRSfsFNwcc99eLCMsvg8gzw3XxNZ6s9ynKB7KpYmpiP2OFozZ1clxsNNqu8tOZ5fY2jzpDZb6DSUIaeja84selbvEgt4ZeArfCQCxJ53in-Sas9-ubcB1YZ9YqKMtTIj7cA logcat: https://luci-logdog.appspot.com/v/?s=chromium%2Fandroid%2Fswarming%2Flogcats%2F3a6f0529b9099411%2F%2B%2Flogcat_logcat_org.chromium.content.browser.JavaBridgeBasicsTest.testReflectPrivateFieldRaisesException_20171214T181224-UTC_023d8c5ce9a32b99 Stack: 12-14 18:12:25.333 32695 350 W System.err: java.lang.NoSuchFieldException: mField 12-14 18:12:25.340 32695 350 W System.err: at java.lang.Class.getField(Class.java:891) 12-14 18:12:25.340 32695 350 W System.err: at org.chromium.base.SystemMessageHandler.nativeDoRunLoopOnce(Native Method) 12-14 18:12:25.340 32695 350 W System.err: at org.chromium.base.SystemMessageHandler.handleMessage(SystemMessageHandler.java:51) 12-14 18:12:25.340 32695 350 W System.err: at android.os.Handler.dispatchMessage(Handler.java:102) 12-14 18:12:25.340 32695 350 W System.err: at android.os.Looper.loop(Looper.java:148) 12-14 18:12:25.340 32695 350 W System.err: at android.os.HandlerThread.run(HandlerThread.java:61) 12-14 18:12:25.408 32695 32709 D cr_ScreenshotOnFail: [ScreenshotOnFailureStatement.java:80] Saving screenshot of test failure, /sdcard/temp_file-e9fd887b12695.png 12-14 18:12:25.809 32695 32695 D LifecycleMonitor: Lifecycle status change: org.chromium.content_shell_apk.ContentShellActivity@4f725e9 in: PAUSED 12-14 18:12:25.809 32695 32709 I TestRunner: failed: testReflectPrivateFieldRaisesException(org.chromium.content.browser.JavaBridgeBasicsTest) 12-14 18:12:25.810 32695 32709 I TestRunner: ----- begin exception ----- 12-14 18:12:25.814 32695 32709 I TestRunner: java.lang.AssertionError 12-14 18:12:25.814 32695 32709 I TestRunner: at org.junit.Assert.fail(Assert.java:86) 12-14 18:12:25.814 32695 32709 I TestRunner: at org.junit.Assert.assertTrue(Assert.java:41) 12-14 18:12:25.814 32695 32709 I TestRunner: at org.junit.Assert.assertFalse(Assert.java:64) 12-14 18:12:25.814 32695 32709 I TestRunner: at org.junit.Assert.assertFalse(Assert.java:74) 12-14 18:12:25.814 32695 32709 I TestRunner: at org.chromium.content.browser.JavaBridgeBasicsTest.assertNoRaisedException(JavaBridgeBasicsTest.java:132) 12-14 18:12:25.814 32695 32709 I TestRunner: at org.chromium.content.browser.JavaBridgeBasicsTest.testReflectPrivateFieldRaisesException(JavaBridgeBasicsTest.java:730) 12-14 18:12:25.814 32695 32709 I TestRunner: at java.lang.reflect.Method.invoke(Native Method) Looks to me like there's some sort of race condition in the definition of the class being reflected upon. I'm going to disable this test temporarily because it's become very flaky and there are plenty of other tests exercising the Java bridge.
,
Dec 20 2017
Temporarily disabling in https://chromium-review.googlesource.com/835867 .
,
Dec 20 2017
,
Dec 20 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/1bd274a4d2159dd4110dabc274ac7cdcfc6314c0 commit 1bd274a4d2159dd4110dabc274ac7cdcfc6314c0 Author: Kenneth Russell <kbr@chromium.org> Date: Wed Dec 20 03:02:17 2017 Temporarily disable testReflectPrivateFieldRaisesException. This test in JavaBridgeBasicsTest has become very flaky on the android_n5x_swarming_rel trybot. BUG=795378 TBR=torne@chromium.org Change-Id: Ia25a2b3407634a8a926bf618d87c12c9686abd0c Reviewed-on: https://chromium-review.googlesource.com/835867 Reviewed-by: Kenneth Russell <kbr@chromium.org> Commit-Queue: Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#525242} [modify] https://crrev.com/1bd274a4d2159dd4110dabc274ac7cdcfc6314c0/content/public/android/javatests/src/org/chromium/content/browser/JavaBridgeBasicsTest.java
,
Dec 20 2017
Detected 5 new flakes for test/step "org.chromium.content.browser.JavaBridgeBasicsTest#testReflectPrivateFieldRaisesException". To see the actual flakes, please visit https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyYwsSBUZsYWtlIlhvcmcuY2hyb21pdW0uY29udGVudC5icm93c2VyLkphdmFCcmlkZ2VCYXNpY3NUZXN0I3Rlc3RSZWZsZWN0UHJpdmF0ZUZpZWxkUmFpc2VzRXhjZXB0aW9uDA. This message was posted automatically by the chromium-try-flakes app.
,
Dec 20 2017
Assigning to michaelbai@ for triage.
,
Dec 21 2017
It's a little confusing but it looks like https://chromium-review.googlesource.com/835867 did succeed in suppressing these failures. The last flake occurrence in chromium-try-flakes was: https://ci.chromium.org/buildbot/tryserver.chromium.android/android_n5x_swarming_rel/329276 which occurred at commit position 525192. The failure suppression landed at position 525242. It looks like no further flakes have been reported since. Downgrading to P2 and removing sheriff label. |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by chromium...@appspot.gserviceaccount.com
, Dec 16 2017