I took latest Jenkins code from master, applied your patch, and runt the plugin. Now when I click cancel I get.
Mar 23, 2018 8:13:31 PM hudson.model.Run execute SEVERE: Failed to save build record java.nio.channels.ClosedByInterruptException at java.nio.channels.spi.AbstractInterruptibleChannel.end(AbstractInterruptibleChannel.java:202) at sun.nio.ch.FileChannelImpl.write(FileChannelImpl.java:216) at hudson.util.FileChannelWriter.write(FileChannelWriter.java:72) at java.io.Writer.write(Writer.java:192) at hudson.util.AtomicFileWriter.write(AtomicFileWriter.java:162) at java.io.Writer.write(Writer.java:157) at hudson.XmlFile.write(XmlFile.java:189) at hudson.model.Run.save(Run.java:1923) at hudson.model.Run.execute(Run.java:1784) at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43) at hudson.model.ResourceController.execute(ResourceController.java:97) at hudson.model.Executor.run(Executor.java:429) On Friday, March 23, 2018 at 5:30:03 PM UTC+2, Jesse Glick wrote: > > On Fri, Mar 23, 2018 at 8:19 AM, Daniel Anechitoaie > <[email protected] <javascript:>> wrote: > > So it kind of seems that only MasterToSlaveFileCallable supports being > > interrupted while MasterToSlaveCallable it does not. > > That would be odd, since file callables are just a bit of sugar, > internally implemented using generic callables. The one thing that > strikes my notice is that the wrapper translates > `InterruptedException` to `TunneledInterruptedException`, though this > ought to be transparent as `FilePath.act` unwraps that and rethrows > the original `InterruptedException`. > -- You received this message because you are subscribed to the Google Groups "Jenkins Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/91c4946a-7510-4d5a-893e-97e75e0eb2f3%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
