Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[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

@alexander-schranz
Copy link
Contributor

@alexander-schranzalexander-schranz commentedOct 24, 2025
edited
Loading

QA
Branch?7.4
Bug fix?no
New feature?no
Deprecations?no
IssuesFix #...
LicenseMIT

Ubuntu 25.04which is Ubuntus current and latest LTS is still on Redis Extension 6.1.0-3:https://packages.ubuntu.com/search?keywords=php8.4-redis&searchon=names&suite=plucky&section=all

As discussed in#62146 6.0 with@xabbuh is not possible to support without changes but 6.1 should.

@carsonbot
Copy link

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

@xabbuhxabbuh added the Cache labelOct 24, 2025
@carsonbotcarsonbot changed the titleOpen Symfony Cache Ext-Redis Requirement to ^6.1 for Ubuntu 25.04 LTS[Cache] Open Symfony Cache Ext-Redis Requirement to ^6.1 for Ubuntu 25.04 LTSOct 24, 2025
@alexander-schranzalexander-schranzforce-pushed theenhancement/open-ext-redis-requirement-to-6.0 branch from424611c tod888d6eCompareOctober 24, 2025 12:47
@alexander-schranzalexander-schranz changed the title[Cache] Open Symfony Cache Ext-Redis Requirement to ^6.1 for Ubuntu 25.04 LTS[Cache][Messsenger] Open Symfony Cache and Symfony Messenger Redis Ext-Redis Requirement to ^6.1 for Ubuntu 25.04 LTSOct 24, 2025
@alexander-schranz
Copy link
ContributorAuthor

alexander-schranz commentedOct 24, 2025
edited
Loading

Ind888d6e which I reverted to not change any CI specific things:https://github.com/symfony/symfony/actions/runs/18780211842/job/53584212483
The CI runs against 6.1.0 Extension.

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?

@alexander-schranzalexander-schranz marked this pull request as ready for reviewOctober 24, 2025 12:55
@carsonbotcarsonbot added this to the7.4 milestoneOct 24, 2025
@xabbuh
Copy link
Member

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.

alexander-schranz reacted with thumbs up emoji

@alexislefebvre

This comment was marked as resolved.

@alexander-schranzalexander-schranz changed the title[Cache][Messsenger] Open Symfony Cache and Symfony Messenger Redis Ext-Redis Requirement to ^6.1 for Ubuntu 25.04 LTS[Cache][Messsenger] Open Symfony Cache and Symfony Messenger Redis Ext-Redis Requirement to ^6.1 for Ubuntu 25.04Oct 29, 2025
@xabbuh
Copy link
Member

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.

we indeed have compatibility issues:#62222

@xabbuh
Copy link
Member

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&section=all&arch=any&keywords=php8.3-redis&searchon=names

@xabbuh
Copy link
Member

xabbuh commentedOct 30, 2025
edited
Loading

@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)?

nicolas-grekas reacted with thumbs up emoji

@alexander-schranz
Copy link
ContributorAuthor

@xabbuh sounds okay for me, we still could lower later the requirement if we see lot of reports about it.

xabbuh reacted with thumbs up emoji

@alexander-schranzalexander-schranz deleted the enhancement/open-ext-redis-requirement-to-6.0 branchOctober 31, 2025 07:47
@xabbuh
Copy link
Member

after some more feedback: see#62326

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

7.4

Development

Successfully merging this pull request may close these issues.

5 participants

@alexander-schranz@carsonbot@xabbuh@alexislefebvre@nicolas-grekas

[8]ページ先頭

©2009-2025 Movatter.jp