vagrant-libvirt 0.8.0 → 0.9.0

data/lib/vagrant-libvirt/templates/domain.xml.erb

@@ -117,9 +117,9 @@
             @disk_driver_opts.reject { |k,v| v.nil? }
                              .map { |k,v| "#{k}='#{v}'"}
                              .join(' ') -%>/>
-      <source file='<%= volume[:path] %>'/>
+      <source file='<%= volume[:absolute_path] %>'/>
       <%# we need to ensure a unique target dev -%>
-      <target dev='<%= volume[:dev] %>' bus='<%= volume[:bus] %>'/>
+      <target dev='<%= volume[:device] %>' bus='<%= volume[:bus] %>'/>
     </disk>
 <%- end -%>
 <%# additional disks -%>
@@ -230,15 +230,20 @@
       <backend model='random'>/dev/random</backend>
     </rng>
 <%- end -%>
-<%- @pcis.each do |pci| -%>
+<%-
+  @pcis.each do |pci|
+    pci = pci.reject { |k,v| v.nil? }
+    guest_prefix = "guest_"
+    pci_host = pci.select { |k,_| !k.to_s.start_with?(guest_prefix) }
+    pci_guest = pci.select { |k,_| k.to_s.start_with?(guest_prefix) }
+-%>
     <hostdev mode='subsystem' type='pci' managed='yes'>
       <source>
-	<address <%= pci.select { |k,_| [:domain, :bus, :slot, :function].include? k }
-	  .reject { |k,v| v.nil? }
-	  .map { |k,v| "#{k.to_s}='#{v}'" }
-	  .join(' ')
-	-%>/>
+        <address <%= pci_host.map { |k,v| "#{k.to_s}='#{v}'" } .join(' ') -%>/>
       </source>
+    <%- if !pci_guest.empty? -%>
+      <address type='pci' <%= pci_guest.map { |k,v| "#{k.to_s[guest_prefix.length..-1]}='#{v}'" } .join(' ') -%>/>
+    <%- end -%>
     </hostdev>
 <%- end -%>
 <%- @usbs.each do |usb| -%>

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

@@ -0,0 +1,15 @@
+ # Ripped from https://libvirt.org/html/libvirt-libvirt-domain.html#types
+module VagrantPlugins
+  module ProviderLibvirt
+    module Util
+      module DomainFlags
+        # virDomainUndefineFlagsValues
+        VIR_DOMAIN_UNDEFINE_MANAGED_SAVE = 1 # Also remove any managed save
+        VIR_DOMAIN_UNDEFINE_SNAPSHOTS_METADATA = 2 # If last use of domain, then also remove any snapshot metadata
+        VIR_DOMAIN_UNDEFINE_NVRAM = 4 # Also remove any nvram file
+        VIR_DOMAIN_UNDEFINE_KEEP_NVRAM = 8 # Keep nvram file
+        VIR_DOMAIN_UNDEFINE_CHECKPOINTS_METADATA = 16 # If last use of domain, then also remove any checkpoint metadata Future undefine control flags should come here.
+      end
+    end
+  end
+end

data/lib/vagrant-libvirt/util.rb

@@ -9,6 +9,7 @@ module VagrantPlugins
       autoload :NetworkUtil, 'vagrant-libvirt/util/network_util'
       autoload :StorageUtil, 'vagrant-libvirt/util/storage_util'
       autoload :ErrorCodes, 'vagrant-libvirt/util/error_codes'
+      autoload :DomainFlags, 'vagrant-libvirt/util/domain_flags'
       autoload :Ui, 'vagrant-libvirt/util/ui'
     end
   end

data/lib/vagrant-libvirt/version

@@ -1 +1 @@
-0.8.0
+0.9.0

data/locales/en.yml

@@ -58,9 +58,11 @@ en:
     remove_stale_volume: |-
       Remove stale volume...
 
-    destroy:
+    domain_xml:
       obsolete_method: |
-        Destroying machine that was originally created without device aliases (pre 0.6.0), using fallback approach.
+        Machine that was originally created without device aliases (pre 0.6.0), using fall-back approach for device identification.
+
+    destroy:
       unexpected_volumes: |
         Unexpected number of volumes detected, possibly additional volumes attached outside of vagrant-libvirt.
         Attempting to handle this correctly, however may experience unexpected behaviour in the domain destroy.

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/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.rb

@@ -21,15 +21,9 @@ describe VagrantPlugins::ProviderLibvirt::Action::CreateDomain do
   let(:domain_volume) { instance_double(::Fog::Libvirt::Compute::Volume) }
 
   let(:domain_xml) { File.read(File.join(File.dirname(__FILE__), File.basename(__FILE__, '.rb'), domain_xml_file)) }
-  let(:storage_pool_xml) { File.read(File.join(File.dirname(__FILE__), File.basename(__FILE__, '.rb'), storage_pool_xml_file)) }
-  let(:libvirt_storage_pool) { double('storage_pool') }
 
   describe '#call' do
     before do
-      allow_any_instance_of(VagrantPlugins::ProviderLibvirt::Driver)
-        .to receive(:connection).and_return(connection)
-      allow(connection).to receive(:client).and_return(libvirt_client)
-
       allow(connection).to receive(:servers).and_return(servers)
       allow(connection).to receive(:volumes).and_return(volumes)
       allow(volumes).to receive(:all).and_return([domain_volume])
@@ -43,15 +37,17 @@ describe VagrantPlugins::ProviderLibvirt::Action::CreateDomain do
 
       env[:domain_name] = "vagrant-test_default"
 
-      env[:box_volumes] = []
-      env[:box_volumes].push({
+      env[:domain_volumes] = []
+      env[:domain_volumes].push({
+        :device=>'vda',
+        :bus=>'virtio',
+        :cache=>'default',
+        :absolute_path=>'/var/lib/libvirt/images/vagrant-test_default.img',
         :path=>"/test/box.img",
-        :name=>"test_vagrant_box_image_1.1.1_0.img", 
+        :name=>'test_vagrant_box_image_1.1.1_0.img',
         :virtual_size=> ByteNumber.new(5),
+        :pool=>'default',
       })
-      # should be ignored for system session and used for user session
-      allow(Process).to receive(:uid).and_return(9999)
-      allow(Process).to receive(:gid).and_return(9999)
     end
 
     context 'connection => qemu:///system' do
@@ -61,123 +57,99 @@ describe VagrantPlugins::ProviderLibvirt::Action::CreateDomain do
         allow(machine.provider_config).to receive(:qemu_use_session).and_return(false)
       end
 
-      context 'default pool' do
-        it 'should execute correctly' do
-          expect(servers).to receive(:create).with(xml: domain_xml).and_return(machine)
-          expect(volumes).to_not receive(:create) # additional disks only
-
-          expect(subject.call(env)).to be_nil
-        end
-
-        context 'with no box' do
-          let(:storage_pool_xml_file) { 'default_system_storage_pool.xml' }
-          let(:vagrantfile) do
-            <<-EOF
-            Vagrant.configure('2') do |config|
-              config.vm.define :test
-            end
-            EOF
-          end
+      it 'should execute correctly' do
+        expect(servers).to receive(:create).with(xml: domain_xml).and_return(machine)
+        expect(volumes).to_not receive(:create) # additional disks only
 
-          it 'should query for the storage pool path' do
-            expect(libvirt_client).to receive(:lookup_storage_pool_by_name).and_return(libvirt_storage_pool)
-            expect(libvirt_storage_pool).to receive(:xml_desc).and_return(storage_pool_xml)
-            expect(servers).to receive(:create).and_return(machine)
+        expect(subject.call(env)).to be_nil
+      end
 
-            expect(subject.call(env)).to be_nil
-          end
+      context 'additional disks' do
+        let(:disks) do
+          [
+            :device        => 'vdb',
+            :cache         => 'default',
+            :bus           => 'virtio',
+            :type          => 'qcow2',
+            :absolute_path => '/var/lib/libvirt/images/vagrant-test_default-vdb.qcow2',
+            :virtual_size  => ByteNumber.new(20*1024*1024*1024),
+            :pool          => 'default',
+          ]
         end
 
-        context 'additional disks' do
-          let(:vagrantfile_providerconfig) do
-            <<-EOF
-            libvirt.storage :file, :size => '20G'
-            EOF
-          end
-
-          context 'volume create failed' do
-            it 'should raise an exception' do
-              expect(volumes).to receive(:create).and_raise(Libvirt::Error)
-
-              expect{ subject.call(env) }.to raise_error(VagrantPlugins::ProviderLibvirt::Errors::FogCreateDomainVolumeError)
-            end
-          end
+        before do
+          env[:disks] = disks
+        end
 
-          context 'volume create succeeded' do
-            let(:domain_xml_file) { 'additional_disks_domain.xml' }
-
-            it 'should complete' do
-              expect(volumes).to receive(:create).with(
-                hash_including(
-                  :path        => "/var/lib/libvirt/images/vagrant-test_default-vdb.qcow2",
-                  :owner       => 0,
-                  :group       => 0,
-                  :pool_name   => "default",
-                )
-              )
-              expect(servers).to receive(:create).with(xml: domain_xml).and_return(machine)
+        context 'volume create failed' do
+          it 'should raise an exception' do
+            expect(volumes).to receive(:create).and_raise(Libvirt::Error)
 
-              expect(subject.call(env)).to be_nil
-            end
+            expect{ subject.call(env) }.to raise_error(VagrantPlugins::ProviderLibvirt::Errors::FogCreateDomainVolumeError)
           end
         end
 
-        context 'with custom disk device setting' do
-          let(:domain_xml_file) { 'custom_disk_settings.xml' }
-          let(:vagrantfile_providerconfig) {
-            <<-EOF
-              libvirt.disk_device = 'sda'
-            EOF
-          }
+        context 'volume create succeeded' do
+          let(:domain_xml_file) { 'additional_disks_domain.xml' }
 
-          it 'should set the domain device' do
-            expect(ui).to receive(:info).with(/ -- Image\(sda\):.*/)
+          it 'should complete' do
+            expect(volumes).to receive(:create).with(
+              hash_including(
+                :path        => "/var/lib/libvirt/images/vagrant-test_default-vdb.qcow2",
+                :owner       => 0,
+                :group       => 0,
+                :pool_name   => "default",
+              )
+            )
             expect(servers).to receive(:create).with(xml: domain_xml).and_return(machine)
 
             expect(subject.call(env)).to be_nil
           end
         end
+      end
 
-        context 'with two domain disks' do
-          let(:domain_xml_file) { 'two_disk_settings.xml' }
-          let(:domain_volume_2) { double('domain_volume 2') }
-
-          before do
-            expect(volumes).to receive(:all).and_return([domain_volume])
-            expect(volumes).to receive(:all).and_return([domain_volume_2])
-            expect(domain_volume_2).to receive(:pool_name).and_return('default')
-            expect(domain_volume_2).to receive(:path).and_return('/var/lib/libvirt/images/vagrant-test_default_1.img')
-
-            env[:box_volumes].push({
-              :path=>"/test/box_1.img",
-              :name=>"test_vagrant_box_image_1.1.1_1.img",
-              :virtual_size=> ByteNumber.new(5),
-            })
-          end
+      context 'with custom disk device setting' do
+        let(:domain_xml_file) { 'custom_disk_settings.xml' }
 
-          it 'should correctly assign device entries' do
-            expect(ui).to receive(:info).with(/ -- Image\(vda\):.*/)
-            expect(ui).to receive(:info).with(/ -- Image\(vdb\):.*/)
-            expect(servers).to receive(:create).with(xml: domain_xml).and_return(machine)
+        before do
+          env[:domain_volumes][0][:device] = 'sda'
+        end
 
-            expect(subject.call(env)).to be_nil
-          end
+        it 'should set the domain device' do
+          expect(ui).to receive(:info).with(/ -- Image\(sda\):.*/)
+          expect(servers).to receive(:create).with(xml: domain_xml).and_return(machine)
+
+          expect(subject.call(env)).to be_nil
         end
       end
 
-      context 'no default pool' do
-        let(:vagrantfile) do
-          <<-EOF
-          Vagrant.configure('2') do |config|
-            config.vm.define :test
-          end
-          EOF
+      context 'with two domain disks' do
+        let(:domain_xml_file) { 'two_disk_settings.xml' }
+        let(:domain_volume_2) { double('domain_volume 2') }
+
+        before do
+          expect(volumes).to receive(:all).with(name: 'vagrant-test_default.img').and_return([domain_volume])
+          expect(volumes).to receive(:all).with(name: 'vagrant-test_default_1.img').and_return([domain_volume_2])
+          expect(domain_volume_2).to receive(:pool_name).and_return('default')
+
+          env[:domain_volumes].push({
+            :device=>'vdb',
+            :bus=>'virtio',
+            :cache=>'default',
+            :absolute_path=>'/var/lib/libvirt/images/vagrant-test_default_1.img',
+            :path=>"/test/box_1.img",
+            :name=>"test_vagrant_box_image_1.1.1_1.img",
+            :virtual_size=> ByteNumber.new(5),
+            :pool=>'default',
+          })
         end
 
-        it 'should raise an exception' do
-          expect(libvirt_client).to receive(:lookup_storage_pool_by_name).and_return(nil)
+        it 'should list multiple device entries' do
+          expect(ui).to receive(:info).with(/ -- Image\(vda\):.*/)
+          expect(ui).to receive(:info).with(/ -- Image\(vdb\):.*/)
+          expect(servers).to receive(:create).with(xml: domain_xml).and_return(machine)
 
-          expect{ subject.call(env) }.to raise_error(VagrantPlugins::ProviderLibvirt::Errors::NoStoragePool)
+          expect(subject.call(env)).to be_nil
         end
       end
     end
@@ -187,57 +159,51 @@ describe VagrantPlugins::ProviderLibvirt::Action::CreateDomain do
         allow(machine.provider_config).to receive(:qemu_use_session).and_return(true)
       end
 
-      context 'default pool' do
-        it 'should execute correctly' do
-          expect(servers).to receive(:create).and_return(machine)
+      it 'should execute correctly' do
+        expect(servers).to receive(:create).and_return(machine)
 
-          expect(subject.call(env)).to be_nil
+        expect(subject.call(env)).to be_nil
+      end
+
+      context 'additional disks' do
+        let(:vagrantfile_providerconfig) do
+          <<-EOF
+          libvirt.qemu_use_session = true
+          EOF
         end
 
-        context 'with no box' do
-          let(:storage_pool_xml_file) { 'default_user_storage_pool.xml' }
-          let(:vagrantfile) do
-            <<-EOF
-            Vagrant.configure('2') do |config|
-              config.vm.define :test
-              config.vm.provider :libvirt do |libvirt|
-                #{vagrantfile_providerconfig}
-              end
-            end
-            EOF
-          end
+        let(:disks) do
+          [
+            :device        => 'vdb',
+            :cache         => 'default',
+            :bus           => 'virtio',
+            :type          => 'qcow2',
+            :absolute_path => '/var/lib/libvirt/images/vagrant-test_default-vdb.qcow2',
+            :virtual_size  => ByteNumber.new(20*1024*1024*1024),
+            :pool          => 'default',
+          ]
+        end
 
-          it 'should query for the storage pool path' do
-            expect(libvirt_client).to receive(:lookup_storage_pool_by_name).and_return(libvirt_storage_pool)
-            expect(libvirt_storage_pool).to receive(:xml_desc).and_return(storage_pool_xml)
-            expect(servers).to receive(:create).and_return(machine)
+        before do
+          expect(Process).to receive(:uid).and_return(9999).at_least(:once)
+          expect(Process).to receive(:gid).and_return(9999).at_least(:once)
 
-            expect(subject.call(env)).to be_nil
-          end
+          env[:disks] = disks
         end
 
-        context 'additional disks' do
-          let(:vagrantfile_providerconfig) do
-            <<-EOF
-            libvirt.qemu_use_session = true
-            libvirt.storage :file, :size => '20G'
-            EOF
-          end
-
-          context 'volume create succeeded' do
-            it 'should complete' do
-              expect(volumes).to receive(:create).with(
-                hash_including(
-                  :path        => "/var/lib/libvirt/images/vagrant-test_default-vdb.qcow2",
-                  :owner       => 9999,
-                  :group       => 9999,
-                  :pool_name   => "default",
-                )
+        context 'volume create succeeded' do
+          it 'should complete' do
+            expect(volumes).to receive(:create).with(
+              hash_including(
+                :path        => "/var/lib/libvirt/images/vagrant-test_default-vdb.qcow2",
+                :owner       => 9999,
+                :group       => 9999,
+                :pool_name   => "default",
               )
-              expect(servers).to receive(:create).and_return(machine)
+            )
+            expect(servers).to receive(:create).and_return(machine)
 
-              expect(subject.call(env)).to be_nil
-            end
+            expect(subject.call(env)).to be_nil
           end
         end
       end

data/spec/unit/action/create_domain_volume_spec.rb

@@ -26,9 +26,6 @@ describe VagrantPlugins::ProviderLibvirt::Action::CreateDomainVolume do
 
   describe '#call' do
     before do
-      allow_any_instance_of(VagrantPlugins::ProviderLibvirt::Driver)
-        .to receive(:connection).and_return(connection)
-      allow(connection).to receive(:client).and_return(libvirt_client)
       allow(connection).to receive(:volumes).and_return(volumes)
       allow(volumes).to receive(:all).and_return(all)
       allow(all).to receive(:first).and_return(box_volume)

data/spec/unit/action/destroy_domain_spec.rb

@@ -19,10 +19,14 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
 
   let(:domain_xml) { File.read(File.join(File.dirname(__FILE__), File.basename(__FILE__, '.rb'), domain_xml_file)) }
 
+  let(:destroy_method) { double('destroy_method') }
+
   before do
     allow(machine.provider).to receive('driver').and_return(driver)
     allow(driver).to receive(:connection).and_return(connection)
     allow(logger).to receive(:info)
+    allow(domain).to receive(:method).with(:destroy).and_return(destroy_method)
+    allow(destroy_method).to receive(:parameters).and_return([[:opt, :options, :flags]])
   end
 
   describe '#call' do
@@ -49,7 +53,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
 
       context 'when box only has one root disk' do
         it 'calls fog to destroy volumes' do
-          expect(domain).to receive(:destroy).with(destroy_volumes: true)
+          expect(domain).to receive(:destroy).with(destroy_volumes: true, flags: 0)
           expect(subject.call(env)).to be_nil
         end
 
@@ -69,7 +73,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
           end
 
           it 'destroys disks individually' do
-            expect(domain).to receive(:destroy).with(destroy_volumes: false)
+            expect(domain).to receive(:destroy).with(destroy_volumes: false, flags: 0)
             expect(extra_disk).to receive(:destroy) # extra disk remove
             expect(root_disk).to receive(:destroy)  # root disk remove
             expect(subject.call(env)).to be_nil
@@ -81,7 +85,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
         let(:domain_xml_file) { 'box_multiple_disks.xml' }
 
         it 'calls fog to destroy volumes' do
-          expect(domain).to receive(:destroy).with(destroy_volumes: true)
+          expect(domain).to receive(:destroy).with(destroy_volumes: true, flags: 0)
           expect(subject.call(env)).to be_nil
         end
 
@@ -111,7 +115,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
               expect(disk).to receive(:name).and_return(name).at_least(:once)
               expect(disk).to receive(:destroy)
             end
-            expect(domain).to receive(:destroy).with(destroy_volumes: false)
+            expect(domain).to receive(:destroy).with(destroy_volumes: false, flags: 0)
             expect(subject.call(env)).to be_nil
           end
 
@@ -133,7 +137,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
                 next if disk == domain_disks.last.first
                 expect(disk).to receive(:destroy)
               end
-              expect(domain).to receive(:destroy).with(destroy_volumes: false)
+              expect(domain).to receive(:destroy).with(destroy_volumes: false, flags: 0)
               expect(subject.call(env)).to be_nil
             end
 
@@ -141,7 +145,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
               let(:domain_xml_file) { 'box_multiple_disks_and_additional_and_custom_disks_no_aliases.xml' }
 
               it 'only destroys expected disks' do
-                expect(ui).to receive(:warn).with(/Destroying machine that was originally created without device aliases.*/)
+                expect(ui).to receive(:warn).with(/Machine that was originally created without device aliases.*/)
                 expect(ui).to receive(:warn).with(/Unexpected number of volumes detected/)
                 expect(ui).to receive(:warn).with(/box metadata not available to get volume list during destroy, assuming inferred list/)
                 domain_disks.each do |disk, name|
@@ -150,7 +154,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
                   next if domain_disks.last.first == disk
                   expect(disk).to receive(:destroy)
                 end
-                expect(domain).to receive(:destroy).with(destroy_volumes: false)
+                expect(domain).to receive(:destroy).with(destroy_volumes: false, flags: 0)
                 expect(subject.call(env)).to be_nil
               end
 
@@ -168,7 +172,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
                 end
 
                 it 'only destroys expected disks' do
-                  expect(ui).to receive(:warn).with(/Destroying machine that was originally created without device aliases.*/)
+                  expect(ui).to receive(:warn).with(/Machine that was originally created without device aliases.*/)
                   expect(ui).to receive(:warn).with(/Unexpected number of volumes detected/)
                   domain_disks.each do |disk, name|
                     expect(disk).to receive(:name).and_return(name).at_least(:once)
@@ -176,7 +180,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
                     next if domain_disks.last.first == disk
                     expect(disk).to receive(:destroy)
                   end
-                  expect(domain).to receive(:destroy).with(destroy_volumes: false)
+                  expect(domain).to receive(:destroy).with(destroy_volumes: false, flags: 0)
                   expect(subject.call(env)).to be_nil
                 end
               end
@@ -185,6 +189,35 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
         end
       end
 
+      context 'when has nvram' do
+        let(:vagrantfile) do
+          <<-EOF
+          Vagrant.configure('2') do |config|
+            config.vm.define :test
+            config.vm.provider :libvirt do |libvirt|
+              libvirt.nvram = "test"
+            end
+          end
+          EOF
+        end
+
+        it 'sets destroy flags to keep nvram' do
+          expect(domain).to receive(:destroy).with(destroy_volumes: true, flags: VagrantPlugins::ProviderLibvirt::Util::DomainFlags::VIR_DOMAIN_UNDEFINE_KEEP_NVRAM)
+          expect(subject.call(env)).to be_nil
+        end
+
+        context 'when fog does not support destroy with flags' do
+          before do
+            expect(destroy_method).to receive(:parameters).and_return([[:opt, :options]])
+          end
+
+          it 'skips setting additional destroy flags' do
+            expect(domain).to receive(:destroy).with(destroy_volumes: true)
+            expect(subject.call(env)).to be_nil
+          end
+        end
+      end
+
       context 'when has CDROMs attached' do
         let(:vagrantfile_providerconfig) do
           <<-EOF
@@ -197,7 +230,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
           expect(domain).to receive(:volumes).and_return([root_disk, nil])
           expect(libvirt_domain).to receive(:xml_desc).and_return(domain_xml)
 
-          expect(domain).to_not receive(:destroy).with(destroy_volumes: true)
+          expect(domain).to_not receive(:destroy).with(destroy_volumes: true, flags: 0)
           expect(root_disk).to receive(:destroy)  # root disk remove
           expect(subject.call(env)).to be_nil
         end