Просмотр исходного кода

tech(pagerduty): declare url as constant

bergquist 9 лет назад
Родитель
Сommit
7bc80d0374
1 измененных файлов с 11 добавлено и 11 удалено
  1. 11 11
      pkg/services/alerting/notifiers/pagerduty.go

+ 11 - 11
pkg/services/alerting/notifiers/pagerduty.go

@@ -13,6 +13,10 @@ func init() {
 	alerting.RegisterNotifier("pagerduty", NewPagerdutyNotifier)
 	alerting.RegisterNotifier("pagerduty", NewPagerdutyNotifier)
 }
 }
 
 
+var (
+	pagerdutyEventApiUrl string = "https://events.pagerduty.com/generic/2010-04-15/create_event.json"
+)
+
 func NewPagerdutyNotifier(model *m.AlertNotification) (alerting.Notifier, error) {
 func NewPagerdutyNotifier(model *m.AlertNotification) (alerting.Notifier, error) {
 	key := model.Settings.Get("integrationKey").MustString()
 	key := model.Settings.Get("integrationKey").MustString()
 	if key == "" {
 	if key == "" {
@@ -20,16 +24,16 @@ func NewPagerdutyNotifier(model *m.AlertNotification) (alerting.Notifier, error)
 	}
 	}
 
 
 	return &PagerdutyNotifier{
 	return &PagerdutyNotifier{
-		NotifierBase:     NewNotifierBase(model.Id, model.IsDefault, model.Name, model.Type, model.Settings),
-		Key:              key,
-		log:              log.New("alerting.notifier.pagerduty"),
+		NotifierBase: NewNotifierBase(model.Id, model.IsDefault, model.Name, model.Type, model.Settings),
+		Key:          key,
+		log:          log.New("alerting.notifier.pagerduty"),
 	}, nil
 	}, nil
 }
 }
 
 
 type PagerdutyNotifier struct {
 type PagerdutyNotifier struct {
 	NotifierBase
 	NotifierBase
-	Key              string
-	log              log.Logger
+	Key string
+	log log.Logger
 }
 }
 
 
 func (this *PagerdutyNotifier) Notify(evalContext *alerting.EvalContext) error {
 func (this *PagerdutyNotifier) Notify(evalContext *alerting.EvalContext) error {
@@ -37,13 +41,9 @@ func (this *PagerdutyNotifier) Notify(evalContext *alerting.EvalContext) error {
 	metrics.M_Alerting_Notification_Sent_PagerDuty.Inc(1)
 	metrics.M_Alerting_Notification_Sent_PagerDuty.Inc(1)
 
 
 	if evalContext.Rule.State == m.AlertStateAlerting {
 	if evalContext.Rule.State == m.AlertStateAlerting {
-
-		// Pagerduty Events API URL
-		pgEventsUrl := "https://events.pagerduty.com/generic/2010-04-15/create_event.json"
-
 		bodyJSON := simplejson.New()
 		bodyJSON := simplejson.New()
 		bodyJSON.Set("service_key", this.Key)
 		bodyJSON.Set("service_key", this.Key)
-		bodyJSON.Set("description", evalContext.Rule.Name + " - " + evalContext.Rule.Message)
+		bodyJSON.Set("description", evalContext.Rule.Name+" - "+evalContext.Rule.Message)
 		bodyJSON.Set("client", "Grafana")
 		bodyJSON.Set("client", "Grafana")
 		bodyJSON.Set("event_type", "trigger")
 		bodyJSON.Set("event_type", "trigger")
 
 
@@ -66,7 +66,7 @@ func (this *PagerdutyNotifier) Notify(evalContext *alerting.EvalContext) error {
 		body, _ := bodyJSON.MarshalJSON()
 		body, _ := bodyJSON.MarshalJSON()
 
 
 		cmd := &m.SendWebhook{
 		cmd := &m.SendWebhook{
-			Url:        pgEventsUrl,
+			Url:        pagerdutyEventApiUrl,
 			Body:       string(body),
 			Body:       string(body),
 			HttpMethod: "POST",
 			HttpMethod: "POST",
 		}
 		}