vagrant-libvirt 0.10.6 → 0.10.8

data/lib/vagrant-libvirt/cap/synced_folder_9p.rb

@@ -7,7 +7,9 @@ require 'digest/md5'
 
 require 'vagrant/util/subprocess'
 require 'vagrant/errors'
+
 require 'vagrant-libvirt/errors'
+require 'vagrant-libvirt/util/erb_template'
 
 module VagrantPlugins
   module SyncedFolder9P

data/lib/vagrant-libvirt/cap/synced_folder_virtiofs.rb

@@ -5,9 +5,11 @@ require 'ostruct'
 require 'nokogiri'
 require 'digest/md5'
 
-require 'vagrant/util/subprocess'
 require 'vagrant/errors'
+require 'vagrant/util/subprocess'
+
 require 'vagrant-libvirt/errors'
+require 'vagrant-libvirt/util/erb_template'
 
 module VagrantPlugins
   module SyncedFolderVirtioFS

data/lib/vagrant-libvirt/config.rb

@@ -896,8 +896,7 @@ module VagrantPlugins
         @initrd = '' if @initrd == UNSET_VALUE
         @dtb = nil if @dtb == UNSET_VALUE
         @graphics_type = 'vnc' if @graphics_type == UNSET_VALUE
-        @graphics_autoport = 'yes' if @graphics_port == UNSET_VALUE
-        @graphics_autoport = 'no' if @graphics_port != UNSET_VALUE
+        @graphics_autoport = @graphics_port == UNSET_VALUE ? 'yes' : nil
         if (@graphics_type != 'vnc' && @graphics_type != 'spice') ||
            @graphics_passwd == UNSET_VALUE
           @graphics_passwd = nil

data/lib/vagrant-libvirt/driver.rb

@@ -234,12 +234,12 @@ module VagrantPlugins
           @logger.debug(response)
           addresses = JSON.parse(response)
         rescue StandardError => e
-          puts "Unable to receive IP via qemu agent: [#{e.message}]"
           @logger.debug("Unable to receive IP via qemu agent: [#{e.message}]")
         end
 
         unless addresses.nil?
           addresses['return'].each do |interface|
+            next unless interface.key?('hardware-address')
             next unless domain.mac.downcase == interface['hardware-address'].downcase
 
             @logger.debug("Found matching interface: [#{interface['name']}]")

data/lib/vagrant-libvirt/errors.rb

@@ -178,6 +178,10 @@ module VagrantPlugins
         error_key(:no_domain_error)
       end
 
+      class DomainStartError < VagrantLibvirtError
+        error_key(:domain_start_error)
+      end
+
       class AttachDeviceError < VagrantLibvirtError
         error_key(:attach_device_error)
       end

data/lib/vagrant-libvirt/util/compat.rb

@@ -11,7 +11,7 @@ module VagrantPlugins
       module Compat
         def self.action_hook_args(name, action)
           # handle different number of arguments for action_hook depending on vagrant version
-          if Gem::Version.new(Vagrant::VERSION) >= Gem::Version.new('2.2.11')
+          if Gem::Version.new(Vagrant::VERSION) >= Gem::Version.new('2.2.10')
             return name, action
           end
 

data/lib/vagrant-libvirt/util/network_util.rb

@@ -120,7 +120,7 @@ module VagrantPlugins
             logger.debug "In config found network type #{type} options #{original_options}"
             # Options can be specified in Vagrantfile in short format (:ip => ...),
             # or provider format # (:libvirt__network_name => ...).
-            # https://github.com/mitchellh/vagrant/blob/master/lib/vagrant/util/scoped_hash_override.rb
+            # https://github.com/mitchellh/vagrant/blob/main/lib/vagrant/util/scoped_hash_override.rb
             options = scoped_hash_override(original_options, :libvirt)
             # store type in options
             # use default values if not already set

data/lib/vagrant-libvirt/version

@@ -1 +1 @@
-0.10.6
+0.10.8

data/locales/en.yml

@@ -153,6 +153,8 @@ en:
         Error while downloading volume '%{volume_name}' from storage pool '%{pool_name}': %{error_message}
       no_domain_error: |-
         No domain found. %{error_message}
+      domain_start_error: |-
+        Failed to start domain: %{error_message}
       attach_device_error: |-
         Error while attaching new device to domain. %{error_message}
       detach_device_error: |-

data/spec/spec_helper.rb

@@ -38,13 +38,14 @@ rescue LoadError
   end
 end
 
+RSpec.configure do |config|
+  require 'tmpdir'
 
-require 'vagrant-libvirt'
-require 'vagrant-spec/unit'
-
-Dir[File.dirname(__FILE__) + '/support/**/*.rb'].each { |f| require f }
+  # set VAGRANT_HOME before any thing that requires vagrant is loaded to prevent
+  # the global plugin manager from trying to use the default VAGRANT_HOME.
+  temp_dir = Dir.mktmpdir("rspec-")
+  ENV['VAGRANT_HOME'] = temp_dir
 
-RSpec.configure do |config|
   # ensure that setting of LIBVIRT_DEFAULT_URI in the environment is not picked
   # up directly by tests, instead they must set as needed. Some build envs will
   # may have it set to 'qemu:///session'.
@@ -52,10 +53,22 @@ RSpec.configure do |config|
     ENV.delete('LIBVIRT_DEFAULT_URI')
   end
 
+  config.after(:suite) do
+    FileUtils.remove_entry temp_dir
+  end
+
   config.mock_with :rspec do |mocks|
     mocks.verify_partial_doubles = true
   end
 
   # don't run acceptance tests by default
   config.filter_run_excluding :acceptance => true
+
+  config.expect_with :rspec do |c|
+    c.max_formatted_output_length = 2000 if c.respond_to?("max_formatted_output_length=")
+  end
 end
+
+require 'vagrant-spec/unit'
+
+Dir[File.dirname(__FILE__) + '/support/**/*.rb'].each { |f| require f }

data/spec/unit/action/set_boot_order_spec/default.xml

@@ -0,0 +1,76 @@
+<domain type="qemu">
+  <name>vagrant-libvirt_default</name>
+  <uuid>881a931b-0110-4d10-81aa-47a1a19f5726</uuid>
+  <description>Source: /home/test/vagrant-libvirt/Vagrantfile</description>
+  <memory unit="KiB">2097152</memory>
+  <currentMemory unit="KiB">2097152</currentMemory>
+  <vcpu placement="static">2</vcpu>
+  <os>
+    <type arch="x86_64" machine="pc-i440fx-6.0">hvm</type>
+    <boot dev="hd"/>
+  </os>
+  <features>
+    <acpi/>
+    <apic/>
+    <pae/>
+  </features>
+  <cpu mode="host-model" check="partial"/>
+  <clock offset="utc"/>
+  <on_poweroff>destroy</on_poweroff>
+  <on_reboot>restart</on_reboot>
+  <on_crash>destroy</on_crash>
+  <devices>
+    <emulator>/usr/bin/qemu-system-x86_64</emulator>
+    <disk type="file" device="disk">
+      <driver name="qemu" type="qcow2"/>
+      <source file="/var/lib/libvirt/images/vagrant-libvirt_default.img"/>
+      <target dev="vda" bus="virtio"/>
+      <address type="pci" domain="0x0000" bus="0x00" slot="0x03" function="0x0"/>
+    </disk>
+    <disk type="file" device="cdrom">
+      <driver name="qemu" type="raw"/>
+      <source file=""/>
+      <target dev="hda" bus="ide"/>
+      <readonly/>
+      <address type="pci" domain="0x0000" bus="0x00" slot="0x06" function="0x0"/>
+    </disk>
+    <disk type="file" device="cdrom">
+      <driver name="qemu" type="raw"/>
+      <source file=""/>
+      <target dev="hdb" bus="ide"/>
+      <readonly/>
+      <address type="pci" domain="0x0000" bus="0x00" slot="0x07" function="0x0"/>
+    </disk>
+    <controller type="usb" index="0" model="piix3-uhci">
+      <address type="pci" domain="0x0000" bus="0x00" slot="0x01" function="0x2"/>
+    </controller>
+    <controller type="pci" index="0" model="pci-root"/>
+    <interface type="network">
+      <mac address="52:54:00:7d:14:0e"/>
+      <source network="vagrant-libvirt"/>
+      <model type="virtio"/>
+      <address type="pci" domain="0x0000" bus="0x00" slot="0x05" function="0x0"/>
+    </interface>
+    <serial type="pty">
+      <target type="isa-serial" port="0">
+        <model name="isa-serial"/>
+      </target>
+    </serial>
+    <console type="pty">
+      <target type="serial" port="0"/>
+    </console>
+    <input type="mouse" bus="ps2"/>
+    <input type="keyboard" bus="ps2"/>
+    <graphics type="vnc" port="-1" autoport="yes" listen="127.0.0.1" keymap="en-us">
+      <listen type="address" address="127.0.0.1"/>
+    </graphics>
+    <audio id="1" type="none"/>
+    <video>
+      <model type="cirrus" vram="16384" heads="1" primary="yes"/>
+      <address type="pci" domain="0x0000" bus="0x00" slot="0x02" function="0x0"/>
+    </video>
+    <memballoon model="virtio">
+      <address type="pci" domain="0x0000" bus="0x00" slot="0x04" function="0x0"/>
+    </memballoon>
+  </devices>
+</domain>

data/spec/unit/action/set_boot_order_spec/explicit_boot_order.xml

@@ -0,0 +1,77 @@
+<?xml version="1.0"?>
+<domain type="qemu">
+  <name>vagrant-libvirt_default</name>
+  <uuid>881a931b-0110-4d10-81aa-47a1a19f5726</uuid>
+  <description>Source: /home/test/vagrant-libvirt/Vagrantfile</description>
+  <memory unit="KiB">2097152</memory>
+  <currentMemory unit="KiB">2097152</currentMemory>
+  <vcpu placement="static">2</vcpu>
+  <os>
+    <type arch="x86_64" machine="pc-i440fx-6.0">hvm</type>
+    
+  </os>
+  <features>
+    <acpi/>
+    <apic/>
+    <pae/>
+  </features>
+  <cpu mode="host-model" check="partial"/>
+  <clock offset="utc"/>
+  <on_poweroff>destroy</on_poweroff>
+  <on_reboot>restart</on_reboot>
+  <on_crash>destroy</on_crash>
+  <devices>
+    <emulator>/usr/bin/qemu-system-x86_64</emulator>
+    <disk type="file" device="disk">
+      <driver name="qemu" type="qcow2"/>
+      <source file="/var/lib/libvirt/images/vagrant-libvirt_default.img"/>
+      <target dev="vda" bus="virtio"/>
+      <address type="pci" domain="0x0000" bus="0x00" slot="0x03" function="0x0"/>
+    <boot order="1"/></disk>
+    <disk type="file" device="cdrom">
+      <driver name="qemu" type="raw"/>
+      <source file=""/>
+      <target dev="hda" bus="ide"/>
+      <readonly/>
+      <address type="pci" domain="0x0000" bus="0x00" slot="0x06" function="0x0"/>
+    <boot order="2"/></disk>
+    <disk type="file" device="cdrom">
+      <driver name="qemu" type="raw"/>
+      <source file=""/>
+      <target dev="hdb" bus="ide"/>
+      <readonly/>
+      <address type="pci" domain="0x0000" bus="0x00" slot="0x07" function="0x0"/>
+    <boot order="3"/></disk>
+    <controller type="usb" index="0" model="piix3-uhci">
+      <address type="pci" domain="0x0000" bus="0x00" slot="0x01" function="0x2"/>
+    </controller>
+    <controller type="pci" index="0" model="pci-root"/>
+    <interface type="network">
+      <mac address="52:54:00:7d:14:0e"/>
+      <source network="vagrant-libvirt"/>
+      <model type="virtio"/>
+      <address type="pci" domain="0x0000" bus="0x00" slot="0x05" function="0x0"/>
+    <boot order="4"/></interface>
+    <serial type="pty">
+      <target type="isa-serial" port="0">
+        <model name="isa-serial"/>
+      </target>
+    </serial>
+    <console type="pty">
+      <target type="serial" port="0"/>
+    </console>
+    <input type="mouse" bus="ps2"/>
+    <input type="keyboard" bus="ps2"/>
+    <graphics type="vnc" port="-1" autoport="yes" listen="127.0.0.1" keymap="en-us">
+      <listen type="address" address="127.0.0.1"/>
+    </graphics>
+    <audio id="1" type="none"/>
+    <video>
+      <model type="cirrus" vram="16384" heads="1" primary="yes"/>
+      <address type="pci" domain="0x0000" bus="0x00" slot="0x02" function="0x0"/>
+    </video>
+    <memballoon model="virtio">
+      <address type="pci" domain="0x0000" bus="0x00" slot="0x04" function="0x0"/>
+    </memballoon>
+  </devices>
+</domain>

data/spec/unit/action/set_boot_order_spec.rb

@@ -0,0 +1,67 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'support/sharedcontext'
+require 'support/libvirt_context'
+
+require 'vagrant-libvirt/action/set_boot_order'
+
+describe VagrantPlugins::ProviderLibvirt::Action::SetBootOrder do
+  subject { described_class.new(app, env) }
+
+  include_context 'unit'
+  include_context 'libvirt'
+
+  #before do
+  #  allow(driver).to receive(:created?).and_return(true)
+  #end
+
+  describe '#call' do
+    it 'should return early' do
+      expect(connection).to_not receive(:client)
+
+      expect(subject.call(env)).to be_nil
+    end
+
+    context 'with boot_order defined' do
+      let(:domain_xml) { File.read(File.join(File.dirname(__FILE__), File.basename(__FILE__, '.rb'), test_file)) }
+      let(:updated_domain_xml) { File.read(File.join(File.dirname(__FILE__), File.basename(__FILE__, '.rb'), updated_test_file)) }
+      let(:test_file) { 'default.xml' }
+      let(:updated_test_file) { 'explicit_boot_order.xml' }
+      let(:vagrantfile_providerconfig) do
+        <<-EOF
+        libvirt.boot "hd"
+        libvirt.boot "cdrom"
+        libvirt.boot "network" => 'vagrant-libvirt'
+        EOF
+      end
+
+      before do
+        allow(connection).to receive(:client).and_return(libvirt_client)
+        allow(libvirt_client).to receive(:lookup_domain_by_uuid).and_return(libvirt_domain)
+        allow(libvirt_domain).to receive(:xml_desc).and_return(domain_xml)
+        allow(logger).to receive(:debug)
+      end
+
+      it 'should configure the boot order' do
+        expect(libvirt_client).to receive(:define_domain_xml).with(updated_domain_xml)
+        expect(subject.call(env)).to be_nil
+      end
+
+      context 'with multiple networks in bootorder' do
+        let(:vagrantfile_providerconfig) do
+          <<-EOF
+          libvirt.boot "hd"
+          libvirt.boot "cdrom"
+          libvirt.boot "network" => 'vagrant-libvirt'
+          libvirt.boot "network" => 'vagrant-libvirt'
+          EOF
+        end
+
+        it 'should raise an exception' do
+          expect { subject.call(env) }.to raise_error('Defined only for 1 network for boot')
+        end
+      end
+    end
+  end
+end

data/spec/unit/action/start_domain_spec/existing_reordered.xml

@@ -0,0 +1,62 @@
+<domain type='qemu'>
+  <name>vagrant-libvirt_default</name>
+  <uuid>881a931b-0110-4d10-81aa-47a1a19f5726</uuid>
+  <description>Source: /home/test/vagrant-libvirt/Vagrantfile</description>
+  <memory unit='KiB'>2097152</memory>
+  <currentMemory unit='KiB'>2097152</currentMemory>
+  <vcpu placement='static'>2</vcpu>
+  <os>
+    <type arch='x86_64' machine='pc-i440fx-6.0'>hvm</type>
+    <boot dev='hd'/>
+  </os>
+  <features>
+    <acpi/>
+    <apic/>
+    <pae/>
+  </features>
+  <cpu check='partial' mode='host-passthrough'/>
+  <clock offset='utc'/>
+  <on_poweroff>destroy</on_poweroff>
+  <on_reboot>restart</on_reboot>
+  <on_crash>destroy</on_crash>
+  <devices>
+    <emulator>/usr/bin/qemu-system-x86_64</emulator>
+    <disk device='disk' type='file'>
+      <driver name='qemu' type='qcow2'/>
+      <source file='/var/lib/libvirt/images/vagrant-libvirt_default.img'/>
+      <target bus='virtio' dev='vda'/>
+      <address bus='0x00' domain='0x0000' function='0x0' slot='0x03' type='pci'/>
+    </disk>
+    <controller index='0' model='pci-root' type='pci'/>
+    <controller index='0' model='piix3-uhci' type='usb'>
+      <address bus='0x00' domain='0x0000' function='0x2' slot='0x01' type='pci'/>
+    </controller>
+    <interface type='network'>
+      <mac address='52:54:00:7d:14:0e'/>
+      <source network='vagrant-libvirt'/>
+      <model type='virtio'/>
+      <address bus='0x00' domain='0x0000' function='0x0' slot='0x05' type='pci'/>
+    </interface>
+    <serial type='pty'>
+      <target port='0' type='isa-serial'>
+        <model name='isa-serial'/>
+      </target>
+    </serial>
+    <console type='pty'>
+      <target port='0' type='serial'/>
+    </console>
+    <input bus='ps2' type='mouse'/>
+    <input bus='ps2' type='keyboard'/>
+    <graphics autoport='yes' keymap='en-us' listen='127.0.0.1' port='-1' type='vnc'>
+      <listen address='127.0.0.1' type='address'/>
+    </graphics>
+    <audio id='1' type='none'/>
+    <video>
+      <model heads='1' primary='yes' type='cirrus' vram='16384'/>
+      <address bus='0x00' domain='0x0000' function='0x0' slot='0x02' type='pci'/>
+    </video>
+    <memballoon model='virtio'>
+      <address bus='0x00' domain='0x0000' function='0x0' slot='0x04' type='pci'/>
+    </memballoon>
+  </devices>
+</domain>

data/spec/unit/action/start_domain_spec.rb

@@ -42,7 +42,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::StartDomain do
     end
 
     it 'should execute without changing' do
-      expect(ui).to_not receive(:error)
+      expect(ui).to_not receive(:warn)
       expect(libvirt_client).to_not receive(:define_domain_xml)
       expect(libvirt_domain).to receive(:autostart=)
       expect(domain).to receive(:start)
@@ -64,7 +64,27 @@ describe VagrantPlugins::ProviderLibvirt::Action::StartDomain do
       end
 
       it 'should correctly detect the domain was updated' do
-        expect(ui).to_not receive(:error)
+        expect(ui).to_not receive(:warn)
+        expect(libvirt_domain).to receive(:autostart=)
+        expect(connection).to receive(:define_domain).and_return(libvirt_domain)
+        expect(libvirt_domain).to receive(:xml_desc).and_return(domain_xml, updated_domain_xml)
+        expect(domain).to receive(:start)
+
+        expect(subject.call(env)).to be_nil
+      end
+    end
+
+    context 'when xml elements and attributes reordered' do
+      let(:test_file) { 'existing.xml' }
+      let(:updated_test_file) { 'existing_reordered.xml' }
+      let(:vagrantfile_providerconfig) do
+        <<-EOF
+        libvirt.cpu_mode = "host-passthrough"
+        EOF
+      end
+
+      it 'should correctly detect the domain was updated' do
+        expect(ui).to_not receive(:warn)
         expect(libvirt_domain).to receive(:autostart=)
         expect(connection).to receive(:define_domain).and_return(libvirt_domain)
         expect(libvirt_domain).to receive(:xml_desc).and_return(domain_xml, updated_domain_xml)
@@ -88,13 +108,16 @@ describe VagrantPlugins::ProviderLibvirt::Action::StartDomain do
       end
 
       it 'should error and revert the update' do
-        expect(ui).to receive(:error)
+        expect(ui).to receive(:warn).with(/\+  <cpu mode="host-passthrough" \/>.*Typically this means there is a bug in the XML being sent, please log an issue/m)
         expect(connection).to receive(:define_domain).and_return(libvirt_domain)
-        expect(connection).to receive(:define_domain).with(domain_xml) # undo
+        #expect(connection).to receive(:define_domain).with(domain_xml) # undo
         expect(libvirt_domain).to receive(:xml_desc).and_return(domain_xml, updated_domain_xml)
-        expect(domain).to_not receive(:start)
+        #expect(domain).to_not receive(:start)
 
-        expect { subject.call(env) }.to raise_error(VagrantPlugins::ProviderLibvirt::Errors::FogError)
+        #expect { subject.call(env) }.to raise_error(VagrantPlugins::ProviderLibvirt::Errors::UpdateServerError)
+        expect(libvirt_domain).to receive(:autostart=)
+        expect(domain).to receive(:start)
+        expect(subject.call(env)).to be_nil
       end
     end
 
@@ -112,7 +135,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::StartDomain do
       }
 
       it 'should update the domain' do
-        expect(ui).to_not receive(:error)
+        expect(ui).to_not receive(:warn)
         expect(libvirt_domain).to receive(:autostart=)
         expect(connection).to receive(:define_domain).and_return(libvirt_domain)
         expect(libvirt_domain).to receive(:xml_desc).and_return(domain_xml, updated_domain_xml)
@@ -151,7 +174,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::StartDomain do
         let(:updated_test_file) { 'existing_added_nvram.xml' }
 
         it 'should add the nvram element' do
-          expect(ui).to_not receive(:error)
+          expect(ui).to_not receive(:warn)
           expect(connection).to receive(:define_domain).with(updated_domain_xml).and_return(libvirt_domain)
           expect(libvirt_domain).to receive(:xml_desc).and_return(domain_xml, updated_domain_xml)
           expect(libvirt_domain).to receive(:autostart=)
@@ -172,7 +195,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::StartDomain do
         let(:updated_test_file) { 'nvram_domain_other_setting.xml' }
 
         it 'should keep the XML element' do
-          expect(ui).to_not receive(:error)
+          expect(ui).to_not receive(:warn)
           expect(connection).to receive(:define_domain).with(updated_domain_xml).and_return(libvirt_domain)
           expect(libvirt_domain).to receive(:xml_desc).and_return(domain_xml, updated_domain_xml)
           expect(libvirt_domain).to receive(:autostart=)
@@ -186,7 +209,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::StartDomain do
           let(:updated_test_file) { 'nvram_domain_removed.xml' }
 
           it 'should delete the XML element' do
-            expect(ui).to_not receive(:error)
+            expect(ui).to_not receive(:warn)
             expect(connection).to receive(:define_domain).with(updated_domain_xml).and_return(libvirt_domain)
             expect(libvirt_domain).to receive(:xml_desc).and_return(domain_xml, updated_domain_xml)
             expect(libvirt_domain).to receive(:autostart=)
@@ -210,7 +233,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::StartDomain do
         end
 
         it 'should modify the domain tpm_path' do
-          expect(ui).to_not receive(:error)
+          expect(ui).to_not receive(:warn)
           expect(logger).to receive(:debug).with('tpm config changed')
           expect(connection).to receive(:define_domain).with(updated_domain_xml).and_return(libvirt_domain)
           expect(libvirt_domain).to receive(:xml_desc).and_return(domain_xml, updated_domain_xml)
@@ -232,7 +255,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::StartDomain do
         end
 
         it 'should modify the domain tpm_path' do
-          expect(ui).to_not receive(:error)
+          expect(ui).to_not receive(:warn)
           expect(logger).to receive(:debug).with('tpm config changed')
           expect(connection).to receive(:define_domain).with(updated_domain_xml).and_return(libvirt_domain)
           expect(libvirt_domain).to receive(:xml_desc).and_return(domain_xml, updated_domain_xml)
@@ -255,7 +278,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::StartDomain do
         end
 
         it 'should execute without changing' do
-          expect(ui).to_not receive(:error)
+          expect(ui).to_not receive(:warn)
           expect(libvirt_domain).to receive(:autostart=)
           expect(domain).to receive(:start)
 
@@ -275,7 +298,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::StartDomain do
         end
 
         it 'should execute without changing' do
-          expect(ui).to_not receive(:error)
+          expect(ui).to_not receive(:warn)
           expect(libvirt_domain).to receive(:autostart=)
           expect(domain).to receive(:start)
 
@@ -295,7 +318,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::StartDomain do
         end
 
         it 'should modify the domain' do
-          expect(ui).to_not receive(:error)
+          expect(ui).to_not receive(:warn)
           expect(logger).to receive(:debug).with('tpm config changed')
           expect(connection).to receive(:define_domain).with(updated_domain_xml).and_return(libvirt_domain)
           expect(libvirt_domain).to receive(:xml_desc).and_return(domain_xml, updated_domain_xml)
@@ -318,7 +341,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::StartDomain do
         end
 
         it 'should not modify the domain' do
-          expect(ui).to_not receive(:error)
+          expect(ui).to_not receive(:warn)
           expect(logger).to_not receive(:debug).with('clock timers config changed')
           expect(connection).to_not receive(:define_domain)
           expect(libvirt_domain).to receive(:autostart=)
@@ -339,7 +362,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::StartDomain do
         let(:updated_test_file) { 'clock_timer_rtc_tsc.xml' }
 
         it 'should modify the domain' do
-          expect(ui).to_not receive(:error)
+          expect(ui).to_not receive(:warn)
           expect(logger).to receive(:debug).with('clock timers config changed')
           expect(connection).to receive(:define_domain).with(match(/<clock offset='utc'>\s*<timer name='rtc'\/>\s*<timer name='tsc'\/>\s*<\/clock>/)).and_return(libvirt_domain)
           expect(libvirt_domain).to receive(:xml_desc).and_return(domain_xml, updated_domain_xml)
@@ -354,7 +377,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::StartDomain do
         let(:updated_test_file) { 'clock_timer_removed.xml' }
 
         it 'should modify the domain' do
-          expect(ui).to_not receive(:error)
+          expect(ui).to_not receive(:warn)
           expect(logger).to receive(:debug).with('clock timers config changed')
           expect(connection).to receive(:define_domain).with(match(/<clock offset='utc'>\s*<\/clock>/)).and_return(libvirt_domain)
           expect(libvirt_domain).to receive(:xml_desc).and_return(domain_xml, updated_domain_xml)

data/spec/unit/plugin_spec.rb

@@ -3,6 +3,7 @@
 require 'spec_helper'
 require 'support/sharedcontext'
 
+require 'vagrant-libvirt'
 require 'vagrant-libvirt/plugin'