sabbey37 commented on a change in pull request #5732:
URL: https://github.com/apache/geode/pull/5732#discussion_r521637229
##########
File path: ci/scripts/execute_redis_tests.sh
##########
@@ -49,8 +51,8 @@ failCount=0
--redis-port=6379 \
--redis-bind-address=127.0.0.1
-./runtest --host 127.0.0.1 --port 6379 --single unit/type/set --single
unit/expire \
---single unit/type/hash --single unit/type/string --single
unit/type/srandmember \
+./runtest --host 127.0.0.1 --port 6380 --single unit/type/set --single
unit/expire \
Review comment:
We should change this port back to 6379 (since the Geode Redis server
without a password is started on `6379`, which is why we get the error
`Executing test client: NOAUTH Authentication required` in the RedisTests)
----------------------------------------------------------------
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:
[email protected]