fix: revert essentials database source ip fix as this will be fixed API side

Author: burythehammer

CHANGELOG.md

@@ -6,11 +6,15 @@ See updating [Changelog example here](https://keepachangelog.com/en/1.0.0/)
 # 2.7.0 (22nd October 2025)
 
 ## Added:
-- Support for `auto_minor_version_upgrade` attribute on Pro and Active-Active databases. When set to `true`, enables automatic minor version upgrades for Redis databases.
+- Add auto_minor_version_upgrade field to Pro and Active-Active database resources (default: true) to allow users to control automatic minor version upgrades. This will NOT affect existing databases.
+
+## Changed:
+- Change Redis 8.0 modules validation from hard error to warning since modules are bundled by default in Redis 8+.
 
 ## Fixed:
-- Fixed module validation causing errors for Redis 8. It will now show a warning instead and ignore the modules.
-- Fixed `source_ips` being sent to the API for essentials databases on free plans, which caused errors. The `source_ips` attribute is now only sent for PAYG (Pay-As-You-Go) subscriptions when `enable_payg_features` is true.
+- Fix test error message patterns to match updated API error format.
+- Fix Redis 8 upgrade test expectation (dataset_size_in_gb: 3→1).
+
 
 # 2.6.0 (17th October 2025)
 

provider/resource_rediscloud_essentials_database.go

@@ -138,10 +138,10 @@ func resourceRedisCloudEssentialsDatabase() *schema.Resource {
 				Computed:    true,
 			},
 			"source_ips": {
-				Description:      "Set of CIDR addresses to allow access to the database. Supported only for 'Pay-As-You-Go' subscriptions",
-				Type:             schema.TypeList,
-				Optional:         true,
-				MinItems:         1,
+				Description: "Set of CIDR addresses to allow access to the database.",
+				Type:        schema.TypeList,
+				Optional:    true,
+				MinItems:    1,
 				Elem: &schema.Schema{
 					Type:             schema.TypeString,
 					ValidateDiagFunc: validation.ToDiagFunc(validation.IsCIDR),
@@ -307,7 +307,6 @@ func resourceRedisCloudEssentialsDatabaseCreate(ctx context.Context, d *schema.R
 	api := meta.(*client.ApiClient)
 
 	subId := d.Get("subscription_id").(int)
-	log.Printf("[DEBUG] Creating essentials database in subscription %d", subId)
 
 	utils.SubscriptionMutex.Lock(subId)
 
@@ -318,11 +317,6 @@ func resourceRedisCloudEssentialsDatabaseCreate(ctx context.Context, d *schema.R
 		Replication:        redis.Bool(d.Get("replication").(bool)),
 		PeriodicBackupPath: redis.String(d.Get("periodic_backup_path").(string)),
 	}
-	log.Printf("[DEBUG] Essentials database create request - Name: %s, DataPersistence: %s, DataEviction: %s, Replication: %v",
-		redis.StringValue(createDatabaseRequest.Name),
-		redis.StringValue(createDatabaseRequest.DataPersistence),
-		redis.StringValue(createDatabaseRequest.DataEvictionPolicy),
-		redis.BoolValue(createDatabaseRequest.Replication))
 
 	protocol := d.Get("protocol").(string)
 	if protocol != "" {
@@ -334,14 +328,11 @@ func resourceRedisCloudEssentialsDatabaseCreate(ctx context.Context, d *schema.R
 		createDatabaseRequest.RespVersion = redis.String(respVersion)
 	}
 
-	// Only send source_ips for PAYG plans - free plans don't support this feature
-	if d.Get("enable_payg_features").(bool) {
-		sourceIps := utils.InterfaceToStringSlice(d.Get("source_ips").([]interface{}))
-		if len(sourceIps) == 0 {
-			createDatabaseRequest.SourceIPs = []*string{redis.String("0.0.0.0/0")}
-		} else {
-			createDatabaseRequest.SourceIPs = sourceIps
-		}
+	sourceIps := utils.InterfaceToStringSlice(d.Get("source_ips").([]interface{}))
+	if len(sourceIps) == 0 {
+		createDatabaseRequest.SourceIPs = []*string{redis.String("0.0.0.0/0")}
+	} else {
+		createDatabaseRequest.SourceIPs = sourceIps
 	}
 
 	replicaRaw := d.Get("replica").([]interface{})
@@ -642,14 +633,11 @@ func resourceRedisCloudEssentialsDatabaseUpdate(ctx context.Context, d *schema.R
 		updateDatabaseRequest.RespVersion = redis.String(respVersion)
 	}
 
-	// Only send source_ips for PAYG plans - free plans don't support this feature
-	if d.Get("enable_payg_features").(bool) {
-		sourceIps := utils.InterfaceToStringSlice(d.Get("source_ips").([]interface{}))
-		if len(sourceIps) == 0 {
-			updateDatabaseRequest.SourceIPs = []*string{redis.String("0.0.0.0/0")}
-		} else {
-			updateDatabaseRequest.SourceIPs = sourceIps
-		}
+	sourceIps := utils.InterfaceToStringSlice(d.Get("source_ips").([]interface{}))
+	if len(sourceIps) == 0 {
+		updateDatabaseRequest.SourceIPs = []*string{redis.String("0.0.0.0/0")}
+	} else {
+		updateDatabaseRequest.SourceIPs = sourceIps
 	}
 
 	replicaRaw := d.Get("replica").([]interface{})