Skip to content

Commit 40a1947

Browse files
authored
fix: trigger_after_pct not handled correctly in alert downtime resources (#92)
1 parent 9aae0dc commit 40a1947

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

sysdig/resource_sysdig_monitor_alert_downtime.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import (
88

99
"github.com/hashicorp/terraform-plugin-sdk/v2/diag"
1010
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
11+
"github.com/spf13/cast"
1112

1213
"github.com/draios/terraform-provider-sysdig/sysdig/internal/client/monitor"
1314
)
@@ -139,7 +140,7 @@ func downtimeAlertFromResourceData(d *schema.ResourceData) (alert *monitor.Alert
139140
}
140141

141142
alert.SegmentCondition = &monitor.SegmentCondition{Type: "ANY"}
142-
alert.Condition = fmt.Sprintf("avg(timeAvg(uptime)) <= %.2f", float64(1.0-(d.Get("trigger_after_pct").(int)/100)))
143+
alert.Condition = fmt.Sprintf("avg(timeAvg(uptime)) <= %.2f", 1.0-(cast.ToFloat64(d.Get("trigger_after_pct"))/100.0))
143144

144145
entitiesRaw := d.Get("entities_to_monitor").([]interface{})
145146
for _, entityRaw := range entitiesRaw {
@@ -156,7 +157,7 @@ func downtimeAlertToResourceData(alert *monitor.Alert, data *schema.ResourceData
156157
}
157158

158159
var trigger_after_pct float64
159-
fmt.Sscanf(alert.Condition, "avg(timeAvg(uptime)) <= %.2f", &trigger_after_pct)
160+
fmt.Sscanf(alert.Condition, "avg(timeAvg(uptime)) <= %f", &trigger_after_pct)
160161
trigger_after_pct = (1 - trigger_after_pct) * 100
161162

162163
data.Set("trigger_after_pct", int(trigger_after_pct))

sysdig/resource_sysdig_monitor_alert_downtime_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ resource "sysdig_monitor_alert_downtime" "sample" {
5050
scope = "kubernetes.cluster.name in (\"pulsar\")"
5151
5252
trigger_after_minutes = 10
53-
trigger_after_pct = 100
53+
trigger_after_pct = 99
5454
5555
enabled = false
5656

0 commit comments

Comments
 (0)