Bug 2319777

Summary: Please branch and build php-pecl-redis6 for EPEL 10
Product: [Fedora] Fedora EPEL Reporter: Andrew Bauer <zonexpertconsulting>
Component: php-pecl-redis6Assignee: Remi Collet <fedora>
Status: CLOSED NOTABUG QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: epel10CC: carl, fedora, xavier
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2024-11-19 16:18:36 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 2303880, 2313934    

Description Andrew Bauer 2024-10-18 14:09:47 UTC
Would you please branch and build php-pecl-redis6  for EPEL10.

This is needed for nextcloud. Thank you.

I get it, if this is blocked by:
https://issues.redhat.com/browse/RHEL-55273

I'm creating this bug simply for tracking purposes.

Comment 1 Remi Collet 2024-10-18 15:02:13 UTC
Don't know yet, perhaps redis extension will be part of RHEL-10, waiting for more information.

Comment 2 Remi Collet 2024-10-18 15:03:42 UTC
*** Bug 2319780 has been marked as a duplicate of this bug. ***

Comment 3 Xavier Bachelot 2024-11-08 12:14:29 UTC
Roughtly one month later, php 8.3.12 is in CS10 and the redis extension is still not.
Shall php-pecl-redis6 be built now to unblock nextcloud ?

Comment 4 Remi Collet 2024-11-08 12:46:26 UTC
Still waiting for RHEL-10 status

BTW, such dependencies should be optional (Recommends)
Nextcloud works without redis.

Comment 5 Xavier Bachelot 2024-11-08 13:02:44 UTC
Yes, very good point, same for memcached and apcu.
That will solve the FTI and will automagically use redis when it'll become available either in CS/RHEL or EPEL.
Anyone to implement that or do you want a PR ?

Comment 6 Xavier Bachelot 2024-11-08 13:31:34 UTC
Also candidates for Recommends rather than Requires, as they are optional:
- php-intl
- php-sodium (already a Recommends)
- php-gmp
- php-exif
- php-imagick

Not currently a Requires or Recommends, the weakest Suggests might be better for them: 
- ffmpeg
- libreoffice

Comment 7 Andrew Bauer 2024-11-08 13:48:18 UTC
For the case of Nextcloud and redis, I would argue that it isn't that simple.

Yes, for brand  new installations with a default nextcloud config file, Nextcloud will continue to work. However, the admin panel will complain it is missing. 

For existing systems, already using redis, Nextcloud will crash w/o an obvious message as to the problem if, during an upgrade, the redis package goes away. While the fix would be to simply remove the redis entry from the config file, it will not be obvious to the average end user what the problem is, nor how to fix it.


If there is not an approximate idea when php-pecl-redis6 might become available, I would rather pull nextcloud from el10 repos until php-pecl-redis6 becomes available. Please let me know.

Comment 8 Remi Collet 2024-11-08 13:58:32 UTC
> - php-sodium (already a Recommends)

It is not even needed, also password argon2 which is available using OpenSSL 2 (PHP 8.4 feature backported to 8.3 in F41 and RHEL-10)


> If there is not an approximate idea when php-pecl-redis6 might become available

Before RHEL-10 GA

Comment 9 Remi Collet 2024-11-19 16:18:36 UTC
Package on its way to CentOS Stream 10 (and RHEL-10.0)

See: https://gitlab.com/redhat/centos-stream/rpms/php-pecl-redis6

So not needed in EPEL