Fixes PermaDiff Issue in google_storage_transfer_job.aws_s3_data_source.aws_access_key field (GoogleCloudPlatform#12666)

kautikdk · anoopkverma-google · commit 3e0a969d767d · 2025-01-31T12:17:53.000+05:30
diff --git a/mmv1/third_party/terraform/services/storagetransfer/resource_storage_transfer_job.go.tmpl b/mmv1/third_party/terraform/services/storagetransfer/resource_storage_transfer_job.go.tmpl
@@ -1150,7 +1150,7 @@ func flattenAwsS3Data(awsS3Data *storagetransfer.AwsS3Data, d *schema.ResourceDa
 		"path":        awsS3Data.Path,
 		"role_arn":    awsS3Data.RoleArn,
 	}
-	if _, exist := d.GetOkExists("transfer_spec.0.aws_s3_data_source.0.aws_access_key"); exist{
+	if _, exist := d.GetOk("transfer_spec.0.aws_s3_data_source.0.aws_access_key"); exist{
 		data["aws_access_key"] = flattenAwsAccessKeys(d)
 	}
 	return []map[string]interface{}{data}

Original file line number	Diff line number	Diff line change
`@@ -1150,7 +1150,7 @@ func flattenAwsS3Data(awsS3Data storagetransfer.AwsS3Data, d schema.ResourceDa`
`1150`	`1150`	`"path": awsS3Data.Path,`
`1151`	`1151`	`"role_arn": awsS3Data.RoleArn,`
`1152`	`1152`	`}`
`1153`		`- if _, exist := d.GetOkExists("transfer_spec.0.aws_s3_data_source.0.aws_access_key"); exist{`
	`1153`	`+ if _, exist := d.GetOk("transfer_spec.0.aws_s3_data_source.0.aws_access_key"); exist{`
`1154`	`1154`	`data["aws_access_key"] = flattenAwsAccessKeys(d)`
`1155`	`1155`	`}`
`1156`	`1156`	`return []map[string]interface{}{data}`