Perhaps also worth considering: can we get WindowsStressNew added to the PR
checks?

On Wed, Feb 19, 2020 at 8:50 AM Udo Kohlmeyer <u...@apache.com> wrote:

> Is this something that can be fixed in a short time (2hrs)? If not, can
> be revert and get back to a clean pipeline?
>
> --Udo
>
> On 2/19/20 8:23 AM, Jens Deppe wrote:
> > Thanks Kirk,
> >
> > We're working on fixing this.
> >
> > --Jens
> >
> > On Tue, Feb 18, 2020 at 3:23 PM Kirk Lund <kl...@pivotal.io> wrote:
> >
> >> I just started seeing the Redis PubSubTest fail in IntegrationTest after
> >> rebasing on develop this afternoon. Looks like I have Jens' latest
> commit
> >> for this test:
> >>
> >>
> >>
> >>
> >>
> >>
> >>
> >> *commit 1befce17eaae2403828769840f86639e13fce81f (origin/develop,
> >> origin/HEAD, develop)Author: Jens Deppe <jde...@pivotal.io
> >> <jde...@pivotal.io>>Date:   Tue Feb 18 13:03:19 2020 -0800
> GEODE-7798:
> >> Fix flaky PubSub test (#4714)    *
> >>
> >> *    Authored-by: Jens Deppe <jde...@pivotal.io <jde...@pivotal.io>>>
> Task
> >> :geode-redis:integrationTest*
> >>
> >> Here's the stack traces:
> >>
> >> org.apache.geode.redis.PubSubTest > testPatternWithoutAGlob FAILED
> >>      redis.clients.jedis.exceptions.JedisConnectionException:
> >> java.net.SocketTimeoutException: Read timed out
> >>          at
> >>
> redis.clients.util.RedisInputStream.ensureFill(RedisInputStream.java:202)
> >>          at
> >> redis.clients.util.RedisInputStream.readByte(RedisInputStream.java:40)
> >>          at redis.clients.jedis.Protocol.process(Protocol.java:151)
> >>          at redis.clients.jedis.Protocol.read(Protocol.java:215)
> >>          at
> >>
> redis.clients.jedis.Connection.readProtocolWithCheckingBroken(Connection.java:340)
> >>          at
> >> redis.clients.jedis.Connection.getIntegerReply(Connection.java:265)
> >>          at redis.clients.jedis.Jedis.publish(Jedis.java:2690)
> >>          at
> >>
> org.apache.geode.redis.PubSubTest.testPatternWithoutAGlob(PubSubTest.java:279)
> >>
> >>          Caused by:
> >>          java.net.SocketTimeoutException: Read timed out
> >>              at java.net.SocketInputStream.socketRead0(Native Method)
> >>              at
> >> java.net.SocketInputStream.socketRead(SocketInputStream.java:115)
> >>              at
> java.net.SocketInputStream.read(SocketInputStream.java:168)
> >>              at
> java.net.SocketInputStream.read(SocketInputStream.java:140)
> >>              at
> java.net.SocketInputStream.read(SocketInputStream.java:126)
> >>              at
> >>
> redis.clients.util.RedisInputStream.ensureFill(RedisInputStream.java:196)
> >>              ... 7 more
> >>
> >> org.apache.geode.redis.PubSubTest > testTwoSubscribersOneChannel FAILED
> >>      org.awaitility.core.ConditionTimeoutException: Condition with
> lambda
> >> expression in org.apache.geode.redis.PubSubTest that uses
> >> org.apache.geode.redis.mocks.MockSubscriber was not fulfilled within 1
> >> seconds.
> >>          at
> >> org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:145)
> >>          at
> >> org.awaitility.core.CallableCondition.await(CallableCondition.java:79)
> >>          at
> >> org.awaitility.core.CallableCondition.await(CallableCondition.java:27)
> >>          at
> >> org.awaitility.core.ConditionFactory.until(ConditionFactory.java:902)
> >>          at
> >> org.awaitility.core.ConditionFactory.until(ConditionFactory.java:860)
> >>          at
> org.apache.geode.redis.PubSubTest.waitFor(PubSubTest.java:296)
> >>          at
> >>
> org.apache.geode.redis.PubSubTest.testTwoSubscribersOneChannel(PubSubTest.java:140)
> >>
> >> org.apache.geode.redis.PubSubTest >
> >> testOneSubscriberSubscribingToTwoChannels FAILED
> >>      org.awaitility.core.ConditionTimeoutException: Condition with
> lambda
> >> expression in org.apache.geode.redis.PubSubTest that uses
> >> org.apache.geode.redis.mocks.MockSubscriber was not fulfilled within 1
> >> seconds.
> >>          at
> >> org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:145)
> >>          at
> >> org.awaitility.core.CallableCondition.await(CallableCondition.java:79)
> >>          at
> >> org.awaitility.core.CallableCondition.await(CallableCondition.java:27)
> >>          at
> >> org.awaitility.core.ConditionFactory.until(ConditionFactory.java:902)
> >>          at
> >> org.awaitility.core.ConditionFactory.until(ConditionFactory.java:860)
> >>          at
> org.apache.geode.redis.PubSubTest.waitFor(PubSubTest.java:296)
> >>          at
> >>
> org.apache.geode.redis.PubSubTest.testOneSubscriberSubscribingToTwoChannels(PubSubTest.java:110)
> >>
> >> org.apache.geode.redis.PubSubTest > testPatternAndRegularSubscribe
> FAILED
> >>      org.awaitility.core.ConditionTimeoutException: Condition with
> lambda
> >> expression in org.apache.geode.redis.PubSubTest that uses
> >> org.apache.geode.redis.mocks.MockSubscriber was not fulfilled within 1
> >> seconds.
> >>          at
> >> org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:145)
> >>          at
> >> org.awaitility.core.CallableCondition.await(CallableCondition.java:79)
> >>          at
> >> org.awaitility.core.CallableCondition.await(CallableCondition.java:27)
> >>          at
> >> org.awaitility.core.ConditionFactory.until(ConditionFactory.java:902)
> >>          at
> >> org.awaitility.core.ConditionFactory.until(ConditionFactory.java:860)
> >>          at
> org.apache.geode.redis.PubSubTest.waitFor(PubSubTest.java:296)
> >>          at
> >>
> org.apache.geode.redis.PubSubTest.testPatternAndRegularSubscribe(PubSubTest.java:245)
> >>
> >> org.apache.geode.redis.PubSubTest > testDeadSubscriber FAILED
> >>      org.junit.ComparisonFailure: expected:<[0]L> but was:<[2]L>
> >>          at
> >> jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native
> >> Method)
> >>          at
> >>
> jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
> >>          at
> >>
> jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
> >>          at
> >>
> org.apache.geode.redis.PubSubTest.testDeadSubscriber(PubSubTest.java:207)
> >>
> >> org.apache.geode.redis.PubSubTest > testOneSubscriberOneChannelTwoTimes
> >> FAILED
> >>      org.awaitility.core.ConditionTimeoutException: Condition with
> lambda
> >> expression in org.apache.geode.redis.PubSubTest that uses
> >> org.apache.geode.redis.mocks.MockSubscriber was not fulfilled within 1
> >> seconds.
> >>          at
> >> org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:145)
> >>          at
> >> org.awaitility.core.CallableCondition.await(CallableCondition.java:79)
> >>          at
> >> org.awaitility.core.CallableCondition.await(CallableCondition.java:27)
> >>          at
> >> org.awaitility.core.ConditionFactory.until(ConditionFactory.java:902)
> >>          at
> >> org.awaitility.core.ConditionFactory.until(ConditionFactory.java:860)
> >>          at
> org.apache.geode.redis.PubSubTest.waitFor(PubSubTest.java:296)
> >>          at
> >>
> org.apache.geode.redis.PubSubTest.testOneSubscriberOneChannelTwoTimes(PubSubTest.java:176)
> >>
> >> 82 tests completed, 6 failed
> >>
>

Reply via email to