recipe-maintainers/discourse
0
0
Fork 0
Code Issues Pull Requests 5 Actions Packages Projects Releases Wiki Activity

fix(app): use stack-prefixed hostnames for redis and db #8

Open
autonomic-bot wants to merge 1 commits from fix-redis-host into main
pull from: fix-redis-host
merge into: recipe-maintainers:main
Conversation 0 Commits 1 Files Changed 1 +4 -2
autonomic-bot commented 2026-06-23 16:48:22 +00:00
Owner
Copy Link

The app service was missing DISCOURSE_REDIS_HOST entirely, so the bitnami image defaulted to the bare hostname redis. When a service is attached to multiple overlay networks (proxy + internal), bare service-name DNS does not resolve reliably — the stack-prefixed name (${STACK_NAME}_redis) is required. This caused the app to WARN "Hostname redis could not be resolved" and fall into a restart loop.

sidekiq had the same problem for both DB and Redis: DISCOURSE_DATABASE_HOST=db and DISCOURSE_REDIS_HOST=redis used bare hostnames instead of the stack-prefixed service names.

Changes:

  • app: add DISCOURSE_REDIS_HOST=${STACK_NAME}_redis
  • sidekiq: fix DISCOURSE_DATABASE_HOST=db → ${STACK_NAME}_db
  • sidekiq: fix DISCOURSE_REDIS_HOST=redis → ${STACK_NAME}_redis

No image or version bump — this is a compose env-var fix only.

cc @trav @notplants

The `app` service was missing `DISCOURSE_REDIS_HOST` entirely, so the bitnami image defaulted to the bare hostname `redis`. When a service is attached to multiple overlay networks (proxy + internal), bare service-name DNS does not resolve reliably — the stack-prefixed name (`${STACK_NAME}_redis`) is required. This caused the app to WARN "Hostname redis could not be resolved" and fall into a restart loop. `sidekiq` had the same problem for both DB and Redis: `DISCOURSE_DATABASE_HOST=db` and `DISCOURSE_REDIS_HOST=redis` used bare hostnames instead of the stack-prefixed service names. **Changes:** - `app`: add `DISCOURSE_REDIS_HOST=${STACK_NAME}_redis` - `sidekiq`: fix `DISCOURSE_DATABASE_HOST=db` → `${STACK_NAME}_db` - `sidekiq`: fix `DISCOURSE_REDIS_HOST=redis` → `${STACK_NAME}_redis` No image or version bump — this is a compose env-var fix only. cc @trav @notplants
autonomic-bot added 1 commit 2026-06-23 16:48:22 +00:00
fix(app): use stack-prefixed hostnames for redis and db 2dae2ac3ca 
The app service was missing DISCOURSE_REDIS_HOST entirely, so the
bitnami image defaulted to the bare hostname 'redis', which does not
resolve when the service is attached to multiple overlay networks
(proxy + internal) — causing a restart loop.

sidekiq had the same problem worse: both DISCOURSE_DATABASE_HOST=db
and DISCOURSE_REDIS_HOST=redis used bare hostnames instead of the
stack-prefixed service names. On a swarm stack the reliable DNS name
is ${STACK_NAME}_db / ${STACK_NAME}_redis.
autonomic-bot requested review from trav 2026-06-23 16:48:22 +00:00
autonomic-bot requested review from notplants 2026-06-23 16:48:23 +00:00
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin fix-redis-host:fix-redis-host
git checkout fix-redis-host
Sign in to join this conversation.
Reviewers
No Reviewers
trav
notplants
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
autonomic autonomic-bot notplants trav
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: recipe-maintainers/discourse#8
No description provided.
Delete Branch "fix-redis-host"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?