mockgcp: monitoringDashboard should default minAlignmentPeriod in tim…

…eSeriesTable
gemmahou · Jul 1, 2024 · 0f2be4c · 0f2be4c
1 parent f527f6e
commit 0f2be4c
Show file tree

Hide file tree

Showing 4 changed files with 28 additions and 5 deletions.
diff --git a/mockgcp/mockmonitoring/dashboard.go b/mockgcp/mockmonitoring/dashboard.go
@@ -18,6 +18,7 @@ import (
 	"context"
 	"fmt"
 	"strings"
+	"time"
 
 	"google.golang.org/grpc/codes"
 	"google.golang.org/grpc/status"
@@ -123,6 +124,12 @@ func (d *dashboardDefaulter) visitWidget(obj *pb.Widget) {
 
 	case *pb.Widget_Text:
 		d.visitTextWidget(content)
+
+	case *pb.Widget_PieChart:
+		d.visitPieChart(content.PieChart)
+
+	case *pb.Widget_TimeSeriesTable:
+		d.visitTimeSeriesTable(content.TimeSeriesTable)
 	}
 }
 
@@ -153,6 +160,17 @@ func (d *dashboardDefaulter) visitTextWidget(obj *pb.Widget_Text) {
 	}
 }
 
+func (d *dashboardDefaulter) visitPieChart(obj *pb.PieChart) {
+}
+
+func (d *dashboardDefaulter) visitTimeSeriesTable(obj *pb.TimeSeriesTable) {
+	for _, dataSet := range obj.DataSets {
+		if dataSet.MinAlignmentPeriod == nil {
+			dataSet.MinAlignmentPeriod = durationpb.New(time.Duration(0))
+		}
+	}
+}
+
 type dashboardValidator struct {
 	errors []error
 }

diff --git a/...itoringdashboard/monitoringdashboardfull/_generated_export_monitoringdashboardfull.golden b/...itoringdashboard/monitoringdashboardfull/_generated_export_monitoringdashboardfull.golden
@@ -106,7 +106,8 @@ spec:
           - column: column2
             visible: false
           dataSets:
-          - timeSeriesQuery:
+          - minAlignmentPeriod: 0s
+            timeSeriesQuery:
               timeSeriesFilter:
                 aggregation:
                   alignmentPeriod: "60"

diff --git a/...e/testdata/basic/monitoring/v1beta1/monitoringdashboard/monitoringdashboardfull/_http.log b/...e/testdata/basic/monitoring/v1beta1/monitoringdashboard/monitoringdashboardfull/_http.log
@@ -578,6 +578,7 @@ X-Xss-Protection: 0
               ],
               "dataSets": [
                 {
+                  "minAlignmentPeriod": "0s",
                   "timeSeriesQuery": {
                     "timeSeriesFilter": {
                       "aggregation": {
@@ -812,6 +813,7 @@ X-Xss-Protection: 0
               ],
               "dataSets": [
                 {
+                  "minAlignmentPeriod": "0s",
                   "timeSeriesQuery": {
                     "timeSeriesFilter": {
                       "aggregation": {
@@ -1261,6 +1263,7 @@ X-Xss-Protection: 0
               ],
               "dataSets": [
                 {
+                  "minAlignmentPeriod": "0s",
                   "timeSeriesQuery": {
                     "timeSeriesFilter": {
                       "aggregation": {
@@ -1495,6 +1498,7 @@ X-Xss-Protection: 0
               ],
               "dataSets": [
                 {
+                  "minAlignmentPeriod": "0s",
                   "timeSeriesQuery": {
                     "timeSeriesFilter": {
                       "aggregation": {

diff --git a/...ta/basic/monitoring/v1beta1/monitoringdashboard/monitoringdashboardmosaiclayout/_http.log b/...ta/basic/monitoring/v1beta1/monitoringdashboard/monitoringdashboardmosaiclayout/_http.log
@@ -297,7 +297,7 @@ X-Xss-Protection: 0
         "widget": {
           "text": {
             "content": "CollapsibleGroup content",
-            "format": "MARKDOWN",
+            "format": 1,
             "style": {}
           }
         },
@@ -370,7 +370,7 @@ X-Xss-Protection: 0
         "widget": {
           "text": {
             "content": "CollapsibleGroup content",
-            "format": "MARKDOWN",
+            "format": 1,
             "style": {}
           }
         },
@@ -496,7 +496,7 @@ X-Xss-Protection: 0
         "widget": {
           "text": {
             "content": "CollapsibleGroup content",
-            "format": "MARKDOWN",
+            "format": 1,
             "style": {}
           }
         },
@@ -569,7 +569,7 @@ X-Xss-Protection: 0
         "widget": {
           "text": {
             "content": "CollapsibleGroup content",
-            "format": "MARKDOWN",
+            "format": 1,
             "style": {}
           }
         },