[ 
https://issues.apache.org/jira/browse/GEODE-8694?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17230666#comment-17230666
 ] 

ASF GitHub Bot commented on GEODE-8694:
---------------------------------------

sabbey37 commented on a change in pull request #5732:
URL: https://github.com/apache/geode/pull/5732#discussion_r521637555



##########
File path: 
geode-redis/src/integrationTest/java/org/apache/geode/redis/internal/GeodeServerRunTest.java
##########
@@ -29,7 +28,8 @@
   public static GeodeRedisServerRule server = new GeodeRedisServerRule();
 
   @Test
-  @Ignore("This is a no-op test to conveniently run redis api for geode server 
for local development/testing purposes")

Review comment:
       This should be ignored (and is why the Integration Tests and Stress New 
Test timed out).




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Use fork of Redis in Gemfire org to run Redis tests in CI
> ---------------------------------------------------------
>
>                 Key: GEODE-8694
>                 URL: https://issues.apache.org/jira/browse/GEODE-8694
>             Project: Geode
>          Issue Type: Improvement
>          Components: redis
>            Reporter: Sarah Abbey
>            Assignee: Helena Bales
>            Priority: Trivial
>              Labels: pull-request-available
>
> Currently, we are using a contributor's personal fork of Redis to run tests 
> against Geode Redis in CI.  This should be switched to the fork of Redis in 
> the Gemfire org.  Ideally, we will eventually be able to run tests from the 
> root Redis repo (currently we are unable to since we are not implementing all 
> the necessary commands).



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to