Log error in action when testing failed (#304)

* Log error in action when testing failed.

* Fix the typo.

* Rm testing directory when clean in CI.

* Add mock error.

* Revert "Add mock error."

This reverts commit c04e4f97c1c2a6d20d1649672e1f7cef7b7ae540.

* Upload logs to artifact in action.

* Pass parameters instead of environment to subcommand.

* Mock ci error.

* Mock error.

* Remove the mock error.

* Avoid print log to screen.

* Store logs of different matrix.

* Change the log path.

Co-authored-by: Yee <2520865+yixinglu@users.noreply.github.com>
Co-authored-by: laura-ding <48548375+laura-ding@users.noreply.github.com>

.github/workflows/pull_request.yml

@@ -128,8 +128,14 @@ jobs:
       - name: Pytest
         env:
           NEBULA_TEST_LOGS_DIR: ${{ github.workspace }}/build
-        run: ./ci/test.sh test
+        run: ./ci/test.sh test --rm_dir=false
         timeout-minutes: 25
+      - name: Upload logs
+        uses: actions/upload-artifact@v2
+        if: ${{ failure() }}
+        with:	
+          name: ${{ matrix.os }}-${{ matrix.compiler }}-nebula-test-logs
+          path:  build/server_*/logs/
       - name: Cleanup
         if: ${{ always() }}
-        run: rm -rf build modules
+        run: rm -rf build modules && rm -rf /tmp/nebula-*

ci/test.sh

@@ -97,9 +97,10 @@ function run_test() {
         -n=8 \
         --dist=loadfile \
         --debug_log=false \
+        ${@:1} \
         $testpath
 
-    $BUILD_DIR/tests/ntr --debug_log=false $PROJ_DIR/tests/job/*
+    $BUILD_DIR/tests/ntr --debug_log=false ${@:1} $PROJ_DIR/tests/job/*
 }
 
 function test_in_cluster() {
@@ -132,7 +133,7 @@ case "$1" in
         run_ctest
         ;;
     test)
-        run_test
+        run_test "${@:2}"
         ;;
     k8s)
         prepare