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

FutureImpl does not cancel it`s start future

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved (View Workflow)
    • Minor
    • Resolution: Fixed
    • core
    • None

    Description

      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.

      Attachments

        Activity

          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_issue_link 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_issue_link 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 dogfood added a comment -

          Integrated in jenkins_main_trunk #3915

          Result = SUCCESS

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

          Is this LTS-worthy?

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

          People

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

            Dates

              Created:
              Updated:
              Resolved: