• Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Critical Critical
    • swarm-plugin
    • None

      I'm sorry, I keep finding bugs in my own code...
      I noticed that the swarm client wouldn't recognize all tools, some were added as Ant installations. That's because Ant is the first item in the dropdown.
      Turns out the way I generate the toolinstallation key is not entirely correct. The keys that I have encountered that don't work are those for Ant and Maven. In my tests I always used Groovy, JDK and Ant...Ant seemed to work because it default to the first item in the listbox, being Ant..

      I have asked for some help here: https://groups.google.com/forum/#!topic/jenkinsci-dev/QwYKkMvtw9s

          [JENKINS-25064] Swarm toollocations don't work for all tools

          Found a solution but I'm gonna test first a bit

          Nico Mommaerts added a comment - Found a solution but I'm gonna test first a bit

          Code changed in jenkins
          User: nico.mommaerts
          Path:
          plugin/src/main/java/hudson/plugins/swarm/PluginImpl.java
          http://jenkins-ci.org/commit/swarm-plugin/73da69a23a67a7af32bc8df24fa9ff10906f6de9
          Log:
          [FIXED JENKINS-25064] Instead of constructing the toollocation key, just use the existing descriptor

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: nico.mommaerts Path: plugin/src/main/java/hudson/plugins/swarm/PluginImpl.java http://jenkins-ci.org/commit/swarm-plugin/73da69a23a67a7af32bc8df24fa9ff10906f6de9 Log: [FIXED JENKINS-25064] Instead of constructing the toollocation key, just use the existing descriptor

          Code changed in jenkins
          User: Peter Jönsson
          Path:
          plugin/src/main/java/hudson/plugins/swarm/PluginImpl.java
          http://jenkins-ci.org/commit/swarm-plugin/0dddc0fccbbe5797ef4b8a032ed19948092da18a
          Log:
          Merge pull request #20 from mrdfuse/JENKINS-25064

          [FIXED JENKINS-25064] Instead of constructing the toollocation key, just use the existing descriptor

          Compare: https://github.com/jenkinsci/swarm-plugin/compare/30ee1f19cb0d...0dddc0fccbbe

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Peter Jönsson Path: plugin/src/main/java/hudson/plugins/swarm/PluginImpl.java http://jenkins-ci.org/commit/swarm-plugin/0dddc0fccbbe5797ef4b8a032ed19948092da18a Log: Merge pull request #20 from mrdfuse/ JENKINS-25064 [FIXED JENKINS-25064] Instead of constructing the toollocation key, just use the existing descriptor Compare: https://github.com/jenkinsci/swarm-plugin/compare/30ee1f19cb0d...0dddc0fccbbe

            mindjiver Peter Jönsson
            mr_dfuse Nico Mommaerts
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: