fix: Solve crash when updating Dashboard v3 (#89)

* fix: Solve crash when updating Dashboard v3

* style: Format files

* fix: Increase timeout while removing teams

* style: Format files

Author: tembleking

sysdig/resource_sysdig_monitor_dashboard.go

@@ -9,6 +9,7 @@ import (
 	"github.com/hashicorp/terraform-plugin-sdk/v2/diag"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/validation"
+	"github.com/spf13/cast"
 
 	"github.com/draios/terraform-provider-sysdig/sysdig/internal/client/monitor/model"
 )
@@ -227,7 +228,7 @@ func resourceSysdigDashboardDelete(ctx context.Context, data *schema.ResourceDat
 
 func dashboardFromResourceData(data *schema.ResourceData) (dashboard *model.Dashboard, err error) {
 	dashboard = model.NewDashboard(data.Get("name").(string), data.Get("description").(string)).AsPublic(data.Get("public").(bool))
-
+	dashboard.Version = cast.ToInt(data.Get("version"))
 	dashboard.PublicToken = data.Get("public_token").(string)
 
 	panels, err := panelsFromResourceData(data)

sysdig/resource_sysdig_monitor_dashboard_test.go

@@ -41,6 +41,9 @@ func TestAccDashboard(t *testing.T) {
 				ImportState:       true,
 				ImportStateVerify: true,
 			},
+			{
+				Config: multipleUpdatedPanelsDashboard(rText()),
+			},
 		},
 	})
 }
@@ -148,3 +151,59 @@ resource "sysdig_monitor_dashboard" "dashboard" {
 }
 `, name, name)
 }
+
+func multipleUpdatedPanelsDashboard(name string) string {
+	return fmt.Sprintf(`
+resource "sysdig_monitor_dashboard" "dashboard" {
+	name = "TERRAFORM TEST - METRIC %s"
+	description = "TERRAFORM TEST - METRIC %s"
+
+	panel {
+		pos_x = 0
+		pos_y = 0
+		width = 12 # Maximum size: 24
+		height = 6
+		type = "timechart"
+		name = "example panel"
+		description = "description"
+
+		query {
+			promql = "avg(avg_over_time(sysdig_host_cpu_used_percent[$__interval]))"
+			unit = "percent"
+		}
+		query {
+			promql = "avg(avg_over_time(sysdig_host_cpu_used_percent[$__interval]))"
+			unit = "number"
+		}
+	}
+
+	panel {
+		pos_x = 12
+		pos_y = 0
+		width = 12
+		height = 12
+		type = "number"
+		name = "example panel - 2"
+		description = "description of panel 2"
+
+		query {
+			promql = "avg(avg_over_time(sysdig_host_cpu_used_percent[$__interval]))"
+			unit = "time"
+		}
+	}
+
+	panel {
+		pos_x = 12
+		pos_y = 12
+		width = 12
+		height = 6
+		type = "text"
+		name = "example panel - 2"
+		content = "description of panel 2"
+		visible_title = true
+		autosize_text = true
+		transparent_background = true
+	}
+}
+`, name, name)
+}

sysdig/resource_sysdig_monitor_team.go

@@ -28,7 +28,7 @@ func resourceSysdigMonitorTeam() *schema.Resource {
 			Create: schema.DefaultTimeout(timeout),
 			Update: schema.DefaultTimeout(timeout),
 			Read:   schema.DefaultTimeout(timeout),
-			Delete: schema.DefaultTimeout(timeout),
+			Delete: schema.DefaultTimeout(5 * time.Minute), // Removing the team is for some reason slower.
 		},
 
 		Schema: map[string]*schema.Schema{

sysdig/resource_sysdig_secure_team.go

@@ -28,7 +28,7 @@ func resourceSysdigSecureTeam() *schema.Resource {
 			Create: schema.DefaultTimeout(timeout),
 			Update: schema.DefaultTimeout(timeout),
 			Read:   schema.DefaultTimeout(timeout),
-			Delete: schema.DefaultTimeout(timeout),
+			Delete: schema.DefaultTimeout(5 * time.Minute), // Removing the team is for some reason slower.
 		},
 
 		Schema: map[string]*schema.Schema{