ChatOps: Avoid failing the whole action chain when result output is d…

…isabled

contrib/chatops/actions/format_result.py

@@ -36,9 +36,8 @@ def run(self, execution):
 
             result = getattr(alias, 'result', None)
             if result:
-                enabled = result.get('enabled', True)
-
-                if enabled and 'format' in alias.result:
+                if not result.get('enabled', True):
+                    raise Exception("Output of this template is disabled.")
+                if 'format' in alias.result:
                     template = alias.result['format']
-
         return self.jinja.from_string(template).render(context)

contrib/chatops/actions/skip_message.py

@@ -0,0 +1,13 @@
+from st2actions.runners.pythonrunner import Action
+
+__all__ = [
+    'SkipMessageAction'
+]
+
+
+class SkipMessageAction(Action):
+    def run(self):
+        # That's a placeholder for format_result failure
+        # to avoid failing action chain on aliases with 
+        # disabled output.
+        return 

contrib/chatops/actions/skip_message.yaml

@@ -0,0 +1,7 @@
+---
+name: "skip_message"
+pack: chatops
+runner_type: "run-python"
+description: "Skip sending an execution result for chatops"
+enabled: true
+entry_point: "skip_message.py"

contrib/chatops/actions/workflows/post_result.yaml

@@ -17,6 +17,7 @@ chain:
     publish:
       message: "{{format_result.result}}"
     on-success: "post_message"
+    on-failure: "skip_message"
   -
     name: "post_message"
     ref: "chatops.post_message"
@@ -25,3 +26,6 @@ chain:
       channel: "{{channel}}"
       whisper: "{{whisper}}"
       message: "{{message}}"
+  -
+    name: "skip_message"
+    ref: "chatops.skip_message"