----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53752/ -----------------------------------------------------------
Review request for geode, Barry Oglesby, Jason Huynh, Dan Smith, and xiaojian zhou. Repository: geode Description ------- This issue was reported in CI monitoring. More investigation of the failure logs led to the following findings. 1. During test setup VM3 is missing. 2. Also the log show [setup] START TEST WanAutoDiscoveryDUnitTest.test_RingTopology instead of [setup] START TEST WanAutoDiscoveryDUnitTest.test_NY_Recognises_TK_AND_HK_Simultaneously 3. By evaluating the stacktrace we see that NPE when we force async[2].getReturnValue() to be null. It the return value of WANTestBase.createSecondLocator(1, lnLocPort1) which simple returns an port number which is not in use. There is an infinite while loop which breaks only if it finds a port not in use. IMO we remove the flaky tag and monitor the test. Please do let me know if you can find an explanation for this failure. Diffs ----- geode-wan/src/test/java/org/apache/geode/internal/cache/wan/misc/WanAutoDiscoveryDUnitTest.java 388760e Diff: https://reviews.apache.org/r/53752/diff/ Testing ------- Ran it for multiple iterations. Thanks, nabarun nag
