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

libvirt-slave builds broken connection URI

    Details

    • Similar Issues:

      Description

      When building the connection URI in Hypervisor.java, the query parameter "?no_tty=1" is always appended. This does not take into account, that users might already have added their own parameters in the advanced settings already.

      E.g: In order to use a different SSH-key location and disable host-key verification, I use the following Hypervisor-System-URL:

      system?keyfile=/etc/jenkins/id_dsa_remoting&no_verify=1

      The attached patch fixes this by using '&' resp. '?' conditionally.

      Please note, that the patch also contains an update in the (rather old) plugin's pom.xml to make it build with current jenkins (and JDK 1.7).

        Attachments

          Activity

          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: tastybug
          Path:
          src/main/java/hudson/plugins/libvirt/Hypervisor.java
          http://jenkins-ci.org/commit/libvirt-slave-plugin/a9c654c3fd3636c79f6532f6e52e4fd05ab0a602
          Log:
          -Fixing JENKINS-14617

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: tastybug Path: src/main/java/hudson/plugins/libvirt/Hypervisor.java http://jenkins-ci.org/commit/libvirt-slave-plugin/a9c654c3fd3636c79f6532f6e52e4fd05ab0a602 Log: -Fixing JENKINS-14617
          Hide
          tastybug Philipp Bartsch added a comment -

          Fixed in 1.7

          Show
          tastybug Philipp Bartsch added a comment - Fixed in 1.7
          Hide
          gkr G. Kr. added a comment -

          closing as fixed

          Show
          gkr G. Kr. added a comment - closing as fixed

            People

            • Assignee:
              tastybug Philipp Bartsch
              Reporter:
              felfert Fritz Elfert
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: