vagrant-libvirt 0.9.0 → 0.10.0

data/spec/unit/action_spec.rb

@@ -170,4 +170,92 @@ describe VagrantPlugins::ProviderLibvirt::Action do
       end
     end
   end
+
+  describe '#action_snapshot_delete' do
+    context 'when not created' do
+      before do
+        allow_action_env_result(VagrantPlugins::ProviderLibvirt::Action::IsCreated, false)
+      end
+
+      it 'should cause an error' do
+        expect{ machine.action(:snapshot_delete, snapshot_opts: {})}.to raise_error(Vagrant::Errors::VMNotCreatedError)
+      end
+    end
+
+    context 'when created' do
+      before do
+        allow_action_env_result(VagrantPlugins::ProviderLibvirt::Action::IsCreated, true)
+      end
+
+      context 'when running' do
+        before do
+          allow_action_env_result(VagrantPlugins::ProviderLibvirt::Action::IsRunning, true)
+        end
+
+        it 'should call SnapshotDelete' do
+          expect_any_instance_of(VagrantPlugins::ProviderLibvirt::Action::SnapshotDelete).to receive(:call).and_return(0)
+          expect(machine.action(:snapshot_delete, snapshot_opts: {})).to match(hash_including({:action_name => :machine_action_snapshot_delete}))
+        end
+      end
+    end
+  end
+
+
+  describe '#action_snapshot_restore' do
+    context 'when not created' do
+      before do
+        allow_action_env_result(VagrantPlugins::ProviderLibvirt::Action::IsCreated, false)
+      end
+
+      it 'should cause an error' do
+        expect{ machine.action(:snapshot_restore, snapshot_opts: {})}.to raise_error(Vagrant::Errors::VMNotCreatedError)
+      end
+    end
+
+    context 'when created' do
+      before do
+        allow_action_env_result(VagrantPlugins::ProviderLibvirt::Action::IsCreated, true)
+      end
+
+      context 'when running' do
+        before do
+          allow_action_env_result(VagrantPlugins::ProviderLibvirt::Action::IsRunning, true)
+        end
+
+        it 'should call SnapshotRestore' do
+          expect_any_instance_of(VagrantPlugins::ProviderLibvirt::Action::SnapshotRestore).to receive(:call).and_return(0)
+          expect(machine.action(:snapshot_restore, snapshot_opts: {})).to match(hash_including({:action_name => :machine_action_snapshot_restore}))
+        end
+      end
+    end
+  end
+
+  describe '#action_snapshot_save' do
+    context 'when not created' do
+      before do
+        allow_action_env_result(VagrantPlugins::ProviderLibvirt::Action::IsCreated, false)
+      end
+
+      it 'should cause an error' do
+        expect{ machine.action(:snapshot_save, snapshot_opts: {})}.to raise_error(Vagrant::Errors::VMNotCreatedError)
+      end
+    end
+
+    context 'when created' do
+      before do
+        allow_action_env_result(VagrantPlugins::ProviderLibvirt::Action::IsCreated, true)
+      end
+
+      context 'when running' do
+        before do
+          allow_action_env_result(VagrantPlugins::ProviderLibvirt::Action::IsRunning, true)
+        end
+
+        it 'should call SnapshotSave' do
+          expect_any_instance_of(VagrantPlugins::ProviderLibvirt::Action::SnapshotSave).to receive(:call).and_return(0)
+          expect(machine.action(:snapshot_save, snapshot_opts: {})).to match(hash_including({:action_name => :machine_action_snapshot_save}))
+        end
+      end
+    end
+  end
 end

data/spec/unit/cap/synced_folder_9p_spec.rb

@@ -0,0 +1,120 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'support/sharedcontext'
+
+require 'vagrant-libvirt/cap/synced_folder_9p'
+require 'vagrant-libvirt/util/unindent'
+
+describe VagrantPlugins::SyncedFolder9P::SyncedFolder do
+  include_context 'unit'
+  include_context 'libvirt'
+
+  subject { described_class.new }
+
+  describe '#prepare' do
+    let(:synced_folders) { {
+      "/vagrant" => {
+        :hostpath => '/home/test/default',
+        :disabled=>false,
+        :guestpath=>'/vagrant',
+        :type => :"9p",
+      },
+    } }
+
+    let(:expected_xml) {
+      <<-EOF.unindent
+      <filesystem type="mount" accessmode="passthrough">
+        <driver type="path" wrpolicy="immediate"></driver>
+        <source dir="/home/test/default"></source>
+        <target dir="1ef53e1c9b6a5a0c695c80665331652\"></target>
+      </filesystem>
+      EOF
+    }
+
+    before do
+      allow(libvirt_client).to receive(:lookup_domain_by_uuid).and_return(libvirt_domain)
+      allow(logger).to receive(:debug)
+    end
+
+    it 'should attach device xml' do
+      expect(libvirt_domain).to receive(:attach_device).with(expected_xml, 0)
+
+      subject.prepare(machine, synced_folders, {})
+    end
+
+    context 'multiple folders' do
+      let(:synced_folders) { {
+        "/vagrant" => {
+          :hostpath => '/home/test/default',
+          :disabled=>false,
+          :guestpath=>'/vagrant',
+          :type => :"9p",
+        },
+        "/custom" => {
+          :hostpath => '/home/test/custom',
+          :disabled=>false,
+          :guestpath=>'/custom',
+          :type => :"9p",
+        },
+      } }
+
+      let(:expected_xml) {
+        [ <<-XML1.unindent, <<-XML2.unindent ]
+        <filesystem type="mount" accessmode="passthrough">
+          <driver type="path" wrpolicy="immediate"></driver>
+          <source dir="/home/test/default"></source>
+          <target dir="1ef53e1c9b6a5a0c695c80665331652\"></target>
+        </filesystem>
+        XML1
+        <filesystem type="mount" accessmode="passthrough">
+          <driver type="path" wrpolicy="immediate"></driver>
+          <source dir="/home/test/custom"></source>
+          <target dir=\"a2a1a8b6d98be8f790f3c987e006d13\"></target>
+        </filesystem>
+        XML2
+      }
+
+      it 'should attach all device xml' do
+        expect(libvirt_domain).to receive(:attach_device).with(expected_xml[0], 0)
+        expect(libvirt_domain).to receive(:attach_device).with(expected_xml[1], 0)
+        expect(ui).to receive(:info).with(/Configuring 9p devices for shared folders/).once
+
+        subject.prepare(machine, synced_folders, {})
+      end
+    end
+  end
+
+  describe '#usable?' do
+    context 'with libvirt provider' do
+      before do
+        allow(machine).to receive(:provider_name).and_return(:libvirt)
+        allow(libvirt_client).to receive(:libversion).and_return(8002000)
+      end
+
+      it 'should be' do
+        expect(subject).to be_usable(machine)
+      end
+
+      context 'with version less than 1.2.2' do
+        before do
+          allow(libvirt_client).to receive(:libversion).and_return(1002001)
+        end
+
+        it 'should not be' do
+          expect(subject).to_not be_usable(machine)
+        end
+      end
+    end
+
+    context 'with other provider' do
+      before do
+        allow(machine).to receive(:provider_name).and_return(:virtualbox)
+      end
+
+      it 'should not be' do
+        expect(subject).to_not be_usable(machine)
+      end
+    end
+  end
+end

data/spec/unit/cap/synced_folder_virtiofs_spec.rb

@@ -0,0 +1,120 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'support/sharedcontext'
+
+require 'vagrant-libvirt/cap/synced_folder_virtiofs'
+require 'vagrant-libvirt/util/unindent'
+
+describe VagrantPlugins::SyncedFolderVirtioFS::SyncedFolder do
+  include_context 'unit'
+  include_context 'libvirt'
+
+  subject { described_class.new }
+
+  describe '#prepare' do
+    let(:synced_folders) { {
+      "/vagrant" => {
+        :hostpath => '/home/test/default',
+        :disabled=>false,
+        :guestpath=>'/vagrant',
+        :type => :virtiofs,
+      },
+    } }
+
+    let(:expected_xml) {
+      <<-EOF.unindent
+      <filesystem type="mount" accessmode="passthrough">
+        <driver type="virtiofs"></driver>
+        <source dir="/home/test/default"></source>
+        <target dir="1ef53e1c9b6a5a0c695c80665331652\"></target>
+      </filesystem>
+      EOF
+    }
+
+    before do
+      allow(libvirt_client).to receive(:lookup_domain_by_uuid).and_return(libvirt_domain)
+      allow(logger).to receive(:debug)
+    end
+
+    it 'should attach device xml' do
+      expect(libvirt_domain).to receive(:attach_device).with(expected_xml, 0)
+
+      subject.prepare(machine, synced_folders, {})
+    end
+
+    context 'multiple folders' do
+      let(:synced_folders) { {
+        "/vagrant" => {
+          :hostpath => '/home/test/default',
+          :disabled=>false,
+          :guestpath=>'/vagrant',
+          :type => :virtiofs,
+        },
+        "/custom" => {
+          :hostpath => '/home/test/custom',
+          :disabled=>false,
+          :guestpath=>'/custom',
+          :type => :virtiofs,
+        },
+      } }
+
+      let(:expected_xml) {
+        [ <<-XML1.unindent, <<-XML2.unindent ]
+        <filesystem type="mount" accessmode="passthrough">
+          <driver type="virtiofs"></driver>
+          <source dir="/home/test/default"></source>
+          <target dir="1ef53e1c9b6a5a0c695c80665331652\"></target>
+        </filesystem>
+        XML1
+        <filesystem type="mount" accessmode="passthrough">
+          <driver type="virtiofs"></driver>
+          <source dir="/home/test/custom"></source>
+          <target dir=\"a2a1a8b6d98be8f790f3c987e006d13\"></target>
+        </filesystem>
+        XML2
+      }
+
+      it 'should attach all device xml' do
+        expect(libvirt_domain).to receive(:attach_device).with(expected_xml[0], 0)
+        expect(libvirt_domain).to receive(:attach_device).with(expected_xml[1], 0)
+        expect(ui).to receive(:info).with(/Configuring virtiofs devices for shared folders/).once
+
+        subject.prepare(machine, synced_folders, {})
+      end
+    end
+  end
+
+  describe '#usable?' do
+    context 'with libvirt provider' do
+      before do
+        allow(machine).to receive(:provider_name).and_return(:libvirt)
+        allow(libvirt_client).to receive(:libversion).and_return(8002000)
+      end
+
+      it 'should be' do
+        expect(subject).to be_usable(machine)
+      end
+
+      context 'with version less than 6.2.0' do
+        before do
+          allow(libvirt_client).to receive(:libversion).and_return(6001000)
+        end
+
+        it 'should not be' do
+          expect(subject).to_not be_usable(machine)
+        end
+      end
+    end
+
+    context 'with other provider' do
+      before do
+        allow(machine).to receive(:provider_name).and_return(:virtualbox)
+      end
+
+      it 'should not be' do
+        expect(subject).to_not be_usable(machine)
+      end
+    end
+  end
+end

data/spec/unit/config_spec.rb

@@ -577,24 +577,28 @@ describe VagrantPlugins::ProviderLibvirt::Config do
 
   def assert_invalid
     subject.finalize!
-    errors = subject.validate(machine)
-    raise "No errors: #{errors.inspect}" if errors.values.all?(&:empty?)
+    errors = subject.validate(machine).values.first
+    expect(errors).to_not be_empty
+    errors
   end
 
   def assert_valid
     subject.finalize!
-    errors = subject.validate(machine)
-    raise "Errors: #{errors.inspect}" unless errors.values.all?(&:empty?)
+    errors = subject.validate(machine).values.first
+    expect(errors).to be_empty
   end
 
   describe '#validate' do
+    before do
+      allow(machine).to receive(:provider_config).and_return(subject)
+      allow(machine).to receive(:ui).and_return(ui)
+    end
+
     it 'is valid with defaults' do
       assert_valid
     end
 
     context 'with disks defined' do
-      before { expect(machine).to receive(:provider_config).and_return(subject).at_least(:once) }
-
       it 'is valid if relative path used for disk' do
         subject.storage :file, path: '../path/to/file.qcow2'
         assert_valid
@@ -630,6 +634,53 @@ describe VagrantPlugins::ProviderLibvirt::Config do
       end
     end
 
+    context 'with public_network defined' do
+      let(:host_devices) { [
+        instance_double(Socket::Ifaddr),
+        instance_double(Socket::Ifaddr),
+      ] }
+      before do
+        machine.config.vm.network :public_network, dev: 'eth0', ip: "192.168.2.157"
+        expect(Socket).to receive(:getifaddrs).and_return(host_devices)
+        expect(host_devices[0]).to receive(:name).and_return('eth0')
+        expect(host_devices[1]).to receive(:name).and_return('virbr0')
+      end
+
+      it 'should validate use of existing device' do
+        assert_valid
+      end
+
+      context 'when default device not on host' do
+        before do
+          machine.config.vm.network :public_network, dev: 'eno1', ip: "192.168.2.157"
+        end
+
+        it 'should be invalid' do
+          assert_invalid
+        end
+      end
+
+      context 'when using excluded host device' do
+        before do
+          machine.config.vm.network :public_network, dev: 'virbr0', ip: "192.168.2.157"
+        end
+
+        it 'should be invalid' do
+          assert_invalid
+        end
+
+        context 'when user overrides to allow device' do
+          before do
+            subject.host_device_exclude_prefixes = []
+          end
+
+          it 'should validate' do
+            assert_valid
+          end
+        end
+      end
+    end
+
     context 'with nvram defined' do
       before do
         subject.nvram = '/path/to/some/nvram'
@@ -662,6 +713,60 @@ describe VagrantPlugins::ProviderLibvirt::Config do
         assert_valid
       end
     end
+
+    context 'with sysinfo defined' do
+      context 'when invalid block name provided' do
+        it 'should be invalid' do
+          subject.sysinfo = {'bad bios': {'vendor': 'some vendor'}}
+
+          errors = assert_invalid
+          expect(errors).to include(match(/invalid sysinfo element 'bad bios';/))
+        end
+      end
+
+      context 'when invalid element name provided' do
+        it 'should be invalid' do
+          subject.sysinfo = {'bios': {'bad vendor': 'some vendor'}}
+
+          errors = assert_invalid
+          expect(errors).to include(match(/'sysinfo.bios' does not support entry name 'bad vendor'/))
+        end
+      end
+
+      context 'when empty element value provided' do
+        it 'should succeed with a warning' do
+          expect(ui).to receive(:warn).with(/Libvirt Provider: sysinfo.bios.vendor is nil or empty/)
+          subject.sysinfo = {'bios': {'vendor': ''}}
+
+          assert_valid
+        end
+      end
+
+      context 'when handling "oem strings"' do
+        it 'should succeed' do
+          subject.sysinfo = {'oem strings': ['string 1']}
+
+          assert_valid
+        end
+
+        context 'when empty entries' do
+          it 'should succeed with a warning' do
+            expect(ui).to receive(:warn).with(/Libvirt Provider: 'sysinfo.oem strings' contains an empty/)
+            subject.sysinfo = {'oem strings': ['']}
+
+            assert_valid
+          end
+        end
+
+        context 'when non string passed' do
+          it 'should be invalid' do
+            subject.sysinfo = {'oem strings': [true]}
+
+            assert_invalid
+          end
+        end
+      end
+    end
   end
 
   describe '#merge' do
@@ -725,5 +830,27 @@ describe VagrantPlugins::ProviderLibvirt::Config do
                                                 include(name: 'hpet'))
       end
     end
+
+    context 'sysinfo' do
+      it 'should merge' do
+        one.sysinfo = {
+          'bios' => {'vendor': 'Some Vendor'},
+          'system' => {'manufacturer': 'some manufacturer'},
+          'oem strings' => ['string 1'],
+        }
+        two.sysinfo = {
+          'bios' => {'vendor': 'Another Vendor'},
+          'system' => {'serial': 'AABBCCDDEE'},
+          'oem strings' => ['string 2'],
+        }
+
+        subject.finalize!
+        expect(subject.sysinfo).to eq(
+          'bios' => {'vendor': 'Another Vendor'},
+          'system' => {'manufacturer': 'some manufacturer', 'serial': 'AABBCCDDEE'},
+          'oem strings' => ['string 1', 'string 2'],
+        )
+      end
+    end
   end
 end

data/spec/unit/driver_spec.rb

@@ -197,7 +197,7 @@ describe VagrantPlugins::ProviderLibvirt::Driver do
           allow(machine.provider_config).to receive(:qemu_use_session).and_return(true)
         end
 
-        it 'should retreive the address via the system dhcp-leases API' do
+        it 'should retrieve the address via the system dhcp-leases API' do
           expect(domain).to receive(:mac).and_return("52:54:00:8b:dc:5f")
           expect(subject).to receive(:system_connection).and_return(system_connection1)
           expect(system_connection1).to receive(:list_all_networks).and_return(networks)

data/spec/unit/plugin_spec.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'support/sharedcontext'
+
+require 'vagrant-libvirt/plugin'
+
+
+describe VagrantPlugins::ProviderLibvirt::Plugin do
+  subject { described_class.new }
+
+  include_context 'unit'
+
+  describe '#action_hook remove_libvirt_image' do
+    before do
+      # set up some dummy boxes
+      box_path = File.join(env[:env].boxes.directory, 'vagrant-libvirt-VAGRANTSLASH-test', '0.0.1')
+      ['libvirt', 'virtualbox'].each do |provider|
+        provider_path = File.join(box_path, provider)
+        FileUtils.mkdir_p(provider_path)
+        metadata = {'provider': provider}
+        File.open(File.join(provider_path, 'metadata.json'), "w") { |f| f.write metadata.to_json }
+      end
+    end
+
+    it 'should call the action hook after box remove' do
+      expect(VagrantPlugins::ProviderLibvirt::Action).to receive(:remove_libvirt_image).and_return(Vagrant::Action::Builder.new)
+      expect {
+        env[:env].action_runner.run(
+          Vagrant::Action.action_box_remove, {
+            box_name: 'vagrant-libvirt/test',
+            box_provider: 'libvirt',
+            box_version: '0.0.1',
+            force_confirm_box_remove: true,
+            box_remove_all_versions: false,
+            ui: ui,
+          }
+        )
+      }.to_not raise_error
+    end
+  end
+end

data/spec/unit/templates/domain_all_settings.xml

@@ -23,7 +23,16 @@
     <kernel></kernel>
     <initrd></initrd>
     <cmdline></cmdline>
+    <smbios mode='sysinfo'/>
   </os>
+  <sysinfo type='smbios'>
+    <system>
+      <entry name='serial'>AAAAAAAA</entry>
+    </system>
+    <oemStrings>
+      <entry>AAAAAAAA</entry>
+    </oemStrings>
+  </sysinfo>
   <features>
     <acpi/>
     <apic/>
@@ -99,7 +108,7 @@
     </graphics>
     <video>
       <model type='cirrus' vram='16384' heads='1'>
-	<acceleration accel3d='yes'/>
+        <acceleration accel3d='yes'/>
       </model>
     </video>
     <rng model='virtio'>
@@ -123,9 +132,9 @@
     </hostdev>
     <hostdev mode='subsystem' type='usb'>
       <source startupPolicy='mandatory'>
-	<vendor id='0x1234'/>
-	<product id='0xabcd'/>
-	<address bus='1' device='2'/>
+        <vendor id='0x1234'/>
+        <product id='0xabcd'/>
+        <address bus='1' device='2'/>
       </source>
     </hostdev>
     <redirdev bus='usb' type='tcp'>

data/spec/unit/templates/domain_scsi_bus_storage.xml

@@ -0,0 +1,44 @@
+<domain type='' xmlns:qemu='http://libvirt.org/schemas/domain/qemu/1.0'>
+  <name></name>
+  <title></title>
+  <description></description>
+  <uuid></uuid>
+  <memory></memory>
+  <vcpu>1</vcpu>
+  <cpu mode='host-model'>
+    <model fallback='allow'></model>
+  </cpu>
+  <os>
+    <type>hvm</type>
+    <kernel></kernel>
+    <initrd></initrd>
+    <cmdline></cmdline>
+  </os>
+  <features>
+    <acpi/>
+    <apic/>
+    <pae/>
+  </features>
+  <clock offset='utc'>
+  </clock>
+  <devices>
+    <disk type='file' device='disk'>
+      <alias name='ua-box-volume-0'/>
+      <driver name='qemu' type='qcow2' cache='unsafe'/>
+      <source file='/var/lib/libvirt/images/test.qcow2'/>
+      <target dev='vda' bus='scsi'/>
+    </disk>
+    <controller type='scsi' model='virtio-scsi' index='0'/>
+    <serial type='pty'>
+      <target port='0'/>
+    </serial>
+    <console type='pty'>
+      <target port='0'/>
+    </console>
+    <input type='mouse' bus='ps2'/>
+    <graphics type='vnc' port='-1' autoport='yes' listen='127.0.0.1' keymap='en-us'/>
+    <video>
+      <model type='cirrus' vram='16384' heads='1'/>
+    </video>
+  </devices>
+</domain>

data/spec/unit/templates/domain_scsi_device_storage.xml

@@ -0,0 +1,44 @@
+<domain type='' xmlns:qemu='http://libvirt.org/schemas/domain/qemu/1.0'>
+  <name></name>
+  <title></title>
+  <description></description>
+  <uuid></uuid>
+  <memory></memory>
+  <vcpu>1</vcpu>
+  <cpu mode='host-model'>
+    <model fallback='allow'></model>
+  </cpu>
+  <os>
+    <type>hvm</type>
+    <kernel></kernel>
+    <initrd></initrd>
+    <cmdline></cmdline>
+  </os>
+  <features>
+    <acpi/>
+    <apic/>
+    <pae/>
+  </features>
+  <clock offset='utc'>
+  </clock>
+  <devices>
+    <disk type='file' device='disk'>
+      <alias name='ua-box-volume-0'/>
+      <driver name='qemu' type='qcow2' cache='unsafe'/>
+      <source file='/var/lib/libvirt/images/test.qcow2'/>
+      <target dev='sda' bus='scsi'/>
+    </disk>
+    <controller type='scsi' model='virtio-scsi' index='0'/>
+    <serial type='pty'>
+      <target port='0'/>
+    </serial>
+    <console type='pty'>
+      <target port='0'/>
+    </console>
+    <input type='mouse' bus='ps2'/>
+    <graphics type='vnc' port='-1' autoport='yes' listen='127.0.0.1' keymap='en-us'/>
+    <video>
+      <model type='cirrus' vram='16384' heads='1'/>
+    </video>
+  </devices>
+</domain>