Merge pull request #22019 from vespa-engine/ean/reduce-mails-sent

Limit email notification to prodution only

1 files changed, 8 insertions, 0 deletions

diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/notify/Notifier.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/notify/Notifier.java
index e186541c85c..594908a3bc1 100644
--- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/notify/Notifier.java
+++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/notify/Notifier.java
@@ -38,6 +38,9 @@ public class Notifier {
         if (notifications.isEmpty()) {
             return;
         }
+        if (skipSource(source)) {
+            return;
+        }
         var tenant = curatorDb.readTenant(source.tenant());
         tenant.stream().forEach(t -> {
             if (t instanceof CloudTenant) {
@@ -51,6 +54,11 @@ public class Notifier {
         });
     }
 
+    private boolean skipSource(NotificationSource source) {
+        // Limit sources to production systems only. Dev and test systems cause too much noise at the moment.
+        return source.jobType().map(t -> !t.isProduction()).orElse(false);
+    }
+
     public void dispatch(Notification notification) {
         dispatch(List.of(notification), notification.source());
     }