provider/pd: don't bail out when snoozing an incident that's already acknowledged

adamdecaf · adamdecaf · commit 06fa663108ee · 2024-10-24T10:49:37.000-05:00
diff --git a/internal/provider/pd/incidents.go b/internal/provider/pd/incidents.go
@@ -91,7 +91,10 @@ func (c *client) snoozeIncident(ctx context.Context, logger log.Logger, inc *pag
 	}
 	_, err := c.underlying.ManageIncidentsWithContext(ctx, c.pdConfig.From, update)
 	if err != nil {
-		return fmt.Errorf("incident acknowledged: %w", err)
+		// Ignore this error if the incident is already acknowledged
+		if !strings.Contains(strings.ToLower(err.Error()), "incident already acknowledged") {
+			return fmt.Errorf("incident acknowledged: %w", err)
+		}
 	}
 
 	// Snooze the incident

Original file line number	Diff line number	Diff line change
`@@ -91,7 +91,10 @@ func (c client) snoozeIncident(ctx context.Context, logger log.Logger, inc pag`
`91`	`91`	`}`
`92`	`92`	`_, err := c.underlying.ManageIncidentsWithContext(ctx, c.pdConfig.From, update)`
`93`	`93`	`if err != nil {`
`94`		`- return fmt.Errorf("incident acknowledged: %w", err)`
	`94`	`+ // Ignore this error if the incident is already acknowledged`
	`95`	`+ if !strings.Contains(strings.ToLower(err.Error()), "incident already acknowledged") {`
	`96`	`+ return fmt.Errorf("incident acknowledged: %w", err)`
	`97`	`+ }`
`95`	`98`	`}`
`96`	`99`
`97`	`100`	`// Snooze the incident`