Skip to content

fix(docker-compose): update to redis:7#1328

Merged
fbacall merged 1 commit into
ElixirTeSS:masterfrom
kennethrioja:docker-compose-redis-update
Jun 22, 2026
Merged

fix(docker-compose): update to redis:7#1328
fbacall merged 1 commit into
ElixirTeSS:masterfrom
kennethrioja:docker-compose-redis-update

Conversation

@kennethrioja

Copy link
Copy Markdown
Contributor

Summary of changes

Upgrading redis in docker-compose.yml to version 7 as the latest gem/ruby/ror made some of the gems only compatible with version 7 of redis

Motivation and context

Tests weren't passing under redis:6

Checklist

  • I have read and followed the CONTRIBUTING guide.
  • I confirm that I have the authority necessary to make this contribution on behalf of its copyright owner and agree to license it to the TeSS codebase under the BSD license.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Upgrades the Redis container image used by TeSS’s local and production Docker Compose setups to Redis 7 to restore compatibility with newer Ruby/Rails/gem dependencies and get tests passing again.

Changes:

  • Update docker-compose.yml Redis image from redis:6 to redis:7.
  • Update docker-compose-prod.yml Redis image from redis:6 to redis:7.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
docker-compose.yml Bumps the dev/test Redis service image to Redis 7.
docker-compose-prod.yml Bumps the production Redis service image to Redis 7.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kennethrioja

Copy link
Copy Markdown
Contributor Author

Update: tests are passing, however my Sidekiq container doesn't initiate, the reason seems to be You are connected to Redis 6.2.22, Sidekiq requires Redis 7.0.0 or greater

image

Then when I am trying to test a source/content provider (UI), it tries to connect endlessly "Waiting for test to begin"...

@fbacall fbacall merged commit a0982e6 into ElixirTeSS:master Jun 22, 2026
7 checks passed
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.

3 participants