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

          relates to

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-41511 Cannot change any property in configureSecurity when slave agent port is enforced

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved
          links to

          Web Link PR #2545

            Activity

            Descending order - Click to sort in ascending order
            Hide
            Permalink
            danielbeck Daniel Beck added a comment -

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

            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.
            Hide
            Permalink
            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)
            Hide
            Permalink
            danielbeck Daniel Beck added a comment -

            https://wiki.jenkins-ci.org/display/JENKINS/GitHub+commit+messages

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

              People

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

                Dates

                Created:
                Updated:
                Resolved: