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

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

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • core

      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

          [JENKINS-38187] jenkins.model.Jenkins.slaveAgentPort is not honored since Jenkins 2.0

          Daniel Beck added a comment -

          Daniel Beck added a comment - https://wiki.jenkins-ci.org/display/JENKINS/GitHub+commit+messages

          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)

          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)

          Daniel Beck added a comment -

          Daniel Beck added a comment - Forgot to document on https://wiki.jenkins-ci.org/display/JENKINS/Features+controlled+by+system+properties . Not cool.

            vlatombe Vincent Latombe
            vlatombe Vincent Latombe
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: