Alerting: Propagate failures to delete dashboard alerts (#20507)
* Propagate failures to delete dashboard alerts
This commit is contained in:
@@ -334,13 +334,13 @@ func DeleteDashboard(cmd *models.DeleteDashboardCommand) error {
|
|||||||
|
|
||||||
for _, id := range dashIds {
|
for _, id := range dashIds {
|
||||||
if err := deleteAlertDefinition(id.Id, sess); err != nil {
|
if err := deleteAlertDefinition(id.Id, sess); err != nil {
|
||||||
return nil
|
return err
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if err := deleteAlertDefinition(dashboard.Id, sess); err != nil {
|
if err := deleteAlertDefinition(dashboard.Id, sess); err != nil {
|
||||||
return nil
|
return err
|
||||||
}
|
}
|
||||||
|
|
||||||
for _, sql := range deletes {
|
for _, sql := range deletes {
|
||||||
|
|||||||
Reference in New Issue
Block a user