Zakelly commented on code in PR #2175:
URL: https://github.com/apache/kvrocks/pull/2175#discussion_r1536795627


##########
.github/workflows/kvrocks.yaml:
##########
@@ -293,6 +301,30 @@ jobs:
           fi
           ./x.py test go build $GOCASE_RUN_ARGS ${{ matrix.ignore_when_tsan}}
 
+      - name: Install redis-py
+        run: pip3 install redis==4.3.6
+
+      - name: Run kvrocks2redis Test
+        # Currently, when enabling Tsan/Asan or running in macOS 11/14, the 
value mismatch in destination redis server.
+        # See https://github.com/apache/kvrocks/issues/2195.
+        if: ${{ !contains(matrix.name, 'Tsan') && !contains(matrix.name, 
'Asan') && !startsWith(matrix.os, 'macos') }}
+        run: |
+          ulimit -c unlimited
+          export LSAN_OPTIONS="suppressions=$(realpath 
./tests/lsan-suppressions)"
+          export TSAN_OPTIONS="suppressions=$(realpath 
./tests/tsan-suppressions)"
+          $HOME/local/bin/redis-server --daemonize yes
+          mkdir -p data
+          ./build/kvrocks --dir `pwd`/data --pidfile `pwd`/kvrocks.pid 
--daemonize yes
+          sleep 15s
+          echo -en "data-dir `pwd`/data\ndaemonize yes\noutput-dir 
./\nnamespace.__namespace 127.0.0.1 6379\n" >> ./kvrocks2redis-ci.conf
+          cat ./kvrocks2redis-ci.conf
+          ./build/kvrocks2redis -c ./kvrocks2redis-ci.conf
+          sleep 15s
+          python3 utils/kvrocks2redis/tests/populate-kvrocks.py --password="" 
--flushdb=true
+          sleep 15s
+          ps aux

Review Comment:
   To check the kvrocks, kvrocks2redis and redis-server process exist. It may 
be useful when test fails.



-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to