[JENKINS-43276] CoreWrapperStep should run SimpleBuildWrapper.setUp asynchronously

Type: Bug
Resolution: Fixed
Priority: Major
Component/s: workflow-basic-steps-plugin
Labels:

Similar Issues:
Powered by SuggestiMate

Show
Released As:
workflow-basic-steps 2.14

CoreStep runs SimpleBuildStep.perform in a background thread, via SynchronousNonBlockingStepExecution, so while pending ~~JENKINS-30383~~ there is a high chance that a Jenkins restart will cause a build failure, at least the CPS VM thread is not blocked. However CoreWrapperStep runs SimpleBuildWrapper.setUp directly from Execution.start, since there is no block-scoped equivalent of SynchronousNonBlockingStepExecution, which means that a slow wrapper could block the VM thread indefinitely.

blocks

JENKINS-26056 Workflow build step for Ant

Resolved

depends on

JENKINS-30383 SynchronousNonBlockingStepExecution should allow restart of idempotent steps

Resolved

relates to

JENKINS-49337 GeneralNonBlockingStepExecution (was: InterruptedException from step tail call)

Closed

JENKINS-26055 DurableTaskStep

Resolved

JENKINS-32986 hard killing a pipeline leaves the JVM CPS thread running.

Open

JENKINS-35420 Realtime junit report to support Pipeline (was: throws ClassCastException when pipeline run)

Resolved

JENKINS-42561 Users should be able to custom configure the timeout on pipeline build wrappers/steps

Resolved

links to

PR 78

(2 relates to, 1 links to)

Jesse Glick added a comment - 2017-03-31 16:04

~~JENKINS-26055~~ (or ~~JENKINS-37011~~) would make it easier to write a regular Step which as part of its functionality launches an external process with an unknown run duration.

Jesse Glick added a comment - 2017-03-31 16:04 JENKINS-26055 (or JENKINS-37011 ) would make it easier to write a regular Step which as part of its functionality launches an external process with an unknown run duration.

Jesse Glick added a comment - 2017-08-17 20:08

See ~~JENKINS-26056~~ for an example svanoort.

Jesse Glick added a comment - 2017-08-17 20:08 See JENKINS-26056 for an example svanoort .

Jesse Glick added a comment - 2018-12-06 20:47

Defining the necessary API in ~~JENKINS-49337~~.

Jesse Glick added a comment - 2018-12-06 20:47 Defining the necessary API in JENKINS-49337 .

Devin Nusbaum added a comment - 2019-01-14 18:50

The fix for this issue was just released in version 2.14 of Pipeline Basic Steps Plugin.

Devin Nusbaum added a comment - 2019-01-14 18:50 The fix for this issue was just released in version 2.14 of Pipeline Basic Steps Plugin.

Assignee:: Jesse Glick

Reporter:: Jesse Glick

Votes:: 0 Vote for this issue

Watchers:: 2 Start watching this issue

Created:: 2017-03-31 15:59

Updated:: 2019-01-14 18:50

Resolved:: 2019-01-14 15:57

Details

Description

Attachments

Issue Links

Activity

Collapse comment: Jesse Glick added a comment - 2017-03-31 16:04

Expand comment: Jesse Glick added a comment - 2017-03-31 16:04

Collapse comment: Jesse Glick added a comment - 2017-08-17 20:08

Expand comment: Jesse Glick added a comment - 2017-08-17 20:08

Collapse comment: Jesse Glick added a comment - 2018-12-06 20:47

Expand comment: Jesse Glick added a comment - 2018-12-06 20:47

Collapse comment: Devin Nusbaum added a comment - 2019-01-14 18:50

Expand comment: Devin Nusbaum added a comment - 2019-01-14 18:50

People

Dates