looks good. anyway require more tests.

lib/vagrant-libvirt/action.rb

@@ -31,20 +31,18 @@ module VagrantPlugins
               b2.use CreateDomain
 
               b2.use Provision
-              b2.use CreateNetworks
-              b2.use CreateNetworkInterfaces
-
               b2.use PrepareNFSValidIds
               b2.use SyncedFolderCleanup
               b2.use SyncedFolders
+              b2.use PrepareNFSSettings
+              b2.use ShareFolders
+              b2.use CreateNetworks
+              b2.use CreateNetworkInterfaces
 
               b2.use StartDomain
               b2.use WaitTillUp
 
               b2.use ForwardPorts
-
-              b2.use PrepareNFSSettings
-              b2.use ShareFolders
               b2.use SetHostname
               # b2.use SyncFolders
             else

lib/vagrant-libvirt/action/prepare_nfs_settings.rb

@@ -1,10 +1,13 @@
 require 'nokogiri'
+require 'socket'
+require 'timeout'
+
 module VagrantPlugins
   module ProviderLibvirt
     module Action
       class PrepareNFSSettings
         include Vagrant::Action::Builtin::MixinSyncedFolders
-        
+
         def initialize(app,env)
           @app = app
           @logger = Log4r::Logger.new("vagrant::action::vm::nfs")
@@ -16,8 +19,8 @@ module VagrantPlugins
 
           if using_nfs?
             @logger.info("Using NFS, preparing NFS settings by reading host IP and machine IP")
-            env[:nfs_host_ip]    = read_host_ip(env[:machine],env)
+            env[:nfs_host_ip]    = read_host_ip(env[:machine])
-            env[:nfs_machine_ip] = env[:machine].ssh_info[:host]
+            env[:nfs_machine_ip] = read_machine_ip(env[:machine])
 
             @logger.info("host IP: #{env[:nfs_host_ip]} machine IP: #{env[:nfs_machine_ip]}")
 
@@ -32,37 +35,54 @@ module VagrantPlugins
           !!synced_folders(@machine)[:nfs]
         end
 
-        # Returns the IP address of the first host only network adapter
+        # Returns the IP address of the host
         #
         # @param [Machine] machine
         # @return [String]
-        def read_host_ip(machine,env)
+        def read_host_ip(machine)
-          nets = env[:libvirt_compute].list_networks
+          UDPSocket.open do |s|
-          if nets.size == 1
+            s.connect(machine.ssh_info[:host], 1)
-            net = nets.first
+            s.addr.last
-          else
-            domain = env[:libvirt_compute].servers.get(machine.id.to_s)
-            xml=Nokogiri::XML(domain.to_xml)
-            networkname = xml.xpath('/domain/devices/interface/source').first.attributes['network'].value.to_s
-            @logger.info("Using network named #{networkname}")
-            net = env[:libvirt_compute].list_networks.find {|netw| netw[:name] == networkname}
           end
-          # FIXME better implement by libvirt xml parsing
-          `ip addr show | grep -A 2 #{net[:bridge_name]} | grep -i 'inet ' | tr -s ' ' | cut -d' ' -f3 | cut -d'/' -f 1`.chomp
         end
 
-        # Returns the IP address of the guest by looking at the first
+        # Returns the IP address of the guest
-        # enabled host only network.
         #
+        # @param [Machine] machine
         # @return [String]
         def read_machine_ip(machine)
-          machine.config.vm.networks.each do |type, options|
+          # check host only ip
-            if type == :private_network && options[:ip].is_a?(String)
+          ssh_host = machine.ssh_info[:host]
-              return options[:ip]
+          return ssh_host if ping(ssh_host)
-            end
+
+          # check other ips
+          command = "ip addr show | grep -i 'inet ' | grep -v '127.0.0.1' | tr -s ' ' | cut -d' ' -f3 | cut -d'/' -f 1"
+          result  = ""
+          machine.communicate.execute(command) do |type, data|
+            result << data if type == :stdout
           end
 
-          nil
+          ips = result.chomp.split("\n")
+          @logger.info("guest IPs: #{ips.join(', ')}")
+          ips.each do |ip|
+            next if ip == ssh_host
+            return ip if ping(ip)
+          end
+        end
+
+        private
+
+        # Check if we can open a connection to the host
+        def ping(host, timeout = 3)
+          timeout(timeout) do
+            s = TCPSocket.new(host, 'echo')
+            s.close
+          end
+          true
+        rescue Errno::ECONNREFUSED
+          true
+        rescue Timeout::Error, StandardError
+          false
         end
       end
     end

lib/vagrant-libvirt/action/read_ssh_info.rb

@@ -48,8 +48,8 @@ module VagrantPlugins
             :forward_agent => machine.config.ssh.forward_agent,
             :forward_x11   => machine.config.ssh.forward_x11,
           }
-          
+
-          ssh_info[:proxy_command] = "ssh '#{machine.provider_config.host}' -l '#{machine.provider_config.username}' nc %h %p" if machine.provider_config.connect_via_ssh
+          ssh_info[:proxy_command] = "ssh '#{machine.provider_config.host}' -l '#{machine.provider_config.username}' -i '#{machine.provider_config.id_ssh_key_file}' nc %h %p" if machine.provider_config.connect_via_ssh
 
           ssh_info
         end

lib/vagrant-libvirt/config.rb

@@ -248,7 +248,7 @@ module VagrantPlugins
           # set ssh key for access to libvirt host
           uri << "\&keyfile="
           # if no slash, prepend $HOME/.ssh/
-          uri << "#{`echo ${HOME}`.chomp}/.ssh/" if @id_ssh_key_file !~ /\A\//
+          @id_ssh_key_file.prepend("#{`echo ${HOME}`.chomp}/.ssh/") if @id_ssh_key_file !~ /\A\//
           uri << @id_ssh_key_file
         end
         # set path to libvirt socket