Upgrade Grails Cache Redis Plugin and Prepare for New Grails 6.x Release #9

sbglasius · 2025-06-11T08:01:07Z

This PR includes a series of updates and improvements to the Grails Cache Redis plugin in preparation for a new release. Key changes:

🛠️ Framework and Dependency Updates
Upgraded Grails to 6.2.x

Updated Gradle and associated build dependencies to align with newer Grails versions

✅ Functional Improvements
Implemented get() method to satisfy the Cache interface

Allowed cache-redis to be disabled independently of grails-cache

Replaced deprecated JedisShardInfo with RedisStandaloneConfiguration for better compatibility with newer Redis clients

📘 Documentation
Updated plugin documentation to reflect recent changes and usage patterns

These are the versions we're using for most of our Spieker Point applications.

I assume that the `get(String, Callable<T>)` method signature was added to Spring Framework's Cache interface sometime after this cache-redis plugin was originally developed. For now we'll just ignore the Callable argument, since it isn't having an impact on our usage.

There is now a separate `grails.cache.redis.enabled` config option to allow the cache-redis plugin to be disabled while still having the grails-cache plugin enabled. The change is based off of grails-plugins#8

This is related to ARKLEM-846.

* All functional tests now uses Micronaut Http Client * Using test-containers for Redis instance * Updating documentation

Copilot

Pull Request Overview

This PR upgrades the Grails Cache Redis plugin and its test suite for Grails 6.x by modernizing build scripts, configs, documentation, and CI/CD.

Replace Groovy-based logging with XML Logback config and adjust application YAML.
Revamp Gradle wrapper scripts (gradlew, gradlew.bat) and bump to Gradle 7.6.4; update plugin/dependency versions.
Add Apache 2.0 license, update README, and introduce GitHub Actions workflows (CI, release, release notes, Renovate).

Reviewed Changes

Copilot reviewed 77 out of 77 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
functional-tests/grails-app/conf/logback.xml	Added XML logging config for functional tests
functional-tests/grails-app/conf/logback.groovy	Removed old Groovy logging config
functional-tests/grails-app/conf/application.yml	Updated datasource settings for all environments
functional-tests/grails-app/conf/TestCacheConfig.groovy	Removed test cache configuration
functional-tests/gradlew.bat	Revamped Windows Gradle wrapper script
functional-tests/gradlew	Revamped Unix Gradle wrapper script
functional-tests/gradle/wrapper/gradle-wrapper.properties	Bumped Gradle distribution to 7.6.4
functional-tests/gradle.properties	Updated Grails and Gradle plugin versions
functional-tests/build.gradle	Migrated build script to modern Gradle DSL
functional-tests/.gitignore	Added `/bin/` to ignore list
buildSrc/settings.gradle	Configured version catalogs for buildSrc
buildSrc/build.gradle	Added buildSrc plugin dependencies
README.md	Updated badges, documentation links, and contributing
LICENSE	Added Apache 2.0 license text
.travis.yml	Removed Travis CI config
.github/workflows/release.yml	Added release GitHub Action workflow
.github/workflows/release-notes.yml	Added release-drafter workflow
.github/workflows/gradle.yml	Added Java CI workflow
.github/renovate.json	Added Renovate bot configuration
.github/release-drafter.yml	Added release drafter configuration

Comments suppressed due to low confidence (3)

functional-tests/grails-app/conf/TestCacheConfig.groovy:1

The test cache configuration was removed entirely; ensure that critical cache behaviors (e.g., eviction and persistence settings) are still covered by existing tests or add new ones.

config = {

functional-tests/grails-app/conf/application.yml:45

[nitpick] Using a magic number (2) for isolation level is unclear; consider re-adding or updating the comment (e.g., # TRANSACTION_READ_COMMITTED) for readability.

defaultTransactionIsolation: 2

functional-tests/gradlew.bat:74

The original %CMD_LINE_ARGS% was replaced with %*, but the -Dorg.gradle.appname JVM arg and potentially other options may not be applied correctly on Windows; please verify argument propagation and restore missing flags if needed.

"%JAVA_EXE%" %DEFAULT_JVM_OPTS% %JAVA_OPTS% %GRADLE_OPTS% "-Dorg.gradle.appname=%APP_BASE_NAME%" -classpath "%CLASSPATH%" org.gradle.wrapper.GradleWrapperMain %*

sbglasius · 2025-06-17T07:55:17Z

@matrei any chance you have time to look at this so that I can release a RC1 or perhaps even a GA?

matrei · 2025-06-11T10:44:21Z

src/main/groovy/grails/plugin/cache/redis/GrailsRedisCache.java

     * @param name        cache name
-     * @param prefix
-     * @param template
-     * @param ttl
+     * @param prefix prefix to use
+     * @param template Spring template
+     * @param ttl time to live for cache


Can we align the space usage for all the @params?

matrei · 2025-06-17T08:31:09Z

gradle/buildsrc.libs.versions.toml

@@ -0,0 +1,9 @@
+[versions]
+grails-gradle-plugin = '6.2.0'


Update to 6.2.4?

matrei · 2025-06-17T08:32:53Z

gradle/buildsrc.libs.versions.toml

@@ -0,0 +1,9 @@
+[versions]
+grails-gradle-plugin = '6.2.0'
+nexus-publish-gradle-plugin = '1.3.0'


Update to 2.0.0?

matrei · 2025-06-17T08:33:57Z

gradle/buildsrc.libs.versions.toml

+[versions]
+grails-gradle-plugin = '6.2.0'
+nexus-publish-gradle-plugin = '1.3.0'
+asciidoctor-gradle-plugin = '4.0.2'


Update to 4.0.4?

matrei · 2025-06-17T08:39:15Z

gradle/grails-plugin-config.gradle

+tasks.named('jar', Jar) {
+    enabled = true // Enable the jar task again, as the bootJar task has been disabled
+    archiveClassifier = '' // Remove '-plain' suffix from jar file name
+    exclude('_testemails', 'messages*.properties')


Remove? (This looks like leftover from copying from grails-mail)

matrei · 2025-06-17T08:41:39Z

gradle.properties

-gradleWrapperVersion=2.13
+projectVersion=6.0.0-SNAPSHOT
+grailsVersion=6.2.3
+javaVersion=17


Remove? This seems unused and also has the wrong value.

matrei · 2025-06-17T08:41:53Z

gradle.properties

+projectVersion=6.0.0-SNAPSHOT
+grailsVersion=6.2.3
+javaVersion=17
+asciidoctorGradlePluginVersion=4.0.4


Remove? This seems unused and the value is set in buildsrc.libs.versions.toml.

matrei · 2025-06-17T08:44:04Z

gradle.properties

+javaVersion=17
+asciidoctorGradlePluginVersion=4.0.4
+
+grailsGradlePluginVersion=6.2.4


Remove? This seems unused and the value is set in buildsrc.libs.versions.toml.

matrei · 2025-06-17T08:45:56Z

gradle.properties

+asciidoctorGradlePluginVersion=4.0.4
+
+grailsGradlePluginVersion=6.2.4
+version=0.1


Remove? This seems unused and also has the wrong value.

matrei · 2025-06-17T08:48:16Z

gradle/publishing.gradle

+    publications {
+        register('grailsMailPlugin', MavenPublication) {
+            from javaComponent.get()
+            versionMapping {


I think we can remove this versionMapping block.

sbglasius · 2025-06-18T11:19:22Z

@matrei see #13 for post-review fixes

erik-h and others added 7 commits March 9, 2021 18:12

Update Grails and Gradle versions

a0ee3e9

These are the versions we're using for most of our Spieker Point applications.

Allow cache-redis to be disabled separately from grails-cache

ffec16f

There is now a separate `grails.cache.redis.enabled` config option to allow the cache-redis plugin to be disabled while still having the grails-cache plugin enabled. The change is based off of grails-plugins#8

Fix a bug in this plugin that is causing file upload to fail in DECK.

ef222eb

This is related to ARKLEM-846.

Prepare for 2.0.1.BUILD-SNAPSHOT release.

1ed40f2

Upgraded to Grails 6.2.x

e091fe9

* All functional tests now uses Micronaut Http Client * Using test-containers for Redis instance * Updating documentation

Replaced deprecated JedisShardInfo with RedisStandaloneConfiguration

a0da3be

sbglasius requested a review from matrei June 11, 2025 08:01

Updated build logic for new release

a2dde6e

sbglasius force-pushed the 6.x branch from 5442e71 to a2dde6e Compare June 11, 2025 08:25

sbglasius added 2 commits June 11, 2025 10:45

Moved support classes into org.grails.cache.redis.internal

f029989

Fixed javadoc warnings

fc69658

sbglasius requested a review from Copilot June 11, 2025 09:02

sbglasius self-assigned this Jun 11, 2025

Copilot AI reviewed Jun 11, 2025

View reviewed changes

sbglasius merged commit 0ea3bbd into grails-plugins:6.x Jun 11, 2025

matrei reviewed Jun 17, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Upgrade Grails Cache Redis Plugin and Prepare for New Grails 6.x Release #9

Upgrade Grails Cache Redis Plugin and Prepare for New Grails 6.x Release #9

Uh oh!

sbglasius commented Jun 11, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

sbglasius commented Jun 17, 2025

Uh oh!

matrei Jun 11, 2025

Uh oh!

matrei Jun 17, 2025

Uh oh!

matrei Jun 17, 2025

Uh oh!

matrei Jun 17, 2025

Uh oh!

matrei Jun 17, 2025

Uh oh!

matrei Jun 17, 2025

Uh oh!

matrei Jun 17, 2025

Uh oh!

matrei Jun 17, 2025

Uh oh!

matrei Jun 17, 2025

Uh oh!

matrei Jun 17, 2025

Uh oh!

sbglasius commented Jun 18, 2025

Uh oh!

Uh oh!

Upgrade Grails Cache Redis Plugin and Prepare for New Grails 6.x Release #9

Upgrade Grails Cache Redis Plugin and Prepare for New Grails 6.x Release #9

Uh oh!

Conversation

sbglasius commented Jun 11, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

sbglasius commented Jun 17, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sbglasius commented Jun 18, 2025

Uh oh!

Uh oh!