Skip to content

fix: add missing variable for cache#46

Open
utnapischtim wants to merge 1 commit into
inveniosoftware:masterfrom
utnapischtim:add-cache-redis-url
Open

fix: add missing variable for cache#46
utnapischtim wants to merge 1 commit into
inveniosoftware:masterfrom
utnapischtim:add-cache-redis-url

Conversation

@utnapischtim

Copy link
Copy Markdown
Contributor
  • this makes it possible to run also redis on a different port

Comment thread docker_services_cli/config.py Outdated
"cache": {"CACHE_TYPE": "redis"},
"mq": {
"BROKER_URL": "redis://localhost:6380/0",
"CACHE_REDIS_URL": "redis://127.0.0.1:6380/0",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"CACHE_REDIS_URL": "redis://127.0.0.1:6380/0",

minor: do we need that for mq/Celery? I remember there's something about a results backend for Celery, but that's maybe under a different env/config?

* this makes it possible to run also redis on a different port
@utnapischtim utnapischtim force-pushed the add-cache-redis-url branch from 658262f to 800f170 Compare March 7, 2025 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants