Fix absoluteDomainSuppress to check more than one domain (#3711) (#2241)

Signed-off-by: Modular Magician <magic-modules@google.com>
hashicorp · Jun 26, 2020 · 9eb8807 · 9eb8807
1 parent 0eddadb
commit 9eb8807
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/.changelog/3711.txt b/.changelog/3711.txt
@@ -0,0 +1,3 @@
+```release-note:bug
+compute: Fixed an issue in `google_compute_managed_ssl_certificate` where multiple fully qualified domain names would cause a permadiff
+```
diff --git a/google-beta/common_diff_suppress.go b/google-beta/common_diff_suppress.go
@@ -79,7 +79,7 @@ func rfc3339TimeDiffSuppress(k, old, new string, d *schema.ResourceData) bool {
 
 // For managed SSL certs, if new is an absolute FQDN (trailing '.') but old isn't, treat them as equals.
 func absoluteDomainSuppress(k, old, new string, _ *schema.ResourceData) bool {
-	if k == "managed.0.domains.0" {
+	if strings.HasPrefix(k, "managed.0.domains.") {
 		return old == strings.TrimRight(new, ".")
 	}
 	return old == new