Skip to content

Commit

Permalink
Remove set_host_default from vagrant_libvirt
Browse files Browse the repository at this point in the history
The previous change broke the proper update of /etc/hosts by beaker
which, in turn, broke multi-node testing.

Fixes: voxpupuli#49
  • Loading branch information
trevor-vaughan committed May 25, 2021
1 parent 30b89c8 commit 1cfc4b6
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 7 deletions.
6 changes: 0 additions & 6 deletions lib/beaker/hypervisor/vagrant_libvirt.rb
Original file line number Diff line number Diff line change
Expand Up @@ -29,10 +29,4 @@ def self.build_options(options)

options['libvirt'].map { |k, v| " node.#{k} = '#{v}'" }
end

private

def set_host_default_ip(host)
# In vagrant-libvirt hosts get a management IP, no need for a default
end
end
2 changes: 1 addition & 1 deletion spec/beaker/hypervisor/vagrant_libvirt_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@
end

it "has no private network" do
is_expected.not_to include('v.vm.network :private_network')
is_expected.to include('v.vm.network :private_network')
end

it "can specify the memory as an integer" do
Expand Down

0 comments on commit 1cfc4b6

Please sign in to comment.