[JIRA] (JENKINS-35712) Failed step doesn't show as FlowNode

2016-08-29 Thread mne...@cloudbees.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Michael Neale edited a comment on  JENKINS-35712  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: Failed step doesn't show as FlowNode   
 

  
 
 
 
 

 
 [~vivek] for this to work, however, the whole job would need to return no steps, and then it will default to showing just the full log... right [~jamesdumay] ? Or do we rely on the user to pop open the log when they see that things failed but not where? what should this look like? To be clear, this is a sytactic error - will show that bit error message I think right now if you try it (which is ok). With pipeline-config the error message will be clearer, but there will be no flow nodes in either case. This is ok IF this defaults to showing just the log (ie error message).  Where this is bad is if the build starts, but then fails on that parallel branch ... then it isn't clear what to show. What is expected behavior, if we can't get a flownode?   
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-35712) Failed step doesn't show as FlowNode

2016-08-29 Thread mne...@cloudbees.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Michael Neale edited a comment on  JENKINS-35712  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: Failed step doesn't show as FlowNode   
 

  
 
 
 
 

 
 [~vivek] for this to work, however, the whole job would need to return no steps, and then it will default to showing just the full log... right [~jamesdumay] ? Or do we rely on the user to pop open the log when they see that things failed but not where? what should this look like?To be clear, this is a sytactic error - will show that bit error message I think right now if you try it (which is ok). With pipeline-config the error message will be clearer, but there will be no flow nodes in either case. This is ok IF this defaults to showing just the log (ie error message).   
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-35712) Failed step doesn't show as FlowNode

2016-08-29 Thread mne...@cloudbees.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Michael Neale commented on  JENKINS-35712  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: Failed step doesn't show as FlowNode   
 

  
 
 
 
 

 
 Vivek Pandey for this to work, however, the whole job would need to return no steps, and then it will default to showing just the full log... right James Dumay ? Or do we rely on the user to pop open the log when they see that things failed but not where? what should this look like?   
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-35712) Failed step doesn't show as FlowNode

2016-08-29 Thread vivek.pan...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Vivek Pandey commented on  JENKINS-35712  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: Failed step doesn't show as FlowNode   
 

  
 
 
 
 

 
 Jesse Glick I see. Referring to Kohsuke Kawaguchi's comment, something that can be implemented? Not sure what kind of changes its going to result in to though.  Michael Neale Since we show logs only for steps, we can use full log to show such errors as workaround. We flag failing node (branch or stage) status in the json DAG using parallel's endNode. If there is UI to show node level log, such error can be returned as log?  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-35712) Failed step doesn't show as FlowNode

2016-08-29 Thread jgl...@cloudbees.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Jesse Glick resolved as Not A Defect  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
 A FlowNode is only created when there is a successful attempt to at least start a Step. In this case you should just see a failure attached to the end block of the parallel branch.  
 

  
 
 
 
 

 
 Jenkins /  JENKINS-35712  
 
 
  Failed step doesn't show as FlowNode   
 

  
 
 
 
 

 
Change By: 
 Jesse Glick  
 
 
Status: 
 Open Resolved  
 
 
Resolution: 
 Not A Defect  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails 

[JIRA] (JENKINS-35712) Failed step doesn't show as FlowNode

2016-08-08 Thread mne...@cloudbees.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Michael Neale commented on  JENKINS-35712  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: Failed step doesn't show as FlowNode   
 

  
 
 
 
 

 
 (BTW the only blue ocean workaround for this is to fall back to showing full log, none others are planned or known about).   
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-35712) Failed step doesn't show as FlowNode

2016-08-08 Thread jdu...@cloudbees.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 James Dumay commented on  JENKINS-35712  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: Failed step doesn't show as FlowNode   
 

  
 
 
 
 

 
 Patrick Wolf / Jesse Glick any progress on this one?  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-35712) Failed step doesn't show as FlowNode

2016-06-16 Thread mne...@cloudbees.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Michael Neale commented on  JENKINS-35712  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: Failed step doesn't show as FlowNode   
 

  
 
 
 
 

 
 One option would be to "fail safe" and show a log if there is a failed state on job (or anything) but no corresponding step flownode. This would at least tell the user what was wrong, which is often enough.  Not ideal though...   
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-35712) Failed step doesn't show as FlowNode

2016-06-14 Thread k...@kohsuke.org (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Kohsuke Kawaguchi commented on  JENKINS-35712  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: Failed step doesn't show as FlowNode   
 

  
 
 
 
 

 
 Blue Ocean team seems to have some other workarounds in mind that I'm not very familiar with.  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-35712) Failed step doesn't show as FlowNode

2016-06-14 Thread k...@kohsuke.org (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Kohsuke Kawaguchi edited a comment on  JENKINS-35712  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: Failed step doesn't show as FlowNode   
 

  
 
 
 
 

 
 Perhaps naively, I initially thought it makes sense to create a {{FlowNode}} when an exception is thrown then I realized that this is more complicated, as there isn't an easy way to recognize the origin of the exception.This example could have been {{sh ssdsds()}} and when {{ssdsds()}} invocation throws an exception, we can't tell if that's because there's no such function, or whether this function actually exists and its execution has thrown an exception.  If the point in which an exception is thrown is responsible for creating a record, then in the former case the point that invokes {{ssdsds()}} would add the record, but in the latter case the code inside {{ssdsds()}} should have done that.Maybe it's possible to have the code that catches the exception (usually {{CpsBodyExecution}}) to create a record?  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-35712) Failed step doesn't show as FlowNode

2016-06-14 Thread k...@kohsuke.org (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Kohsuke Kawaguchi edited a comment on  JENKINS-35712  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: Failed step doesn't show as FlowNode   
 

  
 
 
 
 

 
 Naively Perhaps naively , I  initially  thought it makes sense to create a {{FlowNode}} when an exception is thrown then I realized that this is more complicated, as there isn't an easy way to recognize the origin of the exception.This example could have been {{sh ssdsds()}} and when {{ssdsds()}} invocation throws an exception, we can't tell if that's because there's no such function, or whether this function actually exists and its execution has thrown an exception.Maybe it's possible to have the code that catches the exception (usually  Step  {{CpsBodyExecution}} ) to create a record?  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-35712) Failed step doesn't show as FlowNode

2016-06-14 Thread k...@kohsuke.org (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Kohsuke Kawaguchi edited a comment on  JENKINS-35712  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: Failed step doesn't show as FlowNode   
 

  
 
 
 
 

 
 Naively, I thought it makes sense to create a  `  {{ FlowNode ` }}  when an exception is thrown then I realized that this is more complicated, as there isn't an easy way to recognize the origin of the exception.This example could have been {{sh ssdsds()}} and when {{ssdsds()}} invocation throws an exception, we can't tell if that's because there's no such function, or whether this function actually exists and its execution has thrown an exception.Maybe it's possible to have the code that catches the exception (usually Step) to create a record?  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-35712) Failed step doesn't show as FlowNode

2016-06-14 Thread k...@kohsuke.org (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Kohsuke Kawaguchi commented on  JENKINS-35712  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: Failed step doesn't show as FlowNode   
 

  
 
 
 
 

 
 Naively, I thought it makes sense to create a `FlowNode` when an exception is thrown then I realized that this is more complicated, as there isn't an easy way to recognize the origin of the exception. This example could have been sh ssdsds() and when ssdsds() invocation throws an exception, we can't tell if that's because there's no such function, or whether this function actually exists and its execution has thrown an exception. Maybe it's possible to have the code that catches the exception (usually Step) to create a record?  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-35712) Failed step doesn't show as FlowNode

2016-06-14 Thread vivek.pan...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Vivek Pandey created an issue  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
 Jenkins /  JENKINS-35712  
 
 
  Failed step doesn't show as FlowNode   
 

  
 
 
 
 

 
Issue Type: 
  Bug  
 
 
Assignee: 
 Jesse Glick  
 
 
Components: 
 workflow-plugin  
 
 
Created: 
 2016/Jun/15 1:32 AM  
 
 
Environment: 
 workflow-api 2.0, Jenkins 2.2  
 
 
Priority: 
  Minor  
 
 
Reporter: 
 Vivek Pandey  
 

  
 
 
 
 

 
 If there is step in pipeline script with syntax error, there is error reported during the build however there is no corresponding FlowNode reported.  Here is the sample script  

 

node {
  stage 'build'
  sh 'echo s1'
  stage 'test'
  echo 'Hello World 2'
}
parallel firstBranch: {
  echo 'Hello first'
}, secondBranch: {
 echo 'Hello second'
 sh ssdsds
},
failFast: true
 

 Under the parallel 'secondBranch' the step sh ssdsds is reported as failure during pipeline execution however there is no FlowNode reported corresponding to this step. Below is the error reported in the log for this failed step: 

 

groovy.lang.MissingPropertyException: No such property: ssdsds for class: groovy.lang.Binding
  at groovy.lang.Binding.getVariable(Binding.java:63)
  at org.jenkinsci.plugins.scriptsecurity.sandbox.groovy.SandboxInterceptor.onGetProperty(SandboxInterceptor.java:210)
  at org.kohsuke.groovy.sandbox.impl.Checker$4.call(Checker.java:241)
  at org.kohsuke.groovy.sandbox.impl.Checker.checkedGetProperty(Checker.java:238)