fix run.sh.

apache · Jun 16, 2023 · f4f5d1d · f4f5d1d
1 parent 16511ee
commit f4f5d1d
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 7 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -233,10 +233,10 @@ jobs:
           path: |
             /tmp/ceresdb-stdout.log
 
-  recover-test:
-    name: recover-test
+  recovery-test:
+    name: recovery-test
     runs-on: ubuntu-latest
-    timeout-minutes: 120
+    timeout-minutes: 60
     steps:
       - uses: actions/checkout@v3
         with:

diff --git a/integration_tests/Makefile b/integration_tests/Makefile
@@ -71,4 +71,4 @@ run-prom:
 	cd prom && ./run-tests.sh
 
 run-recovery: clean build-ceresdb kill-old-process
-	cd recovery && sh ./run.sh && sh ./run.sh shard_based
+	cd recovery && ./run.sh && ./run.sh shard_based
diff --git a/integration_tests/recovery/run.sh b/integration_tests/recovery/run.sh
@@ -1,13 +1,15 @@
 #!/usr/bin/env bash
 
-# for compatibility in macos
+set -e
+
 ROOT=`pwd`
+# For compatibility in macos, so convert to milliseconds by adding 3 zeros.
 NOW=`date +%s000`
 BINARY_PATH=${ROOT}/../../target/debug/ceresdb-server
 SERVER_HTTP_ENDPOINT=127.0.0.1:5440
 
 CONFIG_FILE=${ROOT}/../../docs/minimal.toml
-if [[ ${1} == "shard_based" ]]; then
+if [ ${1} == 'shard_based' ]; then
     CONFIG_FILE=${ROOT}/../config/shard-based-recovery.toml
 fi
 
@@ -25,7 +27,7 @@ python3 ./check.py -ts ${NOW}
 
 echo "Flush, restart and check..."
 curl -XPOST ${SERVER_HTTP_ENDPOINT}/debug/flush_memtable
-echo "\nflush finish..."
+echo "\nFlush finish..."
 killall ceresdb-server | true
 nohup ${BINARY_PATH} --config ${CONFIG_FILE} &
 sleep 10