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.
