• Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • core
    • None

      FutureImpl does not cancel its attribute start, which is AsyncFutureImpl instance for identification that task is running.

      methos setAsCancelled() should cancel its attribute start too.

      It causes bug in method getStartCondition() in case cancellation of job in queue. Because start AssyncFuturaImpl is not set as cancelled and thread wich called getStartCondition() wait forever when job is cancelled in queue, because start is never cancelled.

      It can be seen in Build flow plugin.

          [JENKINS-25514] FutureImpl does not cancel it`s start future

          Code changed in jenkins
          User: Eugene Baranov
          Path:
          core/src/main/java/hudson/model/queue/FutureImpl.java
          test/src/test/java/hudson/model/QueueTest.java
          http://jenkins-ci.org/commit/jenkins/3edae667b3955a561296a23122f86b35ec928a98
          Log:
          JENKINS-25514 Fixed lock case in FutureImpl
          when some other process is waiting for build in queue.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Eugene Baranov Path: core/src/main/java/hudson/model/queue/FutureImpl.java test/src/test/java/hudson/model/QueueTest.java http://jenkins-ci.org/commit/jenkins/3edae667b3955a561296a23122f86b35ec928a98 Log: JENKINS-25514 Fixed lock case in FutureImpl when some other process is waiting for build in queue.

          Code changed in jenkins
          User: Oliver Gondža
          Path:
          core/src/main/java/hudson/model/queue/FutureImpl.java
          test/src/test/java/hudson/model/QueueTest.java
          http://jenkins-ci.org/commit/jenkins/5bfaca449ce3b6edbf39a3f623bc8ce62411b37e
          Log:
          Merge pull request #1472 from jbaranov/JENKINS-25514

          JENKINS-25514 Fixed lock case in FutureImpl

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oliver Gondža Path: core/src/main/java/hudson/model/queue/FutureImpl.java test/src/test/java/hudson/model/QueueTest.java http://jenkins-ci.org/commit/jenkins/5bfaca449ce3b6edbf39a3f623bc8ce62411b37e Log: Merge pull request #1472 from jbaranov/ JENKINS-25514 JENKINS-25514 Fixed lock case in FutureImpl

          dogfood added a comment -

          Integrated in jenkins_main_trunk #3915

          Result = SUCCESS

          dogfood added a comment - Integrated in jenkins_main_trunk #3915 Result = SUCCESS

          Daniel Beck added a comment -

          Is this LTS-worthy?

          Daniel Beck added a comment - Is this LTS-worthy?

            lvotypkova Lucie Votypkova
            lvotypkova Lucie Votypkova
            Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: