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

Setting JDK installations not thread-safe

    XMLWordPrintable

Details

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

    Description

      Due to improper synchronization, it's possible that the list of JDKs is empty while adding or removing one. See https://github.com/jenkinsci/jenkins/blob/jenkins-1.612/core/src/main/java/hudson/model/JDK.java#L170.

      Attachments

        Issue Links

          Activity

            daspilker Daniel Spilker added a comment - See https://github.com/jenkinsci/jenkins/pull/1692 for a fix.

            Code changed in jenkins
            User: Daniel Spilker
            Path:
            core/src/main/java/hudson/model/JDK.java
            core/src/main/java/jenkins/model/Jenkins.java
            http://jenkins-ci.org/commit/jenkins/44eeb10461b0f49fe6c0125a9727f57480d10f10
            Log:
            JENKINS-28292 fixed synchronization issue

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Daniel Spilker Path: core/src/main/java/hudson/model/JDK.java core/src/main/java/jenkins/model/Jenkins.java http://jenkins-ci.org/commit/jenkins/44eeb10461b0f49fe6c0125a9727f57480d10f10 Log: JENKINS-28292 fixed synchronization issue

            Code changed in jenkins
            User: Oleg Nenashev
            Path:
            core/src/main/java/hudson/model/JDK.java
            core/src/main/java/jenkins/model/Jenkins.java
            http://jenkins-ci.org/commit/jenkins/67f4dd516633a5ef0f6d0085c0f1b90c5ebf3c8e
            Log:
            Merge pull request #1692 from daspilker/JENKINS-28292

            JENKINS-28292 fixed synchronization issue when setting JDK installations

            Compare: https://github.com/jenkinsci/jenkins/compare/3f0e2926c01b...67f4dd516633

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: core/src/main/java/hudson/model/JDK.java core/src/main/java/jenkins/model/Jenkins.java http://jenkins-ci.org/commit/jenkins/67f4dd516633a5ef0f6d0085c0f1b90c5ebf3c8e Log: Merge pull request #1692 from daspilker/ JENKINS-28292 JENKINS-28292 fixed synchronization issue when setting JDK installations Compare: https://github.com/jenkinsci/jenkins/compare/3f0e2926c01b...67f4dd516633
            dogfood dogfood added a comment -

            Integrated in jenkins_main_trunk #4177

            Result = SUCCESS

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

            Has been fixed in 1.618.

            daspilker Daniel Spilker added a comment - Has been fixed in 1.618.

            Removing lts-candidate label as it is a part of 1.625 baseline.

            olivergondza Oliver Gondža added a comment - Removing lts-candidate label as it is a part of 1.625 baseline.
            lkarnasiewicz Lukasz Karnasiewicz added a comment - Caused JENKINS-31932

            People

              daspilker Daniel Spilker
              daspilker Daniel Spilker
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: