diff --git a/manifests/client.pp b/manifests/client.pp index cb935267..daf03734 100644 --- a/manifests/client.pp +++ b/manifests/client.pp @@ -18,7 +18,7 @@ class sahara::client ( package { 'python-saharaclient': ensure => $package_ensure, name => $::sahara::params::client_package_name, - tag => 'openstack', + tag => ['openstack', 'openstackclient'], } include openstacklib::openstackclient diff --git a/manifests/deps.pp b/manifests/deps.pp index 0c90ac76..9462970d 100644 --- a/manifests/deps.pp +++ b/manifests/deps.pp @@ -47,8 +47,8 @@ class sahara::deps { # will have clients available to create resources. This tag handles the # openstackclient but indirectly since the client is not available in # all catalogs that don't need the client class (like many spec tests) - Package<| tag == 'openstack'|> - ~> Anchor['sahara::service::end'] + Package<| tag == 'openstackclient'|> + -> Anchor['sahara::service::end'] # Installation or config changes will always restart services. Anchor['sahara::install::end'] ~> Anchor['sahara::service::begin'] diff --git a/spec/classes/sahara_client_spec.rb b/spec/classes/sahara_client_spec.rb index 4999b54e..1429b701 100644 --- a/spec/classes/sahara_client_spec.rb +++ b/spec/classes/sahara_client_spec.rb @@ -13,7 +13,7 @@ describe 'sahara::client' do context 'with default parameters' do it { is_expected.to contain_package('python-saharaclient').with( :ensure => 'present', - :tag => 'openstack', + :tag => ['openstack', 'openstackclient'], ) } end @@ -25,7 +25,7 @@ describe 'sahara::client' do it { is_expected.to contain_package('python-saharaclient').with( :ensure => false, :name => platform_params[:client_package_name], - :tag => 'openstack', + :tag => ['openstack', 'openstackclient'], ) } end