Skip to content

Conversation

scottbuscemi
Copy link
Contributor

Summary

https://www.drupal.org/project/redis/releases/8.x-1.10

The latest version of the Redis module fixes an issue that we were previously suggesting a patch for.

@scottbuscemi scottbuscemi requested a review from a team as a code owner August 1, 2025 18:13
Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-9635-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

@lcatlett
Copy link
Member

lcatlett commented Aug 3, 2025

A few things folks should be aware of:

  • This shouldn't be merged until the settings snippets are updated to account for new settings like https://git.drupalcode.org/project/redis/-/commit/ff3890a4da59101d0d59507979a22a017e5a7d66#e81e5be2706877eff882fb5ee8272aa85817cdac_41_42.

  • This release is not just the patch that has now been tested extensively on Pantheon. There are changes to recommended defaults such as eviction policy that have never been tested with any customer set up. A significant example is volatile-lfu is now the recommended default eviction policy, so that's going to either need platform changes, or additional work in the form of a custom module to enforce.

  • Eviction policy is one example but there are multiple critical changes that need to be tested both on setting up redis for the first time on a site, and upgrading existing sites to use these new capabilities / recs

  • the above needs to be coordinated with the d11 roadmap for cache enhancements. Settings like $settings['cache_preload_tags'] are not compatible with the Drupal redis module currently https://www.drupal.org/node/3505248

@scottbuscemi scottbuscemi marked this pull request as draft August 3, 2025 18:55
Copy link
Contributor

@stevector stevector left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like we should address @lcatlett comments before merging #9635 (comment)

@rachelwhitton rachelwhitton added Type: Fix Content Issue or PR to resolve incorrect information in the docs Topic: Drupal Topic: Caching Caching related issues at the CDN or CMS level labels Aug 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Topic: Caching Caching related issues at the CDN or CMS level Topic: Drupal Type: Fix Content Issue or PR to resolve incorrect information in the docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants