diff --git a/manifests/compute.pp b/manifests/compute.pp index 9ab4401..395612f 100644 --- a/manifests/compute.pp +++ b/manifests/compute.pp @@ -143,8 +143,8 @@ class openstack::compute ( if $multi_host { include keystone::python nova_config { - 'DEFAULT/multi_host': value => 'True'; - 'DEFAULT/send_arp_for_ha': value => 'True'; + 'DEFAULT/multi_host': value => true; + 'DEFAULT/send_arp_for_ha': value => true; } if ! $public_interface { fail('public_interface must be defined for multi host compute nodes') @@ -160,8 +160,8 @@ class openstack::compute ( } else { $enable_network_service = false nova_config { - 'DEFAULT/multi_host': value => 'False'; - 'DEFAULT/send_arp_for_ha': value => 'False'; + 'DEFAULT/multi_host': value => false; + 'DEFAULT/send_arp_for_ha': value => false; } } diff --git a/spec/classes/openstack_compute_spec.rb b/spec/classes/openstack_compute_spec.rb index fb2ef6b..325a876 100644 --- a/spec/classes/openstack_compute_spec.rb +++ b/spec/classes/openstack_compute_spec.rb @@ -50,8 +50,8 @@ describe 'openstack::compute' do :libvirt_type => 'kvm', :vncserver_listen => '127.0.0.2' ) - should contain_nova_config('DEFAULT/multi_host').with( :value => 'False' ) - should contain_nova_config('DEFAULT/send_arp_for_ha').with( :value => 'False' ) + should contain_nova_config('DEFAULT/multi_host').with( :value => false ) + should contain_nova_config('DEFAULT/send_arp_for_ha').with( :value => false ) should_not contain_class('nova::api') should contain_class('nova::network').with({ :enabled => false, @@ -123,8 +123,8 @@ describe 'openstack::compute' do :libvirt_type => 'qemu', :vncserver_listen => '127.0.0.1' ) - should contain_nova_config('DEFAULT/multi_host').with( :value => 'False' ) - should contain_nova_config('DEFAULT/send_arp_for_ha').with( :value => 'False' ) + should contain_nova_config('DEFAULT/multi_host').with( :value => false ) + should contain_nova_config('DEFAULT/send_arp_for_ha').with( :value => false ) should_not contain_class('nova::api') should contain_class('nova::network').with({ :enabled => false, @@ -161,8 +161,8 @@ describe 'openstack::compute' do it 'should configure nova for multi-host' do #should contain_class('keystone::python') - should contain_nova_config('DEFAULT/multi_host').with(:value => 'True') - should contain_nova_config('DEFAULT/send_arp_for_ha').with( :value => 'True') + should contain_nova_config('DEFAULT/multi_host').with(:value => true) + should contain_nova_config('DEFAULT/send_arp_for_ha').with( :value => true) should contain_class('nova::network').with({ 'enabled' => true, 'install_service' => true @@ -229,7 +229,7 @@ describe 'openstack::compute' do end it { - should contain_nova_config('DEFAULT/multi_host').with({ 'value' => 'True'}) + should contain_nova_config('DEFAULT/multi_host').with({ 'value' => true}) should contain_class('nova::api') should contain_class('nova::network').with({ 'enabled' => true,