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

jenkins.model.Jenkins.slaveAgentPort is not honored since Jenkins 2.0

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      SetupWizard forcibly disables jnlp even if it is explicitly set through this system property documented on https://wiki.jenkins-ci.org/display/JENKINS/Features+controlled+by+system+properties

        Attachments

          Issue Links

            Activity

            Show
            danielbeck Daniel Beck added a comment - https://wiki.jenkins-ci.org/display/JENKINS/GitHub+commit+messages
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Daniel Beck
            Path:
            core/src/main/java/hudson/security/GlobalSecurityConfiguration.java
            core/src/main/java/jenkins/install/SetupWizard.java
            core/src/main/java/jenkins/model/Jenkins.java
            core/src/main/resources/hudson/security/GlobalSecurityConfiguration/index.groovy
            core/src/main/resources/hudson/security/GlobalSecurityConfiguration/index.properties
            core/src/main/resources/jenkins/model/Jenkins/EnforceSlaveAgentPortAdministrativeMonitor/message.jelly
            core/src/main/resources/jenkins/model/Jenkins/EnforceSlaveAgentPortAdministrativeMonitor/message.properties
            core/src/main/resources/jenkins/model/Messages.properties
            http://jenkins-ci.org/commit/jenkins/5d58ce9f2f16c5849401d7fc761c2035655f9701
            Log:
            Merge pull request #2545 from Vlatombe/JENKINS-38187

            [FIX JENKINS-38187] Fix setting JNLP port through system property
            (cherry picked from commit 3f5fe7f6b9b8dab37e82b360574102c6388729e8)

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Daniel Beck Path: core/src/main/java/hudson/security/GlobalSecurityConfiguration.java core/src/main/java/jenkins/install/SetupWizard.java core/src/main/java/jenkins/model/Jenkins.java core/src/main/resources/hudson/security/GlobalSecurityConfiguration/index.groovy core/src/main/resources/hudson/security/GlobalSecurityConfiguration/index.properties core/src/main/resources/jenkins/model/Jenkins/EnforceSlaveAgentPortAdministrativeMonitor/message.jelly core/src/main/resources/jenkins/model/Jenkins/EnforceSlaveAgentPortAdministrativeMonitor/message.properties core/src/main/resources/jenkins/model/Messages.properties http://jenkins-ci.org/commit/jenkins/5d58ce9f2f16c5849401d7fc761c2035655f9701 Log: Merge pull request #2545 from Vlatombe/ JENKINS-38187 [FIX JENKINS-38187] Fix setting JNLP port through system property (cherry picked from commit 3f5fe7f6b9b8dab37e82b360574102c6388729e8)
            Show
            danielbeck Daniel Beck added a comment - Forgot to document on https://wiki.jenkins-ci.org/display/JENKINS/Features+controlled+by+system+properties . Not cool.

              People

              • Assignee:
                vlatombe Vincent Latombe
                Reporter:
                vlatombe Vincent Latombe
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: