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

libvirt-slaves plugin doesn't start VMs when secondary startup method is JNLP

    Details

    • Similar Issues:

      Description

      When secondary startup method is JNLP (or any other passive startup method that does not support launching the slave agent from the master) the VMs are not started by the libvirt-slaves plugin.

      This default behavior is actually correct as starting the VM does not yet connect it with hudson.

      However when I (have to) use Windows slaves, it is not always possible to use the windows remote deployment launch option. Using JNLP is easy and always possible, however now the VMs are no longer switched on.

      It should be possible to override whether starting a VM is enough to connect the slave or not. (Leaving it to startup scripts inside the OS to connect to the master)

      Attached is a patch force-vm-launch-support.patch and a locally compiled plugin libvirt-slave.hpi that allows to select whether starting the VM is always supported no matter what the secondary launch option does.
      I would highly appreciate if this could be added to the next release.

        Attachments

          Activity

          juergenk juergenk created issue -
          pedjak Predrag Knezevic made changes -
          Field Original Value New Value
          Assignee mmornati [ mmornati ] Philipp Bartsch [ tastybug ]
          Affects Version/s current [ 10162 ]
          Component/s libvirt-slave [ 15985 ]
          Component/s libvirtslave [ 15840 ]
          Component/s plugin [ 15491 ]
          rtyler R. Tyler Croy made changes -
          Workflow JNJira [ 138107 ] JNJira + In-Review [ 174822 ]

            People

            • Assignee:
              tastybug Philipp Bartsch
              Reporter:
              juergenk juergenk
            • Votes:
              1 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated: