In our environment it would be very helpful to be able to set Permissions, who can see specific agent.

      It should be like with jobs/projects, where the method "hasPermission()" is called, before you get all Items of the jenkins-instance.

      So it would be just needed in the Computer.java a new attribute "Permission VIEW", and in the Jenkins.java in the Method "getComputer()" the check, if the user has the Permission to see this agent.

          [JENKINS-16502] Permission to see an agent

          Christian Meyer created issue -

          Code changed in jenkins
          User: Stephen Connolly
          Path:
          changelog.html
          core/src/main/java/hudson/model/Computer.java
          core/src/main/java/jenkins/model/Jenkins.java
          core/src/main/resources/hudson/model/Computer/builds.jelly
          core/src/main/resources/hudson/model/Computer/delete.jelly
          core/src/main/resources/hudson/model/Computer/index.jelly
          core/src/main/resources/hudson/model/Computer/load-statistics.jelly
          core/src/main/resources/hudson/model/Computer/markOffline.jelly
          core/src/main/resources/hudson/model/Computer/setOfflineCause.jelly
          core/src/main/resources/hudson/model/Messages.properties
          http://jenkins-ci.org/commit/jenkins/647695e2ffc2f568ca1b80bc5b0cd7b0658eeb3d
          Log:
          JENKINS-16502 Permission to see an executor/slave

          • This is an initial version of the feature.
          • The information about slave names is still exposed via label autocomplete

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: changelog.html core/src/main/java/hudson/model/Computer.java core/src/main/java/jenkins/model/Jenkins.java core/src/main/resources/hudson/model/Computer/builds.jelly core/src/main/resources/hudson/model/Computer/delete.jelly core/src/main/resources/hudson/model/Computer/index.jelly core/src/main/resources/hudson/model/Computer/load-statistics.jelly core/src/main/resources/hudson/model/Computer/markOffline.jelly core/src/main/resources/hudson/model/Computer/setOfflineCause.jelly core/src/main/resources/hudson/model/Messages.properties http://jenkins-ci.org/commit/jenkins/647695e2ffc2f568ca1b80bc5b0cd7b0658eeb3d Log: JENKINS-16502 Permission to see an executor/slave This is an initial version of the feature. The information about slave names is still exposed via label autocomplete

          Code changed in jenkins
          User: Stephen Connolly
          Path:
          changelog.html
          core/src/main/java/hudson/model/Computer.java
          core/src/main/java/jenkins/model/Jenkins.java
          core/src/main/resources/hudson/model/Computer/builds.jelly
          core/src/main/resources/hudson/model/Computer/delete.jelly
          core/src/main/resources/hudson/model/Computer/index.jelly
          core/src/main/resources/hudson/model/Computer/load-statistics.jelly
          core/src/main/resources/hudson/model/Computer/markOffline.jelly
          core/src/main/resources/hudson/model/Computer/setOfflineCause.jelly
          core/src/main/resources/hudson/model/Messages.properties
          http://jenkins-ci.org/commit/jenkins/903ef107d3b279c2cffce99b4d61734fba286ef0
          Log:
          Merge pull request #951 from stephenc/master

          JENKINS-16502 Permission to see an executor/slave

          Compare: https://github.com/jenkinsci/jenkins/compare/84a176beefd2...903ef107d3b2

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: changelog.html core/src/main/java/hudson/model/Computer.java core/src/main/java/jenkins/model/Jenkins.java core/src/main/resources/hudson/model/Computer/builds.jelly core/src/main/resources/hudson/model/Computer/delete.jelly core/src/main/resources/hudson/model/Computer/index.jelly core/src/main/resources/hudson/model/Computer/load-statistics.jelly core/src/main/resources/hudson/model/Computer/markOffline.jelly core/src/main/resources/hudson/model/Computer/setOfflineCause.jelly core/src/main/resources/hudson/model/Messages.properties http://jenkins-ci.org/commit/jenkins/903ef107d3b279c2cffce99b4d61734fba286ef0 Log: Merge pull request #951 from stephenc/master JENKINS-16502 Permission to see an executor/slave Compare: https://github.com/jenkinsci/jenkins/compare/84a176beefd2...903ef107d3b2

          towards Jenkins 1.533

          Stephen Connolly added a comment - towards Jenkins 1.533
          Stephen Connolly made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]

          dogfood added a comment -

          Integrated in jenkins_main_trunk #2890
          JENKINS-16502 Permission to see an executor/slave (Revision 647695e2ffc2f568ca1b80bc5b0cd7b0658eeb3d)

          Result = UNSTABLE
          Stephen Connolly : 647695e2ffc2f568ca1b80bc5b0cd7b0658eeb3d
          Files :

          • changelog.html
          • core/src/main/resources/hudson/model/Computer/builds.jelly
          • core/src/main/java/hudson/model/Computer.java
          • core/src/main/resources/hudson/model/Computer/setOfflineCause.jelly
          • core/src/main/resources/hudson/model/Computer/delete.jelly
          • core/src/main/resources/hudson/model/Computer/load-statistics.jelly
          • core/src/main/resources/hudson/model/Messages.properties
          • core/src/main/resources/hudson/model/Computer/index.jelly
          • core/src/main/resources/hudson/model/Computer/markOffline.jelly
          • core/src/main/java/jenkins/model/Jenkins.java

          dogfood added a comment - Integrated in jenkins_main_trunk #2890 JENKINS-16502 Permission to see an executor/slave (Revision 647695e2ffc2f568ca1b80bc5b0cd7b0658eeb3d) Result = UNSTABLE Stephen Connolly : 647695e2ffc2f568ca1b80bc5b0cd7b0658eeb3d Files : changelog.html core/src/main/resources/hudson/model/Computer/builds.jelly core/src/main/java/hudson/model/Computer.java core/src/main/resources/hudson/model/Computer/setOfflineCause.jelly core/src/main/resources/hudson/model/Computer/delete.jelly core/src/main/resources/hudson/model/Computer/load-statistics.jelly core/src/main/resources/hudson/model/Messages.properties core/src/main/resources/hudson/model/Computer/index.jelly core/src/main/resources/hudson/model/Computer/markOffline.jelly core/src/main/java/jenkins/model/Jenkins.java

          Code changed in jenkins
          User: Jesse Glick
          Path:
          changelog.html
          core/src/main/java/hudson/model/Computer.java
          core/src/main/java/jenkins/model/Jenkins.java
          core/src/main/resources/hudson/model/Computer/builds.jelly
          core/src/main/resources/hudson/model/Computer/delete.jelly
          core/src/main/resources/hudson/model/Computer/index.jelly
          core/src/main/resources/hudson/model/Computer/load-statistics.jelly
          core/src/main/resources/hudson/model/Computer/markOffline.jelly
          core/src/main/resources/hudson/model/Computer/setOfflineCause.jelly
          core/src/main/resources/hudson/model/Messages.properties
          http://jenkins-ci.org/commit/jenkins/3911ea2c1450e73fabaf51acc1b635a44a5257c6
          Log:
          Revert "JENKINS-16502 Permission to see an executor/slave"

          This reverts commit 647695e2ffc2f568ca1b80bc5b0cd7b0658eeb3d.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: changelog.html core/src/main/java/hudson/model/Computer.java core/src/main/java/jenkins/model/Jenkins.java core/src/main/resources/hudson/model/Computer/builds.jelly core/src/main/resources/hudson/model/Computer/delete.jelly core/src/main/resources/hudson/model/Computer/index.jelly core/src/main/resources/hudson/model/Computer/load-statistics.jelly core/src/main/resources/hudson/model/Computer/markOffline.jelly core/src/main/resources/hudson/model/Computer/setOfflineCause.jelly core/src/main/resources/hudson/model/Messages.properties http://jenkins-ci.org/commit/jenkins/3911ea2c1450e73fabaf51acc1b635a44a5257c6 Log: Revert " JENKINS-16502 Permission to see an executor/slave" This reverts commit 647695e2ffc2f568ca1b80bc5b0cd7b0658eeb3d.

          Code changed in jenkins
          User: Jesse Glick
          Path:
          changelog.html
          core/src/main/java/hudson/model/Computer.java
          core/src/main/java/jenkins/model/Jenkins.java
          core/src/main/resources/hudson/model/Computer/builds.jelly
          core/src/main/resources/hudson/model/Computer/delete.jelly
          core/src/main/resources/hudson/model/Computer/index.jelly
          core/src/main/resources/hudson/model/Computer/load-statistics.jelly
          core/src/main/resources/hudson/model/Computer/markOffline.jelly
          core/src/main/resources/hudson/model/Computer/setOfflineCause.jelly
          core/src/main/resources/hudson/model/Messages.properties
          http://jenkins-ci.org/commit/jenkins/0ebd8a6c3dedd8ff06a08af175571c6bc49893d9
          Log:
          Revert "JENKINS-16502 Permission to see an executor/slave"

          This reverts commit 647695e2ffc2f568ca1b80bc5b0cd7b0658eeb3d.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: changelog.html core/src/main/java/hudson/model/Computer.java core/src/main/java/jenkins/model/Jenkins.java core/src/main/resources/hudson/model/Computer/builds.jelly core/src/main/resources/hudson/model/Computer/delete.jelly core/src/main/resources/hudson/model/Computer/index.jelly core/src/main/resources/hudson/model/Computer/load-statistics.jelly core/src/main/resources/hudson/model/Computer/markOffline.jelly core/src/main/resources/hudson/model/Computer/setOfflineCause.jelly core/src/main/resources/hudson/model/Messages.properties http://jenkins-ci.org/commit/jenkins/0ebd8a6c3dedd8ff06a08af175571c6bc49893d9 Log: Revert " JENKINS-16502 Permission to see an executor/slave" This reverts commit 647695e2ffc2f568ca1b80bc5b0cd7b0658eeb3d.

          Jesse Glick added a comment -

          Reverted since this caused test failures, and these failures in fact seem legitimate: old installations would not have granted the new permission to non-admins, so computers would be hidden by default. Need to do tricks to conditionally enable this permission.

          Jesse Glick added a comment - Reverted since this caused test failures, and these failures in fact seem legitimate: old installations would not have granted the new permission to non-admins, so computers would be hidden by default. Need to do tricks to conditionally enable this permission.
          Jesse Glick made changes -
          Assignee Original: Christian Meyer [ chrissy ] New: Stephen Connolly [ stephenconnolly ]
          Resolution Original: Fixed [ 1 ]
          Status Original: Resolved [ 5 ] New: Reopened [ 4 ]

            Unassigned Unassigned
            chrissy Christian Meyer
            Votes:
            7 Vote for this issue
            Watchers:
            13 Start watching this issue

              Created:
              Updated: