Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-47517

2.85: StackOverflowError between Queue$Task.getCauseOfBlockage and workflow.job.AfterRestartTask.isBuildBlocked

    XMLWordPrintable

Details

    Description

      The fix of JENKINS-20046 seems to have introduced a critical error.

      mvn -f .../jenkinsci/pipeline-stage-step-plugin -Dtest=StageStepTest\#basics -Djenkins.version=2.84 test
      

      is OK. With 2.85:

      ... jenkins.util.AtmostOneTaskExecutor$1 call
      WARNING: null
      java.lang.StackOverflowError
      	at hudson.model.Queue$Task.getCauseOfBlockage(Queue.java:1844)
      	at org.jenkinsci.plugins.workflow.job.AfterRestartTask.isBuildBlocked(AfterRestartTask.java:60)
      	at hudson.model.Queue$Task.getCauseOfBlockage(Queue.java:1844)
      	at org.jenkinsci.plugins.workflow.job.AfterRestartTask.isBuildBlocked(AfterRestartTask.java:60)
      	at hudson.model.Queue$Task.getCauseOfBlockage(Queue.java:1844)
      	at ...
      

      Attachments

        Issue Links

          Activity

            jglick Jesse Glick added a comment -

            Note: I have not yet checked for the user impact of this bug, e.g. whether it causes ATH failures in WorkflowPluginTest.

            jglick Jesse Glick added a comment - Note: I have not yet checked for the user impact of this bug, e.g. whether it causes ATH failures in WorkflowPluginTest .
            jglick Jesse Glick added a comment -

            WorkflowPluginTest against 2.85 does print a StackOverflowError at some point, though the test passes anyway.

            jglick Jesse Glick added a comment - WorkflowPluginTest against 2.85 does print a StackOverflowError at some point, though the test passes anyway.
            jglick Jesse Glick added a comment -

            Does not cause test failure in workflow-job.

            jglick Jesse Glick added a comment - Does not cause test failure in workflow-job .
            jglick Jesse Glick added a comment -

            Merged toward 2.86.

            jglick Jesse Glick added a comment - Merged toward 2.86.

            Not applicable for 2.73 LTS line. As I understand this affects 2.85 only so keeping the candidate flag here to reevaluate in case we pick 2.85 as the next LTS.

            olivergondza Oliver Gondža added a comment - Not applicable for 2.73 LTS line. As I understand this affects 2.85 only so keeping the candidate flag here to reevaluate in case we pick 2.85 as the next LTS.
            jglick Jesse Glick added a comment -

            Yes this applies only to 2.85.

            jglick Jesse Glick added a comment - Yes this applies only to 2.85.
            jglick Jesse Glick added a comment -

            Finally happened upon a case where PCT would have caught this: WorkflowTest.demo in workflow-cps passes against 2.84 but fails against 2.85. (And passes against 2.86 with the addition of -Djenkins-test-harness.version=2.31 needed due to JENKINS-47393.)

            jglick Jesse Glick added a comment - Finally happened upon a case where PCT would have caught this: WorkflowTest.demo in workflow-cps passes against 2.84 but fails against 2.85. (And passes against 2.86 with the addition of -Djenkins-test-harness.version=2.31  needed due to  JENKINS-47393 .)

            People

              jglick Jesse Glick
              jglick Jesse Glick
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: