CacheConnectionTimeoutJUnitTest and ClientHealthMonitorJUnitTest

[Kirk Lund <kl...@apache.org>]

These two tests seem to be passing/failing about 50/50% of the time. I
recommend marking these as FlakyTest while the authors work on making them
pass 100%

:geode-protobuf:integrationTest

org.apache.geode.protocol.acceptance.CacheConnectionTimeoutJUnitTest >
testUnresponsiveClientsGetDisconnected FAILED
    org.awaitility.core.ConditionTimeoutException: Condition defined as a
lambda expression in
org.apache.geode.protocol.acceptance.CacheConnectionTimeoutJUnitTest null
within 145 milliseconds.
        at
org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:104)
        at
org.awaitility.core.AssertionCondition.await(AssertionCondition.java:117)
        at
org.awaitility.core.AssertionCondition.await(AssertionCondition.java:32)
        at
org.awaitility.core.ConditionFactory.until(ConditionFactory.java:809)
        at
org.awaitility.core.ConditionFactory.until(ConditionFactory.java:648)
        at
org.apache.geode.protocol.acceptance.CacheConnectionTimeoutJUnitTest.testUnresponsiveClientsGetDisconnected(CacheConnectionTimeoutJUnitTest.java:138)

14 tests completed, 1 failed

:geode-core:integrationTest

org.apache.geode.internal.cache.tier.sockets.ClientHealthMonitorJUnitTest >
testDeadClientRemovalByServer FAILED
    org.apache.geode.cache.client.ServerConnectivityException: Pool
unexpected closed socket on server connection=Pooled Connection to
localhost:22752: Connection[DESTROYED]). Server unreachable: could not
connect after 1 attempts
        at
org.apache.geode.cache.client.internal.OpExecutorImpl.handleException(OpExecutorImpl.java:798)
        at
org.apache.geode.cache.client.internal.OpExecutorImpl.handleException(OpExecutorImpl.java:635)
        at
org.apache.geode.cache.client.internal.OpExecutorImpl.executeOn(OpExecutorImpl.java:604)
        at
org.apache.geode.cache.client.internal.OpExecutorImpl.executeOn(OpExecutorImpl.java:614)
        at
org.apache.geode.cache.client.internal.PoolImpl.executeOn(PoolImpl.java:826)
        at
org.apache.geode.cache.client.internal.PutOp.execute(PutOp.java:124)
        at
org.apache.geode.cache.client.internal.ServerRegionProxy.putOnForTestsOnly(ServerRegionProxy.java:177)
        at
org.apache.geode.internal.cache.tier.sockets.ClientHealthMonitorJUnitTest.testDeadClientRemovalByServer(ClientHealthMonitorJUnitTest.java:224)

        Caused by:
        java.io.EOFException: The connection has been reset while reading
the header
            at
org.apache.geode.internal.cache.tier.sockets.Message.fetchHeader(Message.java:689)
            at
org.apache.geode.internal.cache.tier.sockets.Message.readHeaderAndPayload(Message.java:703)
            at
org.apache.geode.internal.cache.tier.sockets.Message.read(Message.java:652)
            at
org.apache.geode.internal.cache.tier.sockets.Message.recv(Message.java:1111)
            at
org.apache.geode.cache.client.internal.PutOp$PutOpImpl.attemptReadResponse(PutOp.java:469)
            at
org.apache.geode.cache.client.internal.AbstractOp.attempt(AbstractOp.java:392)
            at
org.apache.geode.cache.client.internal.ConnectionImpl.execute(ConnectionImpl.java:278)
            at
org.apache.geode.cache.client.internal.pooling.PooledConnection.execute(PooledConnection.java:332)
            at
org.apache.geode.cache.client.internal.OpExecutorImpl.executeWithPossibleReAuthentication(OpExecutorImpl.java:900)
            at
org.apache.geode.cache.client.internal.OpExecutorImpl.executeOn(OpExecutorImpl.java:599)
            ... 5 more

3761 tests completed, 1 failed, 141 skipped