Skip to content

Conversation

ClearlyClaire
Copy link
Contributor

Alternative to #30412

On the one hand, some major distributions, like Ubuntu 22.04 LTS, still ship earlier versions of Redis, so this may be a relatively disruptive change for many server administrators.

On the other hand, Redis versions earlier to 6.2 are not officially maintained anymore, and at least one of our dependencies (Stoplight) has silently dropped support for it, so maintaining compatibility with Redis < 6.2 is becoming harder.

@ClearlyClaire ClearlyClaire marked this pull request as ready for review May 24, 2024 09:21
@ClearlyClaire
Copy link
Contributor Author

(If #30412 gets merged, I'll keep this PR as a more long-term one for after 4.3)

@renchap
Copy link
Member

renchap commented Jun 19, 2024

To be merged after 4.4. It may be a good idea to fail to start if Redis is too old?

Copy link
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@mjankowski
Copy link
Contributor

Looks like this needs rebase, but also to clarify -- does "merge after 4.4" mean "merge after we've started on 4.4" (ie, now), or "merge after we RELEASE 4.4" (ie, later)?

@mjankowski mjankowski added deployment Related to runtime configuration, production setups dependencies Pull requests that update a dependency file ruby Pull requests that update Ruby code labels Nov 30, 2024
@mjankowski
Copy link
Contributor

Looks like this needs rebase, but also to clarify -- does "merge after 4.4" mean "merge after we've started on 4.4" (ie, now), or "merge after we RELEASE 4.4" (ie, later)?

Still curious on this -- should this version support drop start hitting nightlies now, or waiting for after next release?

@ClearlyClaire
Copy link
Contributor Author

Well, for a start, mastodon.social is still using Redis < 6.2

@mjankowski
Copy link
Contributor

Feels like a blocker.

@ClearlyClaire ClearlyClaire force-pushed the drop-redis-below-6.2 branch from 2b8292a to 309d8b0 Compare May 19, 2025 09:12
@ClearlyClaire
Copy link
Contributor Author

ClearlyClaire commented May 19, 2025

mastodon.online and mastodon.social are now running Redis 7.2. However, Ubuntu 22.04 LTS, which I expect is still a pretty popular distribution amongst Mastodon server administrators, only provides Redis 6.0…

Also of note, sidekiq 7.x requires Redis 6.2+, and sidekiq 8.x requires Redis 7.0+

Copy link
Contributor

This pull request has resolved merge conflicts and is ready for review.

@ClearlyClaire
Copy link
Contributor Author

Looks like this will be required for sidekiq 7, which we want to move to.

@ClearlyClaire ClearlyClaire added this pull request to the merge queue May 20, 2025
Merged via the queue into main with commit f94b1fc May 20, 2025
31 checks passed
@ClearlyClaire ClearlyClaire deleted the drop-redis-below-6.2 branch May 20, 2025 13:07
@mjankowski
Copy link
Contributor

Sort of related here ... in the aftermath of sidekiq 7 merging and rack 3 presumably merging soon, it occurs to me that we could actually go all the way to sidekiq 8 ... except that does bring a redis 7.0 requirement with it.

I see that 6.2 is past EOL and am curious ... if we are comfortable doing a minimum version bump here which may require some older distro folks to do upgrades beyond stock install - could we go all the way to 7.0? Or is there a sweet spot where 6.2 is more viable for people but 7.0+ would be too far?

@ClearlyClaire
Copy link
Contributor Author

Redis 7.0 should be available on Ubuntu 24.04 LTS as well as the latest Debian stable, so updating all the way to Redis 7.0 does not seem to be a dealbreaker, though this could still be a somewhat steep requirement.

Updating to sidekiq 8 for Mastodon 4.4 seems possible, and if that fails, 4.5 would definitely be an appropriate target.

@mjankowski
Copy link
Contributor

Sounds good ... will open something as draft for now and we can contemplate timing.

@renchap
Copy link
Member

renchap commented May 27, 2025

I think I would prefer have Sidekiq 7 in Mastodon 4.4, and Sidekiq 8 in Mastodon 4.5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file deployment Related to runtime configuration, production setups ruby Pull requests that update Ruby code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants