Skip to content

Conversation

Siddarth-Baldwa
Copy link
Contributor

  1. GCP Backup Replication Target Update via Edit Cluster

    • Users can now update the backup_replication_gcp_target for GCP clusters through the Edit workflow. This enables flexible backup strategies for both SYNCHRONOUS and GEO_PARTITIONED clusters, with validation for cluster type, tier, and cloud provider.
  2. Cluster Data Source Schema Fixes

    • Added missing fields to the data source schema:
      • desired_connection_pooling_state
      • is_preferred
      • is_default
    • This resolves errors when fetching cluster data and ensures the Terraform state is populated correctly.
  3. Backup Region Visibility

    • Introduced the backup_region field to cluster region info, making it easy to identify which regions are designated as backup regions.

Copy link
Collaborator

@posriniv posriniv left a comment

Choose a reason for hiding this comment

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

Looks good to me!

Copy link

@artem-mindrov artem-mindrov left a comment

Choose a reason for hiding this comment

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

Please remove the extra logging before merging

@@ -1973,17 +2004,41 @@ func resourceClusterRead(ctx context.Context, accountId string, projectId string

tflog.Debug(ctx, fmt.Sprintf("For region %v, publicAccess = %v", region, publicAccess))

var backupReplicationGCPTarget types.String
tflog.Info(ctx, fmt.Sprintf("Siddarth printing repl target %v : %v , %v", cluster.ClusterName, info.GetBackupReplicationGcpTarget(), info.HasBackupReplicationGcpTarget()))

Choose a reason for hiding this comment

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

Suggested change
tflog.Info(ctx, fmt.Sprintf("Siddarth printing repl target %v : %v , %v", cluster.ClusterName, info.GetBackupReplicationGcpTarget(), info.HasBackupReplicationGcpTarget()))

@Siddarth-Baldwa Siddarth-Baldwa merged commit 83e0cd5 into main Aug 22, 2025
2 checks passed
@Siddarth-Baldwa Siddarth-Baldwa deleted the CLOUDGA-28235 branch August 22, 2025 05:07
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