Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Conversation
carsonbot commentedOct 24, 2025
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 |
424611c tod888d6eCompareThis reverts commitd888d6e.
alexander-schranz commentedOct 24, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Ind888d6e 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 |
xabbuh commentedOct 30, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@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 |
Uh oh!
There was an error while loading.Please reload this page.
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.