Do not diffSuppress port when port_specification is not "US… (#3316…

…) (#1906) * Do not diffSuppress `port` when `port_specification` is "USED_NAME_PORT" * Formatting * Update health_check.erb Signed-off-by: Modular Magician <magic-modules@google.com>
hashicorp · Mar 27, 2020 · 7189fae · 7189fae
1 parent d833ee1
commit 7189fae
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/.changelog/3316.txt b/.changelog/3316.txt
@@ -0,0 +1,3 @@
+```release-note:bug
+compute: Fixed an issue where `port` could not be removed from health checks
+```
diff --git a/google-beta/resource_compute_health_check.go b/google-beta/resource_compute_health_check.go
@@ -73,7 +73,7 @@ func healthCheckCustomizeDiff(diff *schema.ResourceDiff, v interface{}) error {
 	return nil
 }
 
-func portDiffSuppress(k, old, new string, _ *schema.ResourceData) bool {
+func portDiffSuppress(k, old, new string, d *schema.ResourceData) bool {
 	b := strings.Split(k, ".")
 	if len(b) > 2 {
 		attr := b[2]
@@ -99,7 +99,8 @@ func portDiffSuppress(k, old, new string, _ *schema.ResourceData) bool {
 			oldPort, _ := strconv.Atoi(old)
 			newPort, _ := strconv.Atoi(new)
 
-			if int64(oldPort) == defaultPort && newPort == 0 {
+			portSpec := d.Get(b[0] + ".0.port_specification")
+			if int64(oldPort) == defaultPort && newPort == 0 && portSpec == "USE_FIXED_PORT" {
 				return true
 			}
 		}