Skip to content

Conversation

@burythehammer
Copy link
Collaborator

@burythehammer burythehammer commented Oct 8, 2025

Redis 8.0 Support

  • Add validation to prevent explicit module configuration for Redis 8.0+ - Modules are bundled by default in Redis 8.0, so users cannot explicitly set them. The provider now returns a validation error if users attempt to configure modules
    for Redis 8.0 databases.
  • Fix state drift for Redis 8.0 databases - The API returns all bundled modules for Redis 8.0 databases, but if modules weren't explicitly configured in Terraform, the provider no longer stores them in state. This prevents unnecessary
    destroy/recreate operations.
  • Add support for query_performance_factor on Redis 8.0 - Updated validation logic to allow QPF on Redis 8.0+ databases since RediSearch is bundled by default.
  • Fix subscription state handling after Redis version upgrades - Added wait for subscription to become active after upgrading Redis versions to prevent "SUBSCRIPTION_NOT_ACTIVE" errors during subsequent operations.

Test Improvements

  • Refactor inline Terraform configs to external files - Moved all inline Terraform configuration constants to .tf files in provider/pro/testdata/ for better maintainability and consistency.
  • Added Redis 8.0 tests:
    • TestAccResourceRedisCloudProDatabase_Redis8 - Tests provisioning a Redis 8.0 database
    • TestAccResourceRedisCloudProDatabase_Redis8_Upgrade - Tests upgrading from Redis 7.2 to 8.0
    • TestAccResourceRedisCloudProDatabase_Redis8_ModulesBlocked - Tests that modules cannot be explicitly set on Redis 8.0
  • Optimize test execution time - Reduced resource sizes in test configs and simplified update steps to focus on key changes.

General improvements

  • Refactored pro subscriptions to their own package for better organisation

@burythehammer burythehammer self-assigned this Oct 8, 2025
@burythehammer burythehammer changed the title OPCR-44 disable redis 8 modules Redis 8 Compatibility and Support Oct 9, 2025
@burythehammer burythehammer changed the base branch from redis-8-development to main October 10, 2025 16:03
@burythehammer burythehammer force-pushed the feat/OPCR-44-disable-modules branch from 47c55db to f3f294a Compare October 10, 2025 16:13
@burythehammer burythehammer merged commit 2619620 into main Oct 13, 2025
16 of 17 checks passed
@burythehammer burythehammer deleted the feat/OPCR-44-disable-modules branch October 13, 2025 15:55
burythehammer added a commit that referenced this pull request Nov 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants