Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updating all ruby dependencies to current versions #8431

Merged
merged 49 commits into from Jun 28, 2023

Conversation

SuperTux88
Copy link
Member

There is one update that requires a special mention: The current sidekiq version requires redis 6+, so I don't know if we should mention that in the changelog?

@SuperTux88 SuperTux88 added this to the 1.0.0 milestone Jun 27, 2023
Gemfile Show resolved Hide resolved
@denschub
Copy link
Member

The current sidekiq version requires redis 6+, so I don't know if we should mention that in the changelog?

Even Debian oldstable has Redis 6, so if someone still runs on Redis 5, they should be ashamed of themselves. 😜

@SuperTux88 SuperTux88 merged commit 040e1dc into diaspora:develop Jun 28, 2023
12 of 13 checks passed
@SuperTux88 SuperTux88 deleted the gem-updates branch June 28, 2023 00:39
@SuperTux88
Copy link
Member Author

Even Debian oldstable has Redis 6

@denschub OK, I was actually wrong about that, I only looked at the requirements of the new redis-client gem, which works with 6+, but it looks like sidekiq itself requires 6.2+, and debian oldstable only has 6.0. But there are backports of redis 7, so oldstable still works.

Worse is probably, that ubuntu 22.04 also only has redis 6.0 😞

But there are official apt repos for debian/ubuntu from redis: https://redis.io/docs/getting-started/installation/install-redis-on-linux/

So, we either probably still should add something to the changelog, hinting at these repos, or downgrade sidekiq again until there is an LTS Ubuntu with redis 7?

@denschub
Copy link
Member

@SuperTux88 Ah, that's not great. There's nothing that's relevant to us in the Sidekiq 7 changelog, so I'd opt for downgrading for now.

@SuperTux88
Copy link
Member Author

There's nothing that's relevant to us in the Sidekiq 7 changelog,

I would have wanted to use the new capsules to see if they can replace our own workarounds to prevent to not run more than one export/import in parallel, as this looked really nice. I didn't include this here, as I didn't want to mix upgrades with new features, but planned to look into that next.

So I would have tended to add a note to the changelog, and then just use the newer redis packages from redis. But I can also understand that some podmins maybe don't want to do that and only want to use packages from their official distro repo? So I guess downgrading is also fine. 🤷‍♂️

@denschub
Copy link
Member

I would have wanted to use the new capsules to see if they can replace our own workarounds to prevent to not run more than one export/import in parallel, as this looked really nice.

Yeah, I've seen that. But not having capsules doesn't hurt us at the moment - it only prevents us from cleaning up a bit of code. I don't see this being a reasonable motivation for forcing podmins from the added hassle (and, in turn, from the bunch of support requests we will inevitably get). Maybe if we (read: me) have figured out That One Deployment Thing for 1.0, we can update Sidekiq again.

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.

None yet

2 participants