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

Launch agents on Java 7 from Jenkins 2.60 fails without clear message

    XMLWordPrintable

    Details

    • Similar Issues:
    • Epic Link:

      Description

      Launching Jenkins 2.60 agents on Java 7 should fail with a controlled exception but it is not. The channel is just closed currently without further notice.

      This code needs fix.

      Flagged as LTS candidate as I suspect it will cause pain to users.

        Attachments

          Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-45590 PowerPC Slave

          • Major - Major loss of function.
          • Resolved
          links to

          Web Link Core PR

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            jglick Jesse Glick added a comment -

            Fine for Windows agents. For SSH agents we are apparently using a custom implementation which checks for Java 5!

            Show
            jglick Jesse Glick added a comment - Fine for Windows agents . For SSH agents we are apparently using a custom implementation  which checks for Java 5 !
            Hide
            Permalink
            amuniz Antonio Muñiz added a comment -

            Jesse Glick ssh-slaves should check for Java 7, right? - as its baseline is 1.625. Or just remove the default implementation.

            Show
            amuniz Antonio Muñiz added a comment - Jesse Glick ssh-slaves should check for Java 7, right? - as its baseline is 1.625. Or just remove the default implementation.
            Hide
            Permalink
            danielbeck Daniel Beck added a comment -

            For SSH Slaves, I tried, and failed, in https://github.com/jenkinsci/ssh-slaves-plugin/pull/27#issuecomment-304837590 – CC Oleg Nenashev.

            Show
            danielbeck Daniel Beck added a comment - For SSH Slaves, I tried, and failed, in https://github.com/jenkinsci/ssh-slaves-plugin/pull/27#issuecomment-304837590 – CC Oleg Nenashev .
            Hide
            Permalink
            jglick Jesse Glick added a comment -

            ssh-slaves should check for Java 7, right? - as its baseline is 1.625

            No, because when it is running on 2.54+, it ought to be requiring 8. That is why it should be using the core check, and the core check should be fixed.

            Show
            jglick Jesse Glick added a comment - ssh-slaves should check for Java 7, right? - as its baseline is 1.625 No, because when it is running on 2.54+, it ought to be requiring 8. That is why it should be using the core check, and the core check should be fixed.
            Hide
            Permalink
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Antonio Muniz
            Path:
            core/src/main/java/hudson/slaves/ComputerLauncher.java
            core/src/main/resources/hudson/slaves/Messages.properties
            core/src/test/java/hudson/slaves/ComputerLauncherTest.java
            http://jenkins-ci.org/commit/jenkins/f546ffc9ea09626e2e15bd4d9564ee32da18335d
            Log:
            JENKINS-44851 Agents JVM must be 1.8+ now (#2915)

            • Fix old Java 6 and 7 tests (an exception is expected now)
            • Add new tests to Java 8
            • Fix messages
            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Antonio Muniz Path: core/src/main/java/hudson/slaves/ComputerLauncher.java core/src/main/resources/hudson/slaves/Messages.properties core/src/test/java/hudson/slaves/ComputerLauncherTest.java http://jenkins-ci.org/commit/jenkins/f546ffc9ea09626e2e15bd4d9564ee32da18335d Log: JENKINS-44851 Agents JVM must be 1.8+ now (#2915) JENKINS-44851 Agents JVM must be 1.8+ now JENKINS-44851 Fix tests Fix old Java 6 and 7 tests (an exception is expected now) Add new tests to Java 8 Fix messages

              People

              Assignee:
              amuniz Antonio Muñiz
              Reporter:
              amuniz Antonio Muñiz
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: