diff --git a/manifests/api.pp b/manifests/api.pp index 18ecf71c..92f594c4 100644 --- a/manifests/api.pp +++ b/manifests/api.pp @@ -234,7 +234,11 @@ class glance::api( } if ( $glance::params::api_package_name != $glance::params::registry_package_name ) { - ensure_packages([$glance::params::api_package_name]) + ensure_packages([$glance::params::api_package_name], + { + tag => ['openstack'], + } + ) } Package[$glance::params::api_package_name] -> File['/etc/glance/'] diff --git a/manifests/client.pp b/manifests/client.pp index cbb0f1c0..c2ae0601 100644 --- a/manifests/client.pp +++ b/manifests/client.pp @@ -13,6 +13,7 @@ class glance::client ( package { 'python-glanceclient': ensure => $ensure, name => $::glance::params::client_package_name, + tag => ['openstack'], } } diff --git a/manifests/init.pp b/manifests/init.pp index 200cb811..3bcaeab1 100644 --- a/manifests/init.pp +++ b/manifests/init.pp @@ -21,6 +21,7 @@ class glance( package { $glance::params::api_package_name : ensure => $package_ensure, name => $::glance::params::package_name, + tag => ['openstack'], } } } diff --git a/manifests/registry.pp b/manifests/registry.pp index 83159cb3..05366e9b 100644 --- a/manifests/registry.pp +++ b/manifests/registry.pp @@ -154,7 +154,11 @@ class glance::registry( if ( $glance::params::api_package_name != $glance::params::registry_package_name ) { ensure_packages( [$glance::params::registry_package_name], - { ensure => $package_ensure }) + { + ensure => $package_ensure, + tag => ['openstack'], + } + ) } Package[$glance::params::registry_package_name] -> File['/etc/glance/'] diff --git a/spec/classes/glance_api_spec.rb b/spec/classes/glance_api_spec.rb index b5fd7ad7..7111e562 100644 --- a/spec/classes/glance_api_spec.rb +++ b/spec/classes/glance_api_spec.rb @@ -424,7 +424,9 @@ describe 'glance::api' do end let(:params) { default_params } - it {should contain_package('glance-api')} + it { should contain_package('glance-api').with( + :tag => ['openstack'], + )} end describe 'on RedHat platforms' do @@ -433,7 +435,9 @@ describe 'glance::api' do end let(:params) { default_params } - it { should contain_package('openstack-glance')} + it { should contain_package('openstack-glance').with( + :tag => ['openstack'], + )} end describe 'on unknown platforms' do diff --git a/spec/classes/glance_client_spec.rb b/spec/classes/glance_client_spec.rb index e00e29a9..158b23fb 100644 --- a/spec/classes/glance_client_spec.rb +++ b/spec/classes/glance_client_spec.rb @@ -6,7 +6,8 @@ describe 'glance::client' do it { should contain_class('glance::params') } it { should contain_package('python-glanceclient').with( :name => 'python-glanceclient', - :ensure => 'present' + :ensure => 'present', + :tag => ['openstack'], ) } end diff --git a/spec/classes/glance_registry_spec.rb b/spec/classes/glance_registry_spec.rb index 310863e3..f1b666d5 100644 --- a/spec/classes/glance_registry_spec.rb +++ b/spec/classes/glance_registry_spec.rb @@ -342,8 +342,11 @@ describe 'glance::registry' do # separate package for glance registry. ['present', 'latest'].each do |package_ensure| context "with package_ensure '#{package_ensure}'" do - let(:params) { default_params.merge({ :package_ensure => package_ensure })} - it {should contain_package('glance-registry').with_ensure(package_ensure)} + let(:params) { default_params.merge({ :package_ensure => package_ensure }) } + it { should contain_package('glance-registry').with( + :ensure => package_ensure, + :tag => ['openstack'] + )} end end end diff --git a/spec/classes/glance_spec.rb b/spec/classes/glance_spec.rb index 9af348a2..43d6e70a 100644 --- a/spec/classes/glance_spec.rb +++ b/spec/classes/glance_spec.rb @@ -40,7 +40,7 @@ describe 'glance' do end let(:params) { default_params } - it {should_not contain_package('glance')} + it { should_not contain_package('glance') } end describe 'on RedHat platforms' do @@ -49,7 +49,9 @@ describe 'glance' do end let(:params) { default_params } - it { should contain_package('openstack-glance')} + it { should contain_package('openstack-glance').with( + :tag => ['openstack'], + )} end end