fixed a permadiff in google_monitoring_uptime_check_config.http_check.path (#5818) (#11301)

Signed-off-by: Modular Magician <[email protected]>

Author: modular-magician

.changelog/5818.txt

@@ -0,0 +1,3 @@
+```release-note:bug
+monitoring: fixed a permadiff when `google_monitoring_uptime_check_config.http_check.path` does not begin with "/"
+```

google/resource_monitoring_uptime_check_config.go

@@ -24,6 +24,10 @@ import (
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
 )
 
+func resourceMonitoringUptimeCheckConfigHttpCheckPathDiffSuppress(k, old, new string, d *schema.ResourceData) bool {
+	return old == "/"+new
+}
+
 func resourceMonitoringUptimeCheckConfig() *schema.Resource {
 	return &schema.Resource{
 		Create: resourceMonitoringUptimeCheckConfigCreate,
@@ -128,11 +132,12 @@ func resourceMonitoringUptimeCheckConfig() *schema.Resource {
 							AtLeastOneOf: []string{"http_check.0.auth_info", "http_check.0.port", "http_check.0.headers", "http_check.0.path", "http_check.0.use_ssl", "http_check.0.mask_headers"},
 						},
 						"path": {
-							Type:         schema.TypeString,
-							Optional:     true,
-							Description:  `The path to the page to run the check against. Will be combined with the host (specified within the MonitoredResource) and port to construct the full URL. Optional (defaults to "/").`,
-							Default:      "/",
-							AtLeastOneOf: []string{"http_check.0.auth_info", "http_check.0.port", "http_check.0.headers", "http_check.0.path", "http_check.0.use_ssl", "http_check.0.mask_headers"},
+							Type:             schema.TypeString,
+							Optional:         true,
+							DiffSuppressFunc: resourceMonitoringUptimeCheckConfigHttpCheckPathDiffSuppress,
+							Description:      `The path to the page to run the check against. Will be combined with the host (specified within the MonitoredResource) and port to construct the full URL. If the provided path does not begin with "/", a "/" will be prepended automatically. Optional (defaults to "/").`,
+							Default:          "/",
+							AtLeastOneOf:     []string{"http_check.0.auth_info", "http_check.0.port", "http_check.0.headers", "http_check.0.path", "http_check.0.use_ssl", "http_check.0.mask_headers"},
 						},
 						"port": {
 							Type:         schema.TypeInt,

google/resource_monitoring_uptime_check_config_generated_test.go

@@ -55,7 +55,7 @@ resource "google_monitoring_uptime_check_config" "http" {
   timeout      = "60s"
 
   http_check {
-    path = "/some-path"
+    path = "some-path"
     port = "8010"
     request_method = "POST"
     content_type = "URL_ENCODED"

website/docs/r/monitoring_uptime_check_config.html.markdown

@@ -43,7 +43,7 @@ resource "google_monitoring_uptime_check_config" "http" {
   timeout      = "60s"
 
   http_check {
-    path = "/some-path"
+    path = "some-path"
     port = "8010"
     request_method = "POST"
     content_type = "URL_ENCODED"
@@ -214,7 +214,7 @@ The following arguments are supported:
 
 * `path` -
   (Optional)
-  The path to the page to run the check against. Will be combined with the host (specified within the MonitoredResource) and port to construct the full URL. Optional (defaults to "/").
+  The path to the page to run the check against. Will be combined with the host (specified within the MonitoredResource) and port to construct the full URL. If the provided path does not begin with "/", a "/" will be prepended automatically. Optional (defaults to "/").
 
 * `use_ssl` -
   (Optional)