-
-
Notifications
You must be signed in to change notification settings - Fork 9.8k
[Cache][Messsenger] Open Symfony Cache and Symfony Messenger Redis Ext-Redis Requirement to ^6.1 for Ubuntu 25.04 #62149
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
Conversation
|
Hey! To help keep things organized, we don't allow "Draft" pull requests. Could you please click the "ready for review" button or close this PR and open a new one when you are done? Note that a pull request does not have to be "perfect" or "ready for merge" when you first open it. We just want it to be ready for a first review. Cheers! Carsonbot |
…to ^6.1 for Ubuntu 25.04 LTS
424611c to
d888d6e
Compare
This reverts commit d888d6e.
|
In d888d6e which I reverted to not change any CI specific things: https://github.com/symfony/symfony/actions/runs/18780211842/job/53584212483 There is only the test failing which make sense that the RedisProxy Class method already includes the 6.2 methods @nicolas-grekas do you see a problem if they already exists? Seems like in 7.3 there was no diff between 6.1 and 6.2 done only between 6.1 and 6.0 (here). So think it fine that the method exist when 6.1 is used? |
|
The previous failure indicates that there seems to something incompatible with 6.1. I am going to check later if we maybe miss something already in previous Symfony versions. |
This comment was marked as resolved.
This comment was marked as resolved.
we indeed have compatibility issues: #62222 |
|
FTR, the latest Ubuntu LTS (before 26.04 being released next year) is 24.04 which ships ext-redis 5.3.7: https://packages.ubuntu.com/search?suite=noble§ion=all&arch=any&keywords=php8.3-redis&searchon=names |
|
@alexander-schranz given that the current Ubuntu version comes with ext-redis 6.2.0 and so the upcoming LTS will also have something > 6.1.0 do you still think we need to lower the requirement here (see #62222 of the compatibility code that we would have to ship and this will grow with future ext-redis releases)? |
|
@xabbuh sounds okay for me, we still could lower later the requirement if we see lot of reports about it. |
|
after some more feedback: see #62326 |
Ubuntu 25.04
which is Ubuntus current and latest LTSis still on Redis Extension 6.1.0-3: https://packages.ubuntu.com/search?keywords=php8.4-redis&searchon=names&suite=plucky§ion=allAs discussed in #62146 6.0 with @xabbuh is not possible to support without changes but 6.1 should.