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

Node names can be edited to include slashes and then cannot be removed

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

      Using Hudson v1.392, I did the following:

      • Created a node with a valid name (e.g., "Key Manager Build")
      • Edited the node name to something that should have been marked as invalid (e.g., "IdP / OAuth Build")

      I am now unable to edit or delete this node.

          [JENKINS-8438] Node names can be edited to include slashes and then cannot be removed

          Code changed in hudson
          User: Seiji Sogabe
          Path:
          changelog.html
          core/src/main/java/hudson/slaves/NodeDescriptor.java
          core/src/main/resources/hudson/slaves/Messages.properties
          core/src/main/resources/hudson/slaves/Messages_ja.properties
          http://hudson-labs.org/commit/core/46f2f02f9dbea085e67ff70673f38644b14b4f01
          Log:
          [FIXED JENKINS-8438] Node names can be edited to include slashes and then cannot be removed.

          SCM/JIRA link daemon added a comment - Code changed in hudson User: Seiji Sogabe Path: changelog.html core/src/main/java/hudson/slaves/NodeDescriptor.java core/src/main/resources/hudson/slaves/Messages.properties core/src/main/resources/hudson/slaves/Messages_ja.properties http://hudson-labs.org/commit/core/46f2f02f9dbea085e67ff70673f38644b14b4f01 Log: [FIXED JENKINS-8438] Node names can be edited to include slashes and then cannot be removed.

          jrh3k5 added a comment -

          Just out of curiosity, is there anything, short of editing the underlying XML, that would allow me to fix the node name?

          jrh3k5 added a comment - Just out of curiosity, is there anything, short of editing the underlying XML, that would allow me to fix the node name?

          sogabe added a comment -

          Edit your config.xml as follows and restart Hudson instance.

          <hudson>
          :
          <slaves>
          <slave>
          <name>Win7</name> // <-- Change this to correct name.
          <description>Windows 7</description>
          <remoteFS>/tmp</remoteFS>
          <numExecutors>1</numExecutors>
          <mode>NORMAL</mode>
          <retentionStrategy class="hudson.slaves.RetentionStrategy$Always"/>
          :

          sogabe added a comment - Edit your config.xml as follows and restart Hudson instance. <hudson> : <slaves> <slave> <name>Win7</name> // <-- Change this to correct name. <description>Windows 7</description> <remoteFS>/tmp</remoteFS> <numExecutors>1</numExecutors> <mode>NORMAL</mode> <retentionStrategy class="hudson.slaves.RetentionStrategy$Always"/> :

          jrh3k5 added a comment -

          Awesome, thanks!

          jrh3k5 added a comment - Awesome, thanks!

          dogfood added a comment -

          Integrated in hudson_main_trunk #450
          [FIXED JENKINS-8438] Node names can be edited to include slashes and then cannot be removed.

          Seiji Sogabe :
          Files :

          • core/src/main/resources/hudson/slaves/Messages.properties
          • core/src/main/java/hudson/slaves/NodeDescriptor.java
          • core/src/main/resources/hudson/slaves/Messages_ja.properties
          • changelog.html

          dogfood added a comment - Integrated in hudson_main_trunk #450 [FIXED JENKINS-8438] Node names can be edited to include slashes and then cannot be removed. Seiji Sogabe : Files : core/src/main/resources/hudson/slaves/Messages.properties core/src/main/java/hudson/slaves/NodeDescriptor.java core/src/main/resources/hudson/slaves/Messages_ja.properties changelog.html

            sogabe sogabe
            jrh3k5 jrh3k5
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated:
              Resolved: