vagrant-qemu 0.3.6 → 0.3.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 47636dd2f589218fd058d32661aa2844a842ea0c1fce7c4fbf1bceb5efc58bea
-  data.tar.gz: d4e04639beb03206cdcb5a09c2b63d608efe18b623ede070d6b2a4a1bafb452e
+  metadata.gz: ad58eff4d1ca4ecc7899ccd8f5cf5e13497a0ba7f226dc425858f89a59cc2151
+  data.tar.gz: b91caf53e6f505b0ec6e7a6aefd4773f0205fcfff35f1015f915a4ce9056a0aa
 SHA512:
-  metadata.gz: 80ef6dbcb15d76fc97296b178416aaede61bcf4133fc38cac83fd92854cc1e92c8159b49ff961939e498a344b33a328fe87a6995675b65d91a6286d2ae0334ed
-  data.tar.gz: bad9fba78583cb6cb236e81728eba6d9fd53747b25abfe94b1227e0258364e536c093a126a3daea13a72e7bb1403df86710739032b931012ceee7f2a529dadd1
+  metadata.gz: 56885cedf852f75eaf87b6ab73904eb43a7775d36a7586a2f5558905726ccf0d5ebdafb69ea5fe9f6e34733b404d49245d7a87c0224d6c25e7aeb21b547eedd3
+  data.tar.gz: a7d03a0326ffe78dfad134a11cc402a091a872c97b3bbf5fea0c769a3712f604fc2efa544cff29d2a8ccb34cb82af086a6f5a7ff8dbfb4bdb7cee387354c63e9

data/CHANGELOG.md

@@ -71,3 +71,13 @@
 
 * Config 'image_path' support array type
 * Try to support libvirt box v2 format
+
+# 0.3.7 (2025-02-02)
+
+* Ignore exception after sending 'system_powerdown' cmd, fix windows halt error
+* Move pid file to tmp dir
+* Be able to auto correct ssh port collisions, new config: ssh_auto_correct
+
+# 0.3.8 (2025-02-21)
+
+* Fix regression that ssh_port is not working #68

data/README.md

@@ -86,6 +86,7 @@ This provider exposes a few provider-specific configuration options:
   * `memory` - The memory setting of VM, default: `4G`
 * debug/expert
   * `ssh_host` - The SSH IP used to access VM, default: `127.0.0.1`
+  * `ssh_auto_correct` - Auto correct port collisions for ssh port, default: `false`
   * `net_device` - The network device, default: `virtio-net-device`
   * `drive_interface` - The interface type for the main drive, default `virtio`
   * `image_path` - The path (or array of paths) to qcow2 image for box-less VM, default is nil value
@@ -226,7 +227,6 @@ end
 8. Windows host
 
 Windows version QEMU doesn't support `daemonize` mode and unix socket
-(Notes: not tested)
 
 ```
 Vagrant.configure("2") do |config|
@@ -240,6 +240,32 @@ Vagrant.configure("2") do |config|
 end
 ```
 
+9. Auto port collisions for ssh port (multiple machine)
+
+```
+Vagrant.configure("2") do |config|
+
+  config.vm.define "vm1" do |c|
+    c.vm.box = "ppggff/centos-7-aarch64-2009-4K"
+    c.vm.provider "qemu" do |qe|
+      qe.memory = "2G"
+      qe.ssh_auto_correct = true
+    end
+    c.vm.synced_folder ".", "/vagrant", disabled: true
+  end
+
+  config.vm.define "vm2" do |c|
+    c.vm.box = "ppggff/centos-7-aarch64-2009-4K"
+    c.vm.provider "qemu" do |qe|
+      qe.memory = "2G"
+      qe.ssh_auto_correct = true
+    end
+    c.vm.synced_folder ".", "/vagrant", disabled: true
+  end
+
+end
+```
+
 ## Debug
 
 Serial port is exported to unix socket: `<user_home>/.vagrant.d/tmp/vagrant-qemu/<id>/qemu_socket_serial`, or `debug_port`.
@@ -299,6 +325,21 @@ Vagrant.configure("2") do |config|
 end
 ```
 
+As an alternative solution(helpful for macOS) it is possible to use 9p file system via virtio. 
+
+```
+config.vm.synced_folder ".", "/vagrant", disabled: true
+config.vm.provider "qemu" do |qe|
+    qe.extra_qemu_args = %w(-virtfs local,path=.,mount_tag=shared,security_model=mapped)
+end
+```
+This will pass "current directory" to mount point tagged "shared"
+Use the following /etc/fstab entry on the vagrant vm to mount the shared directory into /home/vagrant/shared
+```
+shared /home/vagrant/shared 9p _netdev,trans=virtio,msize=524288 0
+```
+Please keep in mind that the guest OS will need to install 9p dependencies to handle the 9p filestystem.
+
 ### 2. netcat does not support the -U parameter
 
 I had netcat installed through home brew and it does not support the -U parameter.

data/lib/vagrant-qemu/action/prepare_forwarded_port_collision_params.rb

@@ -16,16 +16,30 @@ module VagrantPlugins
       # Build the remap for any existing collision detections
       remap = {}
       env[:port_collision_remap] = remap
+
+      has_ssh_forward = false
       machine.config.vm.networks.each do |type, options|
         next if type != :forwarded_port
 
-        # remap ssh.host to ssh_port
+        # update ssh.host to ssh_port
         if options[:id] == "ssh"
-          remap[options[:host]] = machine.provider_config.ssh_port
+          options[:host] = machine.provider_config.ssh_port
+          options[:auto_correct] = machine.provider_config.ssh_auto_correct
+          has_ssh_forward = true
           break
         end
       end
 
+      if !has_ssh_forward
+        machine.config.vm.network :forwarded_port,
+          :guest => 22, 
+          :host => machine.provider_config.ssh_port, 
+          :host_ip => "127.0.0.1", 
+          :id => "ssh", 
+          :auto_correct => machine.provider_config.ssh_auto_correct,
+          :protocol => "tcp"
+      end
+
       @app.call(env)
       end
     end

data/lib/vagrant-qemu/action/read_state.rb

@@ -23,6 +23,11 @@ module VagrantPlugins
           else
             env[:machine_state_id] = :not_created
           end
+
+          # Update ssh_port if needed
+          if env[:machine_state_id] == :running
+            env[:machine].provider_config.ssh_port = env[:machine].provider.driver.get_ssh_port(env[:machine].provider_config.ssh_port)
+          end
           @app.call(env)
         end
       end

data/lib/vagrant-qemu/action/start_instance.rb

@@ -11,6 +11,7 @@ module VagrantPlugins
         end
 
         def call(env)
+          fwPorts = forwarded_ports(env)
           options = {
             :ssh_host => env[:machine].provider_config.ssh_host,
             :ssh_port => env[:machine].provider_config.ssh_port,
@@ -23,7 +24,7 @@ module VagrantPlugins
             :drive_interface => env[:machine].provider_config.drive_interface,
             :extra_qemu_args => env[:machine].provider_config.extra_qemu_args,
             :extra_netdev_args => env[:machine].provider_config.extra_netdev_args,
-            :ports => forwarded_ports(env),
+            :ports => fwPorts,
             :control_port => env[:machine].provider_config.control_port,
             :debug_port => env[:machine].provider_config.debug_port,
             :no_daemonize => env[:machine].provider_config.no_daemonize,
@@ -43,7 +44,12 @@ module VagrantPlugins
             next if type != :forwarded_port
 
             # Don't include SSH
-            next if options[:id] == "ssh"
+            if options[:id] == "ssh"
+              if options[:host] != env[:machine].provider_config.ssh_port
+                  env[:machine].provider_config.ssh_port = options[:host]
+              end
+              next
+            end
 
             # Skip port if it is disabled
             next if options[:disabled]

data/lib/vagrant-qemu/action.rb

@@ -116,7 +116,7 @@ module VagrantPlugins
             end
 
             b1.use Provision
-            b1.use EnvSet, port_collision_repair: false
+            b1.use EnvSet, port_collision_repair: true
             b1.use PrepareForwardedPortCollisionParams
             b1.use HandleForwardedPortCollisions
             b1.use SyncedFolderCleanup

data/lib/vagrant-qemu/config.rb

@@ -5,6 +5,7 @@ module VagrantPlugins
     class Config < Vagrant.plugin("2", :config)
       attr_accessor :ssh_host
       attr_accessor :ssh_port
+      attr_accessor :ssh_auto_correct
       attr_accessor :arch
       attr_accessor :machine
       attr_accessor :cpu
@@ -25,6 +26,7 @@ module VagrantPlugins
       def initialize
         @ssh_host = UNSET_VALUE
         @ssh_port = UNSET_VALUE
+        @ssh_auto_correct = UNSET_VALUE
         @arch = UNSET_VALUE
         @machine = UNSET_VALUE
         @cpu = UNSET_VALUE
@@ -55,6 +57,7 @@ module VagrantPlugins
       def finalize!
         @ssh_host = "127.0.0.1" if @ssh_host == UNSET_VALUE
         @ssh_port = 50022 if @ssh_port == UNSET_VALUE
+        @ssh_auto_correct = false if @ssh_auto_correct == UNSET_VALUE
         @arch = "aarch64" if @arch == UNSET_VALUE
         @machine = "virt,accel=hvf,highmem=on" if @machine == UNSET_VALUE
         @cpu = "host" if @cpu == UNSET_VALUE

data/lib/vagrant-qemu/driver.rb

@@ -1,5 +1,6 @@
 require 'childprocess'
 require 'securerandom'
+require 'yaml'
 
 require "vagrant/util/busy"
 require 'vagrant/util/io'
@@ -10,7 +11,7 @@ require_relative "plugin"
 
 module VagrantPlugins
   module QEMU
-	  class Driver
+    class Driver
       # @return [String] VM ID
       attr_reader :vm_id
       attr_reader :data_dir
@@ -45,7 +46,6 @@ module VagrantPlugins
       def start(options)
         if !running?
           id_dir = @data_dir.join(@vm_id)
-          pid_file = id_dir.join("qemu.pid").to_s
 
           image_path = Array.new
           image_count = id_dir.glob("linked-box*.img").count
@@ -57,6 +57,10 @@ module VagrantPlugins
           id_tmp_dir = @tmp_dir.join(@vm_id)
           FileUtils.mkdir_p(id_tmp_dir)
 
+          # dump options
+          options_file = id_tmp_dir.join("options.yml")
+          File.write(options_file, options.to_yaml)
+
           control_socket = ""
           if !options[:control_port].nil?
             control_socket = "port=#{options[:control_port]},host=localhost,ipv4=on"
@@ -113,6 +117,7 @@ module VagrantPlugins
           end
 
           # control
+          pid_file = id_tmp_dir.join("qemu.pid").to_s
           cmd += %W(-chardev socket,id=mon0,#{control_socket},server=on,wait=off)
           cmd += %W(-mon chardev=mon0,mode=readline)
           cmd += %W(-chardev socket,id=ser0,#{debug_socket},server=on,wait=off)
@@ -139,7 +144,7 @@ module VagrantPlugins
             Socket.tcp("localhost", options[:control_port], connect_timeout: 5) do |sock|
               sock.print "system_powerdown\n"
               sock.close_write
-              sock.read
+              sock.read rescue nil
             end
           else
             id_tmp_dir = @tmp_dir.join(@vm_id)
@@ -147,12 +152,24 @@ module VagrantPlugins
             Socket.unix(unix_socket_path) do |sock|
               sock.print "system_powerdown\n"
               sock.close_write
-              sock.read
+              sock.read rescue nil
             end
-         end
+          end
         end
       end
 
+      def get_ssh_port(ssh_port)
+        id_tmp_dir = @tmp_dir.join(@vm_id)
+        options_file = id_tmp_dir.join("options.yml")
+
+        if options_file.file?
+          options = YAML.load_file(options_file) rescue nil
+          ssh_port = options[:ssh_port] if !options.nil? && options.key?(:ssh_port)
+        end
+
+        ssh_port
+      end
+
       def import(options)
         new_id = "vq_" + SecureRandom.urlsafe_base64(8)
 
@@ -185,7 +202,7 @@ module VagrantPlugins
       end
 
       def running?
-        pid_file = @data_dir.join(@vm_id).join("qemu.pid")
+        pid_file = @tmp_dir.join(@vm_id).join("qemu.pid")
         return false if !pid_file.file?
 
         begin

data/lib/vagrant-qemu/version.rb

@@ -1,5 +1,5 @@
 module VagrantPlugins
   module QEMU
-    VERSION = '0.3.6'
+    VERSION = '0.3.8'
   end
 end

data/x

@@ -0,0 +1,76 @@
+diff --git a/lib/vagrant-qemu/action.rb b/lib/vagrant-qemu/action.rb
+index bb87e38..8818aa9 100644
+--- a/lib/vagrant-qemu/action.rb
++++ b/lib/vagrant-qemu/action.rb
+@@ -116,7 +116,7 @@ module VagrantPlugins
+             end
+ 
+             b1.use Provision
+-            b1.use EnvSet, port_collision_repair: false
++            b1.use EnvSet, port_collision_repair: true
+             b1.use PrepareForwardedPortCollisionParams
+             b1.use HandleForwardedPortCollisions
+             b1.use SyncedFolderCleanup
+diff --git a/lib/vagrant-qemu/action/prepare_forwarded_port_collision_params.rb b/lib/vagrant-qemu/action/prepare_forwarded_port_collision_params.rb
+index 159a785..b7c93a3 100644
+--- a/lib/vagrant-qemu/action/prepare_forwarded_port_collision_params.rb
++++ b/lib/vagrant-qemu/action/prepare_forwarded_port_collision_params.rb
+@@ -16,16 +16,23 @@ module VagrantPlugins
+       # Build the remap for any existing collision detections
+       remap = {}
+       env[:port_collision_remap] = remap
++
++      has_ssh_forward = false
+       machine.config.vm.networks.each do |type, options|
+         next if type != :forwarded_port
+ 
+         # remap ssh.host to ssh_port
+         if options[:id] == "ssh"
+           remap[options[:host]] = machine.provider_config.ssh_port
++          has_ssh_forward = true
+           break
+         end
+       end
+ 
++      if !has_ssh_forward
++        machine.config.vm.networks.forward_port(22, machine.provider_config.ssh_port, [id: "ssh", auto_correct: true])
++      end
++
+       @app.call(env)
+       end
+     end
+diff --git a/lib/vagrant-qemu/action/start_instance.rb b/lib/vagrant-qemu/action/start_instance.rb
+index 13c1d9d..b434bb3 100644
+--- a/lib/vagrant-qemu/action/start_instance.rb
++++ b/lib/vagrant-qemu/action/start_instance.rb
+@@ -11,6 +11,7 @@ module VagrantPlugins
+         end
+ 
+         def call(env)
++          fwPorts = forwarded_ports(env)
+           options = {
+             :ssh_host => env[:machine].provider_config.ssh_host,
+             :ssh_port => env[:machine].provider_config.ssh_port,
+@@ -23,7 +24,7 @@ module VagrantPlugins
+             :drive_interface => env[:machine].provider_config.drive_interface,
+             :extra_qemu_args => env[:machine].provider_config.extra_qemu_args,
+             :extra_netdev_args => env[:machine].provider_config.extra_netdev_args,
+-            :ports => forwarded_ports(env),
++            :ports => fwPorts,
+             :control_port => env[:machine].provider_config.control_port,
+             :debug_port => env[:machine].provider_config.debug_port,
+             :no_daemonize => env[:machine].provider_config.no_daemonize,
+@@ -43,7 +44,12 @@ module VagrantPlugins
+             next if type != :forwarded_port
+ 
+             # Don't include SSH
+-            next if options[:id] == "ssh"
++            if options[:id] == "ssh"
++              if options[:host] != env[:machine].provider_config.ssh_port
++                  env[:machine].provider_config.ssh_port = options[:host]
++              end
++              next
++            end
+ 
+             # Skip port if it is disabled
+             next if options[:disabled]

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: vagrant-qemu
 version: !ruby/object:Gem::Version
-  version: 0.3.6
+  version: 0.3.8
 platform: ruby
 authors:
 - ppggff
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-02-27 00:00:00.000000000 Z
+date: 2025-02-21 00:00:00.000000000 Z
 dependencies: []
 description: Enables Vagrant to manage machines with QEMU.
 email: pgf00a@gmail.com
@@ -43,6 +43,7 @@ files:
 - lib/vagrant-qemu/version.rb
 - locales/en.yml
 - vagrant-qemu.gemspec
+- x
 homepage: https://github.com/ppggff/vagrant-qemu
 licenses:
 - MIT