Merge pull request #285 from wttech/notifications-skipped-for-health-checker

Notifications skipped for health checker

Author: krystian-panek-vmltech

core/src/main/java/dev/vml/es/acm/core/code/ExecutionContext.java

@@ -35,6 +35,8 @@ public static String varPath(String executionId) {
 
     private boolean history = true;
 
+    private boolean notifications = true;
+
     private boolean debug = false;
 
     private boolean skipped = false;
@@ -138,6 +140,14 @@ public void setHistory(boolean history) {
         this.history = history;
     }
 
+    public boolean isNotifications() {
+        return notifications;
+    }
+
+    public void setNotifications(boolean notifications) {
+        this.notifications = notifications;
+    }
+
     public boolean isDebug() {
         return debug;
     }

core/src/main/java/dev/vml/es/acm/core/code/Executor.java

@@ -298,7 +298,8 @@ private void handleHistory(ContextualExecution execution) {
 
     private void handleNotifications(ContextualExecution execution) {
         String executableId = execution.getExecutable().getId();
-        if (!config.notificationEnabled()
+        if (!execution.getContext().isNotifications()
+                || !config.notificationEnabled()
                 || !notifier.isConfigured(config.notificationNotifierId())
                 || Arrays.stream(config.notificationExecutableIds())
                         .noneMatch(regex -> Pattern.matches(regex, executableId))) {

core/src/main/java/dev/vml/es/acm/core/instance/HealthChecker.java

@@ -259,6 +259,7 @@ private void checkCodeExecutor(List<HealthIssue> issues, ResourceResolver resour
                 resourceResolver,
                 new CodeOutputMemory())) {
             context.setHistory(false);
+            context.setNotifications(false);
 
             Execution execution = executor.execute(context);
             if (execution.getStatus() != ExecutionStatus.SUCCEEDED) {