Skip to content

Commit

Permalink
Merge pull request #317 from NorskHelsenett/feature/remove-redis-in-f…
Browse files Browse the repository at this point in the history
…avour-of-valkey-in-docker-compose-setup

Feature/remove redis in favor of valkey in docker compose setup
  • Loading branch information
havardelnan authored Feb 3, 2025
2 parents 5d68ec9 + f407171 commit da31fce
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 8 deletions.
2 changes: 1 addition & 1 deletion hacks/docker-compose/api.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ services:
- ENABLE_TRACING=false
- PROFILER_ENABLED=false
- RABBITMQ_HOST=rabbitmq
- REDIS_HOST=redis
- REDIS_HOST=valkey
- MONGODB_HOST=mongodb
- HEALTHCHECK_ENDPOINT=localhost:9999
volumes:
Expand Down
12 changes: 6 additions & 6 deletions hacks/docker-compose/infrastructure.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ services:
condition: service_healthy
rabbitmq:
condition: service_healthy
redis:
valkey:
condition: service_healthy
links:
- mongodb
Expand Down Expand Up @@ -167,16 +167,16 @@ services:
- jaeger
ports:
- '4317:4317'
redis:
image: ${DOCKER_MIRROR}redis/redis-stack
valkey:
image: ${DOCKER_MIRROR}valkey/valkey
environment:
REDIS_ARGS: '--requirepass S3cret!'
VALKEY_EXTRA_FLAGS: '--requirepass S3cret!'
volumes:
- ../data/redis/data:/data
- ../data/valkey/data:/data
ports:
- '6379:6379'
- '8001:8001'
healthcheck:
test: ['CMD', 'redis-cli', 'ping']
test: ['CMD', 'valkey-cli', 'ping']
volumes:
data: {}
2 changes: 1 addition & 1 deletion hacks/docker-compose/vault/entrypoint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,7 @@ vault write database/config/redis \

vault write database/roles/redis-ror-api-role \
db_name="redis" \
creation_statements='["~*", "+@string", "+PING","+json.get", "+json.set"]' \
creation_statements='["~*", "+@string", "+PING"]' \
default_ttl="5m" \
max_ttl="1h"

Expand Down

0 comments on commit da31fce

Please sign in to comment.