diff options
| author | yoav-steinberg <yoav@monfort.co.il> | 2021-11-11 13:04:02 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-11-11 13:04:02 +0200 |
| commit | cd6b3d558be0703b1a43f9fe58fd5f1ed7452829 (patch) | |
| tree | f703402582bf89a68a919e88b889ee50cc97d800 /.github | |
| parent | f069d09ee0e266064ab3b2d63390910accd47fbf (diff) | |
| download | redis-cd6b3d558be0703b1a43f9fe58fd5f1ed7452829.tar.gz | |
Archive external redis log in external tests (#9765)
On test failure store the external redis server logs as CI artifacts so we can review them.
Write test name to server log for external server tests.
This is attempted and silently failed in case external server doesn't support it.
Note that in non-external server mode we use a more robust method of writing to the log which doesn't depend on the
server actually running/working. This isn't possible for externl servers and required for some complex tests which are
skipped in external mode anyway.
Cleanup: remove dup code.
Diffstat (limited to '.github')
| -rw-r--r-- | .github/workflows/external.yml | 16 |
1 files changed, 14 insertions, 2 deletions
diff --git a/.github/workflows/external.yml b/.github/workflows/external.yml index 9b1065649..2dbd4704b 100644 --- a/.github/workflows/external.yml +++ b/.github/workflows/external.yml @@ -15,12 +15,18 @@ jobs: - name: Build run: make REDIS_CFLAGS=-Werror - name: Start redis-server - run: ./src/redis-server --daemonize yes + run: ./src/redis-server --daemonize yes --logfile external-redis.log - name: Run external test run: | ./runtest \ --host 127.0.0.1 --port 6379 \ --tags -slow + - name: Archive redis log + if: ${{ failure() }} + uses: actions/upload-artifact@v2 + with: + name: test-external-redis-log + path: external-redis.log test-external-cluster: runs-on: ubuntu-latest @@ -30,7 +36,7 @@ jobs: - name: Build run: make REDIS_CFLAGS=-Werror - name: Start redis-server - run: ./src/redis-server --cluster-enabled yes --daemonize yes + run: ./src/redis-server --cluster-enabled yes --daemonize yes --logfile external-redis.log - name: Create a single node cluster run: ./src/redis-cli cluster addslots $(for slot in {0..16383}; do echo $slot; done); sleep 5 - name: Run external test @@ -39,4 +45,10 @@ jobs: --host 127.0.0.1 --port 6379 \ --cluster-mode \ --tags -slow + - name: Archive redis log + if: ${{ failure() }} + uses: actions/upload-artifact@v2 + with: + name: test-external-cluster-log + path: external-redis.log |