Преглед на файлове

feat(alerting): sql update fixes

Torkel Ödegaard преди 9 години
родител
ревизия
66c2594262
променени са 3 файла, в които са добавени 6 реда и са изтрити 3 реда
  1. 3 2
      pkg/services/alerting/commands.go
  2. 1 1
      pkg/services/alerting/executor.go
  3. 2 0
      pkg/services/sqlstore/alert.go

+ 3 - 2
pkg/services/alerting/commands.go

@@ -19,8 +19,9 @@ func init() {
 
 
 func updateDashboardAlerts(cmd *UpdateDashboardAlertsCommand) error {
 func updateDashboardAlerts(cmd *UpdateDashboardAlertsCommand) error {
 	saveAlerts := m.SaveAlertsCommand{
 	saveAlerts := m.SaveAlertsCommand{
-		OrgId:  cmd.OrgId,
-		UserId: cmd.UserId,
+		OrgId:       cmd.OrgId,
+		UserId:      cmd.UserId,
+		DashboardId: cmd.Dashboard.Id,
 	}
 	}
 
 
 	extractor := NewDashAlertExtractor(cmd.Dashboard, cmd.OrgId)
 	extractor := NewDashAlertExtractor(cmd.Dashboard, cmd.OrgId)

+ 1 - 1
pkg/services/alerting/executor.go

@@ -69,7 +69,7 @@ func (e *ExecutorImpl) executeQuery(job *AlertJob) (tsdb.TimeSeriesSlice, error)
 }
 }
 
 
 func (e *ExecutorImpl) GetRequestForAlertRule(rule *AlertRule, datasource *m.DataSource) *tsdb.Request {
 func (e *ExecutorImpl) GetRequestForAlertRule(rule *AlertRule, datasource *m.DataSource) *tsdb.Request {
-	log.Debug2("GetRequest", "query", rule.Query.Query, "from", rule.Query.From, "datasourceId", datasource.Id)
+	e.log.Debug("GetRequest", "query", rule.Query.Query, "from", rule.Query.From, "datasourceId", datasource.Id)
 	req := &tsdb.Request{
 	req := &tsdb.Request{
 		TimeRange: tsdb.TimeRange{
 		TimeRange: tsdb.TimeRange{
 			From: "-" + rule.Query.From,
 			From: "-" + rule.Query.From,

+ 2 - 0
pkg/services/sqlstore/alert.go

@@ -184,6 +184,7 @@ func upsertAlerts(alerts []*m.Alert, posted []*m.Alert, sess *xorm.Session) erro
 				update = true
 				update = true
 				alert.Id = k.Id
 				alert.Id = k.Id
 				alertToUpdate = k
 				alertToUpdate = k
+				break
 			}
 			}
 		}
 		}
 
 
@@ -224,6 +225,7 @@ func deleteMissingAlerts(alerts []*m.Alert, posted []*m.Alert, sess *xorm.Sessio
 		for _, k := range posted {
 		for _, k := range posted {
 			if missingAlert.PanelId == k.PanelId {
 			if missingAlert.PanelId == k.PanelId {
 				missing = false
 				missing = false
+				break
 			}
 			}
 		}
 		}