sets summary states better

jghibiki · Apr 10, 2019 · e919f9c · e919f9c
1 parent a420770
commit e919f9c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/schema.json b/schema.json
@@ -33,6 +33,8 @@
     ],
 
     "failure_state": "failed",
+    "success_state": "ok",
+    "pending_state": "pending",
 
     "notifications": [
         {

diff --git a/server/sydney.py b/server/sydney.py
@@ -211,8 +211,9 @@ def notify_state_change(environment, pipeline, step, state):
                     # set parent to new state
                     if pipelines["failure_state"] != parent["state"]:
                         notify_state_change(env["name"], parent_pipeline, parent["name"], pipelines["failure_state"])
-
-
+                elif pipelines["pending_state"] in child_states:
+                    if pipelines["pending_state"] != parent["state"]:
+                        notify_state_change(env["name"], parent_pipeline, parent["name"], pipelines["pending_state"])
                 else:
                     #pick most common state
                     new_state = max(child_states, key=child_states.count)