vagrant-libvirt 0.8.2 → 0.10.1

data/spec/acceptance/provider_settings_spec.rb

@@ -0,0 +1,54 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe 'provider settings', acceptance: true do
+  include_context 'libvirt_acceptance'
+
+  after do
+    assert_execute('vagrant', 'destroy', '--force')
+  end
+
+  context 'with defaults' do
+    before do
+      environment.skeleton('default_settings')
+    end
+
+    it 'should succeed' do
+      status('Test: machine is created successfully')
+      result = environment.execute('vagrant', 'up')
+      expect(result).to exit_with(0)
+
+      status('Test: CPU matches default')
+      expect(result.stdout).to match(/Cpus:\s+1$/)
+
+      status('Test: memory matches default')
+      expect(result.stdout).to match(/Memory:\s+512M/)
+
+      status('Test: default prefix is used')
+      expect(result.stdout).to match(/Name:\s+work_default$/)
+    end
+  end
+
+  context 'with modified config' do
+    before do
+      environment.skeleton('adjusted_settings')
+    end
+
+    it 'should succeed' do
+      status('Test: machine is created successfully')
+      result = environment.execute('vagrant', 'up')
+      expect(result).to exit_with(0)
+
+      status('Test: CPUs are changed')
+      expect(result.stdout).to match(/Cpus:\s+2$/)
+
+      status('Test: memory is changed')
+      expect(result.stdout).to match(/Memory:\s+1000M$/)
+
+      status('Test: default prefix is changed')
+      expect(result.stdout).to match(/Name:\s+changed_default_prefixdefault$/)
+      expect(result.stdout).to match(/\(vda\).*changed_default_prefixdefault\.img/)
+    end
+  end
+end

data/spec/acceptance/simple_vm_provision_via_shell_spec.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe 'simple vm provision via shell', acceptance: true do
+  include_context 'libvirt_acceptance'
+
+  before do
+    environment.skeleton('simple_vm_provision')
+  end
+
+  after do
+    assert_execute('vagrant', 'destroy', '--force')
+  end
+
+  it 'should succeed' do
+    status('Test: machine is created successfully')
+    result = environment.execute('vagrant', 'up')
+    expect(result).to exit_with(0)
+
+    status('Test: provision script executed')
+    expect(result.stdout).to match(/Hello, World/)
+
+    status('Test: reload')
+    result = environment.execute('vagrant', 'reload')
+    expect(result).to exit_with(0)
+
+    status('Test: provision checked if already executed')
+    expect(result.stdout).to match(/Machine already provisioned/)
+  end
+end

data/spec/acceptance/snapshots_spec.rb

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe 'snapshots', acceptance: true do
+  include_context 'libvirt_acceptance'
+
+  after(:each) do
+    assert_execute('vagrant', 'destroy', '--force')
+  end
+
+  before do
+    environment.skeleton('snapshots')
+  end
+
+  it 'should succeed' do
+    status('Test: machine is created successfully')
+    expect(environment.execute('vagrant', 'up')).to exit_with(0)
+
+    status('Test: add test file')
+    expect(environment.execute('vagrant', 'ssh', '--', '-t', 'touch a.txt')).to exit_with(0)
+
+    status('Test: create snapshot')
+    expect(environment.execute('vagrant', 'snapshot', 'save', 'default', 'test')).to exit_with(0)
+
+    status('Test: modify files')
+    expect(environment.execute('vagrant', 'ssh', '--', '-t', 'rm a.txt')).to exit_with(0)
+    expect(environment.execute('vagrant', 'ssh', '--', '-t', 'ls a.txt')).to exit_with(1)
+    expect(environment.execute('vagrant', 'ssh', '--', '-t', 'touch b.txt')).to exit_with(0)
+
+    status('Test: restore snapshot')
+    expect(environment.execute('vagrant', 'snapshot', 'restore', 'test')).to exit_with(0)
+
+    status('Test: files are as expected')
+    expect(environment.execute('vagrant', 'ssh', '--', '-t', 'ls a.txt')).to exit_with(0)
+    expect(environment.execute('vagrant', 'ssh', '--', '-t', 'ls b.txt')).to exit_with(1)
+
+    status('Test: snapshot removal works')
+    expect(environment.execute('vagrant', 'snapshot', 'delete', 'test')).to exit_with(0)
+  end
+end

data/spec/acceptance/support-skeletons/package_complex/Vagrantfile.testbox

@@ -0,0 +1,14 @@
+# -*- mode: ruby -*-
+# vi: set ft=ruby :
+
+Vagrant.configure("2") do |config|
+  config.vm.box = "test-package-complex-domain"
+  config.vm.define 'package-complex'
+  config.vm.synced_folder ".", "/vagrant", disabled: true
+
+  config.vm.provider :libvirt do |libvirt|
+    libvirt.driver = "qemu"
+    libvirt.cpus = 2
+    libvirt.memory = 2048
+  end
+end

data/spec/acceptance/support-skeletons/package_complex/scripts/sysprep.sh

@@ -0,0 +1,32 @@
+#!/bin/sh -eux
+
+# consider purging any packages you don't need here
+
+echo "autoremoving packages and cleaning apt data"
+apt-get -y autoremove;
+apt-get -y clean;
+
+# repeat what machine-ids does in sysprep as this script needs to run via customize
+# which has a bug resulting in the machine-ids being regenerated
+
+if [ -f /etc/machine-id ]
+then
+    truncate --size=0 /etc/machine-id
+fi
+
+if [ -f /var/lib/dbus/machine-id ]
+then
+    truncate --size=0 /run/machine-id
+fi
+
+echo "remove /var/cache"
+find /var/cache -type f -exec rm -rf {} \;
+
+echo "force a new random seed to be generated"
+rm -f /var/lib/systemd/random-seed
+
+# for debian based systems ensure host keys regenerated on boot
+if [ -e /usr/sbin/dpkg-reconfigure ]
+then
+    printf "@reboot root command bash -c 'export PATH=$PATH:/usr/sbin ; export DEBIAN_FRONTEND=noninteractive ; export DEBCONF_NONINTERACTIVE_SEEN=true ; /usr/sbin/dpkg-reconfigure openssh-server &>/dev/null ; /bin/systemctl restart ssh.service ; rm --force /etc/cron.d/keys'\n" > /etc/cron.d/keys
+fi

data/spec/acceptance/support-skeletons/package_simple/Vagrantfile.testbox

@@ -0,0 +1,10 @@
+# -*- mode: ruby -*-
+# vi: set ft=ruby :
+
+Vagrant.configure("2") do |config|
+  config.vm.box = "test-package-simple-domain"
+  config.vm.define 'package-simple'
+  config.ssh.shell = "/bin/sh"
+  config.ssh.insert_key = false
+  config.vm.synced_folder ".", "/vagrant", disabled: true
+end

data/spec/acceptance/two_disks_spec.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe 'handle two disk machine', acceptance: true do
+  include_context 'libvirt_acceptance'
+
+  after do
+    assert_execute('vagrant', 'destroy', '--force')
+  end
+
+  before do
+    environment.skeleton('two_disks')
+    environment.execute(File.expand_path('../../tools/create_box_with_two_disks.sh', __dir__),
+                        environment.homedir.to_s, 'vagrant')
+  end
+
+  it 'should succeed' do
+    status('Test: machine is created successfully')
+    result = environment.execute('vagrant', 'up')
+    expect(result).to exit_with(0)
+
+    status('Test: disk one referenced')
+    expect(result.stdout).to match(/Image\(vda\):.*work_default.img, virtio, 2G/)
+
+    status('Test: disk two referenced')
+    expect(result.stdout).to match(/Image\(vdb\):.*work_default_1.img, virtio, 10G/)
+  end
+end

data/spec/acceptance/use_qemu_agent_for_connectivity_spec.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe 'use qemu agent to determine machine private address', acceptance: true do
+  include_context 'libvirt_acceptance'
+
+  before do
+    environment.skeleton('qemu_agent')
+  end
+
+  after do
+    assert_execute('vagrant', 'destroy', '--force')
+  end
+
+  it 'should succeed' do
+    status('Test: machine is created successfully')
+    result = environment.execute('vagrant', 'up')
+    expect(result).to exit_with(0)
+
+    # extract SSH IP address emitted as it should be the private network since
+    # the mgmt network has not been attached
+    hostname = result.stdout.each_line.find { |line| line.include?('SSH address:') }
+    expect(hostname).to_not be_nil
+    ip_address = hostname.strip.split.last.split(':').first
+    # default private network for vagrant-libvirt unless explicitly configured
+    expect(IPAddr.new('172.28.128.0/255.255.255.0')).to include(IPAddr.new(ip_address))
+
+    # ssh'ing successfully means that the private network is accessible
+    status('Test: machine private network is accessible')
+    result = environment.execute('vagrant', 'ssh', '-c', 'echo "hello, world"')
+    expect(result).to exit_with(0)
+    expect(result.stdout).to match(/hello, world/)
+  end
+end

data/spec/spec_helper.rb

@@ -55,4 +55,7 @@ RSpec.configure do |config|
   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
 end

data/spec/support/acceptance/configuration.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+require 'vagrant-spec/acceptance/configuration'
+
+module VagrantPlugins
+  module VagrantLibvirt
+    module Spec
+      module Acceptance
+        class Configuration < Vagrant::Spec::Acceptance::Configuration
+          attr_accessor :clean_on_fail
+
+          def initialize
+            super
+
+            @clean_on_fail = true
+          end
+        end
+      end
+    end
+  end
+end

data/spec/support/acceptance/context.rb

@@ -0,0 +1,70 @@
+# frozen_string_literal: true
+
+require 'vagrant-spec/acceptance/configuration'
+require "vagrant-spec/acceptance/rspec/matcher_exit_with"
+
+require_relative 'isolated_environment'
+
+shared_context "acceptance" do
+  # Setup the environment so that we have an isolated area
+  # to run Vagrant. We do some configuration here as well in order
+  # to replace "vagrant" with the proper path to Vagrant as well
+  # as tell the isolated environment about custom environmental
+  # variables to pass in.
+  let!(:environment) { new_environment }
+
+  let(:config) { Vagrant::Spec::Acceptance::Configuration.new }
+
+  let(:extra_env) { {} }
+
+  # The skeleton paths that will be used to configure environments.
+  let(:skeleton_paths) do
+    root = Vagrant::Spec.source_root.join("acceptance", "support-skeletons")
+    config.skeleton_paths.dup.unshift(root)
+  end
+
+  after(:each) do |example|
+    if example.exception.nil? || config.clean_on_fail
+      environment.close
+    else
+      example.reporter.message("Temporary work and home dirs (#{environment.workdir} and #{environment.homedir}) not removed to allow debug")
+    end
+  end
+
+  # Creates a new isolated environment instance each time it is called.
+  #
+  # @return [Acceptance::IsolatedEnvironment]
+  def new_environment(env=nil)
+    apps = { "vagrant" => config.vagrant_path }
+    env  = config.env.merge(env || {})
+    env.merge!(extra_env)
+
+    VagrantPlugins::VagrantLibvirt::Spec::AcceptanceIsolatedEnvironment.new(
+      apps: apps,
+      env: env,
+      skeleton_paths: skeleton_paths,
+    )
+  end
+
+  # Executes the given command in the context of the isolated environment.
+  #
+  # @return [Object]
+  def execute(*args, env: nil, log: true, &block)
+    env ||= environment
+    status("Execute: #{args.join(" ")}") if log
+    env.execute(*args, &block)
+  end
+
+  # This method is an assertion helper for asserting that a process
+  # succeeds. It is a wrapper around `execute` that asserts that the
+  # exit status was successful.
+  def assert_execute(*args, env: nil, &block)
+    result = execute(*args, env: env, &block)
+    expect(result).to exit_with(0)
+    result
+  end
+
+  def status(message)
+    RSpec.world.reporter.message(message)
+  end
+end

data/spec/support/acceptance/isolated_environment.rb

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+
+require 'vagrant-spec/acceptance/isolated_environment'
+require 'vagrant-spec/subprocess'
+require 'vagrant-spec/which'
+
+module VagrantPlugins
+  module VagrantLibvirt
+    module Spec
+      class AcceptanceIsolatedEnvironment < Vagrant::Spec::AcceptanceIsolatedEnvironment
+        # Executes a command in the context of this isolated environment.
+        # Any command executed will therefore see our temporary directory
+        # as the home directory.
+        #
+        # If the command has been defined with a special path, then the
+        # command will be replaced with the full path to that command.
+        def execute(command, *args, **options)
+          # Create the command
+          command = replace_command(command)
+          # Use provided command if it is a valid executable
+          if !File.executable?(command)
+            # If it's not a valid executable, search for vagrant
+            command = Vagrant::Spec::Which.which(command)
+          end
+
+          # Build up the options
+          options[:env] = @env.merge(options.delete(:extra_env) || {})
+          options[:notify] = [:stdin, :stderr, :stdout]
+          options[:workdir] = @workdir.to_s
+
+          # Execute, logging out the stdout/stderr as we get it
+          @logger.info("Executing: #{[command].concat(args).inspect}")
+          Vagrant::Spec::Subprocess.execute(command, *args, **options) do |type, data|
+            @logger.debug("#{type}: #{data}") if type == :stdout || type == :stderr
+            yield type, data if block_given?
+          end
+        end
+      end
+    end
+  end
+end

data/spec/support/libvirt_acceptance_context.rb

@@ -0,0 +1,64 @@
+# frozen_string_literal: true
+
+require_relative 'acceptance/context'
+
+FALSEY_VALUES = %w[f false no n 0].freeze
+
+shared_context 'libvirt_acceptance' do
+  include_context 'acceptance'
+
+  # The skeleton paths that will be used to configure environments.
+  let(:skeleton_paths) do
+    root = File.expand_path('../acceptance/support-skeletons', __dir__)
+    config.skeleton_paths.dup.unshift(root)
+  end
+
+  let(:config) do
+    c = VagrantPlugins::VagrantLibvirt::Spec::Acceptance::Configuration.new
+    c.clean_on_fail = FALSEY_VALUES.include?(ENV.fetch('VAGRANT_SPEC_SKIP_CLEANUP', 'false').to_s.downcase)
+
+    c
+  end
+
+  before(:each) do
+    # allow execution environment to cache boxes used
+    symlink_boxes(ENV.fetch('VAGRANT_HOME', nil), environment)
+  end
+
+  after(:each) do
+    # ensure we remove the symlink
+    boxes_symlink = File.join(environment.homedir, 'boxes')
+    File.delete(boxes_symlink) if File.symlink?(boxes_symlink)
+  end
+
+  around do |example|
+    vagrant_cwd = ENV.delete('VAGRANT_CWD')
+    env_provider_before = ENV.fetch('VAGRANT_DEFAULT_PROVIDER', nil)
+    ENV['VAGRANT_DEFAULT_PROVIDER'] = 'libvirt'
+
+    begin
+      example.run
+    ensure
+      ENV['VAGRANT_CWD'] = vagrant_cwd if vagrant_cwd
+      if env_provider_before.nil?
+        ENV.delete('VAGRANT_DEFAULT_PROVIDER')
+      else
+        ENV['VAGRANT_DEFAULT_PROVIDER'] = env_provider_before
+      end
+    end
+  end
+
+  def duplicate_environment(env, *args)
+    dup_env = new_environment(*args)
+    symlink_boxes(env.homedir, dup_env)
+
+    dup_env
+  end
+
+  def symlink_boxes(vagrant_home, target_env)
+    return if vagrant_home.nil?
+
+    # allow use the same boxes location as source environment
+    File.symlink File.realpath(File.join(vagrant_home, 'boxes')), File.join(target_env.homedir, 'boxes')
+  end
+end

data/spec/support/libvirt_context.rb

@@ -10,6 +10,7 @@ shared_context 'libvirt' do
   let(:libvirt_context) { true                      }
   let(:id)              { 'dummy-vagrant_dummy'     }
   let(:connection)      { double('connection') }
+  let(:driver)          { instance_double(VagrantPlugins::ProviderLibvirt::Driver) }
   let(:domain)          { instance_double(::Fog::Libvirt::Compute::Server) }
   let(:libvirt_client)  { instance_double(::Libvirt::Connect) }
   let(:libvirt_domain)  { instance_double(::Libvirt::Domain) }
@@ -33,5 +34,8 @@ shared_context 'libvirt' do
 
     allow(machine).to receive(:id).and_return(id)
     allow(Log4r::Logger).to receive(:new).and_return(logger)
+
+    allow(machine.provider).to receive('driver').and_return(driver)
+    allow(driver).to receive(:connection).and_return(connection)
   end
 end

data/spec/support/sharedcontext.rb

@@ -37,5 +37,6 @@ shared_context 'unit' do
   before (:each) do
     allow(machine).to receive(:guest).and_return(guest)
     allow(machine).to receive(:communicate).and_return(communicator)
+    allow(machine).to receive(:ui).and_return(ui)
   end
 end

data/spec/unit/action/cleanup_on_failure_spec.rb

@@ -34,8 +34,6 @@ describe VagrantPlugins::ProviderLibvirt::Action::CleanupOnFailure do
   let(:state) { double('state') }
 
   before do
-    allow_any_instance_of(VagrantPlugins::ProviderLibvirt::Driver)
-      .to receive(:connection).and_return(connection)
     allow(machine).to receive(:id).and_return('test-machine-id')
     allow(machine).to receive(:state).and_return(state)
 

data/spec/unit/action/create_domain_spec/sysinfo.xml

@@ -0,0 +1,66 @@
+<domain type='kvm' xmlns:qemu='http://libvirt.org/schemas/domain/qemu/1.0'>
+  <name>vagrant-test_default</name>
+  <title></title>
+  <description>Source: /rootpath/Vagrantfile</description>
+  <uuid></uuid>
+  <memory>524288</memory>
+  <vcpu>1</vcpu>
+  <cpu mode='host-model'>
+    <model fallback='allow'></model>
+  </cpu>
+  <os>
+    <type>hvm</type>
+    <kernel></kernel>
+    <initrd></initrd>
+    <cmdline></cmdline>
+    <smbios mode='sysinfo'/>
+  </os>
+  <sysinfo type='smbios'>
+    <bios>
+      <entry name='vendor'>Test Vendor</entry>
+    </bios>
+    <system>
+      <entry name='manufacturer'>Test Manufacturer</entry>
+      <entry name='version'>0.1.0</entry>
+    </system>
+    <baseBoard>
+      <entry name='manufacturer'>Test Manufacturer</entry>
+    </baseBoard>
+    <chassis>
+      <entry name='manufacturer'>Test Manufacturer</entry>
+      <entry name='serial'>AABBCCDDEE</entry>
+    </chassis>
+    <oemStrings>
+      <entry>app1: string1</entry>
+      <entry>app1: string2</entry>
+      <entry>app2: string1</entry>
+      <entry>app2: string2</entry>
+    </oemStrings>
+  </sysinfo>
+  <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='default'/>
+      <source file='/var/lib/libvirt/images/vagrant-test_default.img'/>
+      <target dev='vda' bus='virtio'/>
+    </disk>
+    <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/action/create_domain_spec/sysinfo_only_required.xml

@@ -0,0 +1,49 @@
+<domain type='kvm' xmlns:qemu='http://libvirt.org/schemas/domain/qemu/1.0'>
+  <name>vagrant-test_default</name>
+  <title></title>
+  <description>Source: /rootpath/Vagrantfile</description>
+  <uuid></uuid>
+  <memory>524288</memory>
+  <vcpu>1</vcpu>
+  <cpu mode='host-model'>
+    <model fallback='allow'></model>
+  </cpu>
+  <os>
+    <type>hvm</type>
+    <kernel></kernel>
+    <initrd></initrd>
+    <cmdline></cmdline>
+    <smbios mode='sysinfo'/>
+  </os>
+  <sysinfo type='smbios'>
+    <bios>
+      <entry name='vendor'>Test Vendor</entry>
+    </bios>
+  </sysinfo>
+  <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='default'/>
+      <source file='/var/lib/libvirt/images/vagrant-test_default.img'/>
+      <target dev='vda' bus='virtio'/>
+    </disk>
+    <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>