diff --git a/manifests/client.pp b/manifests/client.pp index c0d1272c..895091f4 100644 --- a/manifests/client.pp +++ b/manifests/client.pp @@ -36,7 +36,7 @@ class ironic::client ( package { 'python-ironicclient': ensure => $package_ensure, name => $::ironic::params::client_package, - tag => ['openstack', 'ironic-support-package'], + tag => ['openstack', 'openstackclient'], } include openstacklib::openstackclient diff --git a/manifests/deps.pp b/manifests/deps.pp index 30abc6b0..acc51d9f 100644 --- a/manifests/deps.pp +++ b/manifests/deps.pp @@ -82,7 +82,11 @@ class ironic::deps { -> Package<| tag == 'ironic-support-package'|> -> Anchor['ironic-inspector::install::end'] -# Installation or config changes will always restart services. + # openstackclient package is needed by transform + Package<| tag == 'openstackclient'|> + -> Anchor['ironic::config::begin'] + + # Installation or config changes will always restart services. Anchor['ironic::install::end'] ~> Anchor['ironic::service::begin'] Anchor['ironic::config::end'] ~> Anchor['ironic::service::begin'] Anchor['ironic-inspector::install::end'] ~> Anchor['ironic-inspector::service::begin'] diff --git a/spec/classes/ironic_client_spec.rb b/spec/classes/ironic_client_spec.rb index 8766fd7d..82e814ec 100644 --- a/spec/classes/ironic_client_spec.rb +++ b/spec/classes/ironic_client_spec.rb @@ -31,7 +31,7 @@ describe 'ironic::client' do is_expected.to contain_package('python-ironicclient').with( :ensure => 'present', :name => platform_params[:client_package], - :tag => ['openstack', 'ironic-support-package'] + :tag => ['openstack', 'openstackclient'] ) end