Fix the out of boundary error (#9628) (#16792)

[upstream:3bacb448e960f8c56905e7012311953e920310e9] Signed-off-by: Modular Magician <magic-modules@google.com>
hashicorp · Dec 13, 2023 · 206fb3b · 206fb3b
1 parent b239600
commit 206fb3b
Show file tree

Hide file tree

Showing 3 changed files with 59 additions and 1 deletion.
diff --git a/.changelog/9628.txt b/.changelog/9628.txt
@@ -0,0 +1,3 @@
+```release-note:bug
+monitoring: fixed the index out of range crash in `dashboard_json` perma-diffs for the resource `google_monitoring_dashboard`
+```
diff --git a/google/services/monitoring/resource_monitoring_dashboard.go b/google/services/monitoring/resource_monitoring_dashboard.go
@@ -33,7 +33,7 @@ func removeComputedKeys(old map[string]interface{}, new map[string]interface{})
 
 		if reflect.ValueOf(v).Kind() == reflect.Slice {
 			for i, j := range v.([]interface{}) {
-				if reflect.ValueOf(j).Kind() == reflect.Map {
+				if reflect.ValueOf(j).Kind() == reflect.Map && len(new[k].([]interface{})) > i {
 					old[k].([]interface{})[i] = removeComputedKeys(j.(map[string]interface{}), new[k].([]interface{})[i].(map[string]interface{}))
 				}
 			}

diff --git a/google/services/monitoring/resource_monitoring_dashboard_test.go b/google/services/monitoring/resource_monitoring_dashboard_test.go
@@ -119,6 +119,15 @@ func TestAccMonitoringDashboard_update(t *testing.T) {
 				ImportStateVerify:       true,
 				ImportStateVerifyIgnore: []string{"project"},
 			},
+			{
+				Config: testAccMonitoringDashboard_gridLayoutUpdate(),
+			},
+			{
+				ResourceName:            "google_monitoring_dashboard.dashboard",
+				ImportState:             true,
+				ImportStateVerify:       true,
+				ImportStateVerifyIgnore: []string{"project"},
+			},
 		},
 	})
 }
@@ -244,6 +253,52 @@ EOF
 `)
 }
 
+func testAccMonitoringDashboard_gridLayoutUpdate() string {
+	return fmt.Sprintf(`
+resource "google_monitoring_dashboard" "dashboard" {
+  dashboard_json = <<EOF
+{
+  "displayName": "Grid Layout Example",
+  "gridLayout": {
+    "columns": "2",
+    "widgets": [
+      {
+        "title": "Widget 1",
+        "xyChart": {
+          "dataSets": [{
+            "timeSeriesQuery": {
+              "timeSeriesFilter": {
+                "filter": "metric.type=\"agent.googleapis.com/nginx/connections/accepted_count\"",
+                "aggregation": {
+                  "perSeriesAligner": "ALIGN_RATE"
+                }
+              },
+              "unitOverride": "1"
+            },
+            "plotType": "LINE"
+          }],
+          "timeshiftDuration": "0s",
+          "yAxis": {
+            "label": "y1Axis",
+            "scale": "LINEAR"
+          }
+        }
+      },
+      {
+        "text": {
+          "content": "Widget 2",
+          "format": "MARKDOWN"
+        }
+      }
+    ]
+  }
+}
+
+EOF
+}
+`)
+}
+
 func testAccMonitoringDashboard_rowLayout() string {
 	return fmt.Sprintf(`
 resource "google_monitoring_dashboard" "dashboard" {