Thanks Owen. It seems you already initiated the revert.
Best regards, -Alberto ________________________________ From: Owen Nichols <onich...@pivotal.io> Sent: Saturday, April 11, 2020 8:04 PM To: dev@geode.apache.org <dev@geode.apache.org> Subject: Re: Help with Concourse-CI redis integration test failing Hi Alberto, it looks like Integration tests have been broken since PR#4937 was merged at the end of the day Friday. However this PR had passed all its PR checks (as of 5 hours before it was merged) so I suspect this was just an unexpected interaction with the PR#4912 that came in just 3 hours before it). In general there is an expectation in the Geode community that when a commit breaks the pipeline, it should be reverted first, then fixed. Alberto, since I don’t think you’re a committer, how about you initiate the revert and I’ll approve it? GitHub makes this pretty easy: go to https://github.com/apache/geode/pull/4937 and click Revert near the bottom-right of the page then click Create pull request. > On Apr 11, 2020, at 10:01 AM, Alberto Gomez <alberto.go...@est.tech> wrote: > > Hi, > > I am seeing the following integration test case fail in the Concourse-CI when > executed on my pull request (https://github.com/apache/geode/pull/4928): > > org.apache.geode.redis.SetsIntegrationTest > > testSRem_should_notRemoveMembersOfSetNotSpecified FAILED > > Interestingly, if I run the test locally it passes. Also, the line of code > where the failure occurs does not point to a valid line in the code I see: > > at > org.apache.geode.redis.SetsIntegrationTest.testSRem_should_notRemoveMembersOfSetNotSpecified(SetsIntegrationTest.java:784 > > Any idea on what could be happening? > > Thanks in advance, > > -Alberto G.
