alertmanager: Add Telegram Receiver type to AlertmanagerConfig CRD

fix unit testing
prometheus-operator · May 9, 2022 · 01bf10d · 01bf10d
1 parent 16422a9
commit 01bf10d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/pkg/alertmanager/amcfg.go b/pkg/alertmanager/amcfg.go
@@ -1537,7 +1537,7 @@ func (tc *telegramConfig) sanitize(amVersion semver.Version, logger log.Logger)
 		return errors.Errorf("mandatory field %q is empty", "chatID")
 	}
 
-	if tc.BotToken != "" {
+	if tc.BotToken == "" {
 		return fmt.Errorf("mandatory field %q is empty", "botToken")
 	}
 

diff --git a/pkg/alertmanager/operator.go b/pkg/alertmanager/operator.go
@@ -1506,9 +1506,9 @@ func checkTelegramConfigs(
 	if len(configs) == 0 {
 		return nil
 	}
-	
-	if amVersion.LT(semver.MustParse("0.24.0") {
-		return fmt.Errorf("'telegramConfigs' is available in Alertmanager >= 0.24.0 only - current %s`, amVersion)
+
+	if amVersion.LT(semver.MustParse("0.24.0")) {
+		return fmt.Errorf(`telegramConfigs' is available in Alertmanager >= 0.24.0 only - current %s`, amVersion)
 	}
 
 	for i, config := range configs {