Merge pull request #56 from KostyaSha/fixCrappyTriggers

Handle triggers start like normal job type. Robust onLoad. (cherry picked from commit 9f43067)
jenkinsci · Jun 19, 2017 · 6ddfce5 · 6ddfce5
1 parent 80c9a82
commit 6ddfce5
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/main/java/org/jenkinsci/plugins/workflow/job/properties/PipelineTriggersJobProperty.java b/src/main/java/org/jenkinsci/plugins/workflow/job/properties/PipelineTriggersJobProperty.java
@@ -100,7 +100,11 @@ public void stopTriggers() {
 
     public void startTriggers(boolean newInstance) {
         for (Trigger trigger : triggers) {
-            trigger.start(owner, newInstance);
+            try {
+                trigger.start(owner, newInstance);
+            } catch (Exception ex) {
+                LOGGER.log(Level.SEVERE, "Can't start trigger.", ex);
+            }
         }
     }
 
@@ -158,7 +162,7 @@ public PipelineTriggersJobProperty reconfigure(@Nonnull StaplerRequest req, @Che
 
             thisProp.startTriggers(true);
             return thisProp;
-        } catch (IOException e) {
+        } catch (Exception e) {
             LOGGER.log(Level.WARNING, "could not configure triggers", e);
         }