Skip to content

Commit

Permalink
Replacing redis-stack-server image usage with client-libs-test. Fixin…
Browse files Browse the repository at this point in the history
…g lib version in setup.py
  • Loading branch information
petyaslavova committed Feb 12, 2025
1 parent 9eddb16 commit 6f42577
Show file tree
Hide file tree
Showing 5 changed files with 25 additions and 15 deletions.
10 changes: 5 additions & 5 deletions .github/actions/run-tests/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ runs:
- name: Setup Test environment
env:
REDIS_VERSION: ${{ inputs.redis-version }}
CLIENT_LIBS_TEST_IMAGE: "redislabs/client-libs-test:${{ inputs.redis-version }}"
CLIENT_LIBS_TEST_IMAGE_TAG: ${{ inputs.redis-version }}
run: |
set -e
Expand All @@ -55,13 +55,13 @@ runs:
# Mapping of redis version to stack version
declare -A redis_stack_version_mapping=(
["7.4.2"]="7.4.0-v2"
["7.2.7"]="7.2.0-v14"
["6.2.17"]="6.2.6-v18"
["7.4.2"]="rs-7.4.0-v2"
["7.2.7"]="rs-7.2.0-v14"
["6.2.17"]="rs-6.2.6-v18"
)
if [[ -v redis_stack_version_mapping[$REDIS_VERSION] ]]; then
export REDIS_STACK_IMAGE="redis/redis-stack-server:${redis_stack_version_mapping[$REDIS_VERSION]}"
export CLIENT_LIBS_TEST_STACK_IMAGE_TAG=${redis_stack_version_mapping[$REDIS_VERSION]}
echo "REDIS_MOD_URL=redis://127.0.0.1:6479/0" >> $GITHUB_ENV
else
echo "Version not found in the mapping."
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/integration.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ env:
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
# this speeds up coverage with Python 3.12: https://github.com/nedbat/coveragepy/issues/1665
COVERAGE_CORE: sysmon
REDIS_STACK_IMAGE: redis/redis-stack-server:latest
CLIENT_LIBS_TEST_STACK_IMAGE_TAG: '8.0-M04-pre'
CURRENT_REDIS_VERSION: '7.4.2'

jobs:
Expand Down Expand Up @@ -180,7 +180,7 @@ jobs:
- name: Run installed unit tests
env:
REDIS_VERSION: ${{ env.CURRENT_REDIS_VERSION }}
CLIENT_LIBS_TEST_IMAGE: "redislabs/client-libs-test:${{ env.CURRENT_REDIS_VERSION }}"
CLIENT_LIBS_TEST_IMAGE_TAG: ${{ env.CURRENT_REDIS_VERSION }}
run: |
bash .github/workflows/install_and_test.sh ${{ matrix.extension }}
Expand Down
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -25,3 +25,4 @@ docker/stunnel/keys
/dockers/cluster/
/dockers/replica/
/dockers/sentinel/
/dockers/redis-stack/
23 changes: 16 additions & 7 deletions docker-compose.yml
Original file line number Diff line number Diff line change
@@ -1,9 +1,14 @@
---
x-client-libs-stack-image: &client-libs-stack-image
image: "redislabs/client-libs-test:${CLIENT_LIBS_TEST_STACK_IMAGE_TAG:-rs-7.4.0-v2}"

x-client-libs-image: &client-libs-image
image: "redislabs/client-libs-test:${CLIENT_LIBS_TEST_IMAGE_TAG:-7.4.2}"

services:

redis:
image: ${CLIENT_LIBS_TEST_IMAGE:-redislabs/client-libs-test:7.4.1}
<<: *client-libs-image
container_name: redis-standalone
environment:
- TLS_ENABLED=yes
Expand All @@ -24,7 +29,7 @@ services:
- all

replica:
image: ${CLIENT_LIBS_TEST_IMAGE:-redislabs/client-libs-test:7.4.1}
<<: *client-libs-image
container_name: redis-replica
depends_on:
- redis
Expand All @@ -43,7 +48,7 @@ services:
- all

cluster:
image: ${CLIENT_LIBS_TEST_IMAGE:-redislabs/client-libs-test:7.4.1}
<<: *client-libs-image
container_name: redis-cluster
environment:
- REDIS_CLUSTER=yes
Expand All @@ -64,7 +69,7 @@ services:
- all

sentinel:
image: ${CLIENT_LIBS_TEST_IMAGE:-redislabs/client-libs-test:7.4.1}
<<: *client-libs-image
container_name: redis-sentinel
depends_on:
- redis
Expand All @@ -86,12 +91,16 @@ services:
- all

redis-stack:
image: ${REDIS_STACK_IMAGE:-redis/redis-stack-server:latest}
<<: *client-libs-stack-image
container_name: redis-stack
environment:
- REDIS_CLUSTER=no
- PORT=6379
command: ${REDIS_EXTRA_ARGS:---enable-debug-command yes --enable-module-command yes --save ""}
ports:
- 6479:6379
environment:
- "REDIS_ARGS=${REDIS_STACK_EXTRA_ARGS:---enable-debug-command yes --enable-module-command yes --save ''}"
volumes:
- "./dockers/redis-stack:/redis/work"
profiles:
- standalone
- all-stack
Expand Down
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
long_description_content_type="text/markdown",
keywords=["Redis", "key-value store", "database"],
license="MIT",
version="5.3.0",
version="5.2.1",
packages=find_packages(
include=[
"redis",
Expand Down

0 comments on commit 6f42577

Please sign in to comment.