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

JNLP slave JVM Options are inconsistently applied

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Minor
    • Resolution: Fixed
    • Component/s: core
    • Labels:
      None
    • Similar Issues:

      Description

      The JVM Options are injected into the JNLP file as the java-vm-args attribute but the command line launch option does not reflect the requirements for JVM Options.

      The "Run from slave command line" option should include the specified JVM options for the non-javaws varietals.

        Attachments

          Activity

          Hide
          danielbeck Daniel Beck added a comment -

          I'm fairly sure this has been reported before (or was that for the installed slave service on Windows?)

          Show
          danielbeck Daniel Beck added a comment - I'm fairly sure this has been reported before (or was that for the installed slave service on Windows?)
          Hide
          stephenconnolly Stephen Connolly added a comment -

          Installed service for windows was reported previously. Nobody from what I could find had reported that the displayed command line launch alternative did not include the JVM options

          Show
          stephenconnolly Stephen Connolly added a comment - Installed service for windows was reported previously. Nobody from what I could find had reported that the displayed command line launch alternative did not include the JVM options
          Hide
          danielbeck Daniel Beck added a comment -

          Stephen Connolly Should this be assigned to you?

          Show
          danielbeck Daniel Beck added a comment - Stephen Connolly Should this be assigned to you?
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Stephen Connolly
          Path:
          core/src/main/resources/hudson/slaves/JNLPLauncher/main.jelly
          http://jenkins-ci.org/commit/jenkins/84e485fc5e608bccf080bf46c0e31c43c6bef474
          Log:
          [FIXED JENKINS-28111] JNLP slave JVM Options are inconsistently applied

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: core/src/main/resources/hudson/slaves/JNLPLauncher/main.jelly http://jenkins-ci.org/commit/jenkins/84e485fc5e608bccf080bf46c0e31c43c6bef474 Log: [FIXED JENKINS-28111] JNLP slave JVM Options are inconsistently applied
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Stephen Connolly
          Path:
          core/src/main/resources/hudson/slaves/JNLPLauncher/main.jelly
          http://jenkins-ci.org/commit/jenkins/ce94434b164d1ac74132cecdc6f123787decbf48
          Log:
          Merge pull request #1676 from stephenc/jenkins-28111

          [FIXED JENKINS-28111] JNLP slave JVM Options are inconsistently applied

          Compare: https://github.com/jenkinsci/jenkins/compare/5b4ed2dae2dd...ce94434b164d

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: core/src/main/resources/hudson/slaves/JNLPLauncher/main.jelly http://jenkins-ci.org/commit/jenkins/ce94434b164d1ac74132cecdc6f123787decbf48 Log: Merge pull request #1676 from stephenc/jenkins-28111 [FIXED JENKINS-28111] JNLP slave JVM Options are inconsistently applied Compare: https://github.com/jenkinsci/jenkins/compare/5b4ed2dae2dd...ce94434b164d
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Stephen Connolly
          Path:
          changelog.html
          http://jenkins-ci.org/commit/jenkins/9c9049eb4da6145261d6b5fda6ed57dd676bf108
          Log:
          Noting merge of JENKINS-28111

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: changelog.html http://jenkins-ci.org/commit/jenkins/9c9049eb4da6145261d6b5fda6ed57dd676bf108 Log: Noting merge of JENKINS-28111
          Hide
          dogfood dogfood added a comment -

          Integrated in jenkins_main_trunk #4114

          Result = SUCCESS

          Show
          dogfood dogfood added a comment - Integrated in jenkins_main_trunk #4114 Result = SUCCESS
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: tfennelly
          Path:
          changelog.html
          cli/pom.xml
          core/pom.xml
          core/src/main/java/hudson/model/Executor.java
          core/src/main/resources/hudson/slaves/JNLPLauncher/main.jelly
          ips/jenkins.xml
          ips/proto.py
          ips/readme.txt
          plugins/pom.xml
          pom.xml
          test/pom.xml
          war/pom.xml
          http://jenkins-ci.org/commit/jenkins/30ece5ffb20d354173cec3b6d7a2ea902a9eb681
          Log:
          Merge branch 'master' into extensionlist-listener

          • master:
            Noting merge of JENKINS-28111
            @jglick wants it this way
            Noting merge of JENKINS-28115
            [FIXED JENKINS-28115] Division by zero in Executor.getProgress()
            [FIXED JENKINS-28111] JNLP slave JVM Options are inconsistently applied
            updated changelog for release
            [maven-release-plugin] prepare for next development iteration
            [maven-release-plugin] prepare release jenkins-1.611
            remove ips profile
            Now that packaging is in https://github.com/jenkinsci/packaging and given that the ips packaging is stale, we should just drop this
          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: tfennelly Path: changelog.html cli/pom.xml core/pom.xml core/src/main/java/hudson/model/Executor.java core/src/main/resources/hudson/slaves/JNLPLauncher/main.jelly ips/jenkins.xml ips/proto.py ips/readme.txt plugins/pom.xml pom.xml test/pom.xml war/pom.xml http://jenkins-ci.org/commit/jenkins/30ece5ffb20d354173cec3b6d7a2ea902a9eb681 Log: Merge branch 'master' into extensionlist-listener master: Noting merge of JENKINS-28111 @jglick wants it this way Noting merge of JENKINS-28115 [FIXED JENKINS-28115] Division by zero in Executor.getProgress() [FIXED JENKINS-28111] JNLP slave JVM Options are inconsistently applied updated changelog for release [maven-release-plugin] prepare for next development iteration [maven-release-plugin] prepare release jenkins-1.611 remove ips profile Now that packaging is in https://github.com/jenkinsci/packaging and given that the ips packaging is stale, we should just drop this

            People

            Assignee:
            stephenconnolly Stephen Connolly
            Reporter:
            stephenconnolly Stephen Connolly
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: