diff --git a/lib/puppet/type/tempest_glance_id_setter.rb b/lib/puppet/type/tempest_glance_id_setter.rb index b2256cd0..c3700edb 100644 --- a/lib/puppet/type/tempest_glance_id_setter.rb +++ b/lib/puppet/type/tempest_glance_id_setter.rb @@ -20,4 +20,7 @@ Puppet::Type.newtype(:tempest_glance_id_setter) do desc 'name of glance image' end + autorequire(:package) do + ['python-openstackclient'] + end end diff --git a/lib/puppet/type/tempest_neutron_net_id_setter.rb b/lib/puppet/type/tempest_neutron_net_id_setter.rb index a39ef861..b7ac56b5 100644 --- a/lib/puppet/type/tempest_neutron_net_id_setter.rb +++ b/lib/puppet/type/tempest_neutron_net_id_setter.rb @@ -14,4 +14,7 @@ Puppet::Type.newtype(:tempest_neutron_net_id_setter) do desc 'The name of the neutron network.' end + autorequire(:package) do + ['python-openstackclient'] + end end diff --git a/manifests/init.pp b/manifests/init.pp index f727506b..2fecfb03 100644 --- a/manifests/init.pp +++ b/manifests/init.pp @@ -257,6 +257,8 @@ class tempest( include ::tempest::params + include ::openstacklib::openstackclient + if $install_from_source { ensure_packages([ 'git', diff --git a/releasenotes/notes/osclient-fccb30722f2d46a0.yaml b/releasenotes/notes/osclient-fccb30722f2d46a0.yaml new file mode 100644 index 00000000..8aed534f --- /dev/null +++ b/releasenotes/notes/osclient-fccb30722f2d46a0.yaml @@ -0,0 +1,5 @@ +--- +features: + - Installation of python-openstackclient by default and make sure the + package is installed before executing our providers that use OpenStack + APIs. diff --git a/spec/classes/tempest_spec.rb b/spec/classes/tempest_spec.rb index 05d7700d..31e6aad9 100644 --- a/spec/classes/tempest_spec.rb +++ b/spec/classes/tempest_spec.rb @@ -160,6 +160,12 @@ describe 'tempest' do ) end + it 'installs python-openstackclient package' do + is_expected.to contain_package('python-openstackclient').with( + :tag => 'openstack' + ) + end + it 'configure tempest config' do is_expected.to contain_tempest_config('compute/change_password_available').with(:value => nil) is_expected.to contain_tempest_config('compute/flavor_ref').with(:value => nil)