Skip to content

Commit e89922e

Browse files
authored
instance_cluster/group_cluster: Suppress whitespace diff for kubernetes_ca_cert (#728)
* instance_cluster/group_cluster: Suppress whitespace diff for kubernetes_ca_cert * Add strings import
1 parent 7676486 commit e89922e

File tree

4 files changed

+13
-5
lines changed

4 files changed

+13
-5
lines changed

gitlab/resource_gitlab_group_cluster.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import (
44
"fmt"
55
"log"
66
"strconv"
7+
"strings"
78

89
"github.com/hashicorp/terraform-plugin-sdk/helper/schema"
910
"github.com/hashicorp/terraform-plugin-sdk/helper/validation"
@@ -79,6 +80,9 @@ func resourceGitlabGroupCluster() *schema.Resource {
7980
"kubernetes_ca_cert": {
8081
Type: schema.TypeString,
8182
Optional: true,
83+
DiffSuppressFunc: func(k, old, new string, d *schema.ResourceData) bool {
84+
return strings.TrimSpace(old) == strings.TrimSpace(new)
85+
},
8286
},
8387
"kubernetes_authorization_type": {
8488
Type: schema.TypeString,

gitlab/resource_gitlab_group_cluster_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -242,7 +242,7 @@ resource gitlab_group_cluster "foo" {
242242
managed = "%s"
243243
kubernetes_api_url = "https://123.123.123"
244244
kubernetes_token = "some-token"
245-
kubernetes_ca_cert = "${trimspace(var.cert)}"
245+
kubernetes_ca_cert = var.cert
246246
kubernetes_authorization_type = "abac"
247247
}
248248
`, groupClusterFakeCert, rInt, rInt, rInt, m)
@@ -272,7 +272,7 @@ resource gitlab_group_cluster "foo" {
272272
domain = "example-new.com"
273273
kubernetes_api_url = "https://124.124.124"
274274
kubernetes_token = "some-token"
275-
kubernetes_ca_cert = "${trimspace(var.cert)}"
275+
kubernetes_ca_cert = var.cert
276276
kubernetes_authorization_type = "%s"
277277
}
278278
`, groupClusterFakeCert, rInt, rInt, rInt, authType)
@@ -313,7 +313,7 @@ resource gitlab_group_cluster "foo" {
313313
managed = "%s"
314314
kubernetes_api_url = "https://123.123.123"
315315
kubernetes_token = "some-token"
316-
kubernetes_ca_cert = "${trimspace(var.cert)}"
316+
kubernetes_ca_cert = var.cert
317317
kubernetes_authorization_type = "abac"
318318
management_project_id = "${gitlab_project.cluster-management.id}"
319319
}

gitlab/resource_gitlab_instance_cluster.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import (
44
"fmt"
55
"log"
66
"strconv"
7+
"strings"
78

89
"github.com/hashicorp/terraform-plugin-sdk/helper/schema"
910
"github.com/hashicorp/terraform-plugin-sdk/helper/validation"
@@ -74,6 +75,9 @@ func resourceGitlabInstanceCluster() *schema.Resource {
7475
"kubernetes_ca_cert": {
7576
Type: schema.TypeString,
7677
Optional: true,
78+
DiffSuppressFunc: func(k, old, new string, d *schema.ResourceData) bool {
79+
return strings.TrimSpace(old) == strings.TrimSpace(new)
80+
},
7781
},
7882
"kubernetes_namespace": {
7983
Type: schema.TypeString,

gitlab/resource_gitlab_instance_cluster_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,7 @@ resource gitlab_instance_cluster "foo" {
232232
managed = "%s"
233233
kubernetes_api_url = "https://123.123.123"
234234
kubernetes_token = "some-token"
235-
kubernetes_ca_cert = "${trimspace(var.cert)}"
235+
kubernetes_ca_cert = var.cert
236236
kubernetes_authorization_type = "abac"
237237
}
238238
`, instanceClusterFakeCert, rInt, m)
@@ -251,7 +251,7 @@ resource gitlab_instance_cluster "foo" {
251251
domain = "example-new.com"
252252
kubernetes_api_url = "https://124.124.124"
253253
kubernetes_token = "some-token"
254-
kubernetes_ca_cert = "${trimspace(var.cert)}"
254+
kubernetes_ca_cert = var.cert
255255
kubernetes_authorization_type = "%s"
256256
}
257257
`, instanceClusterFakeCert, rInt, authType)

0 commit comments

Comments
 (0)