openstack/puppet-heat
Code Issues Proposed changes

Use new openstackclient tag

Browse Source 
The new openstackclient tag was added so that we can get all resources
about openstack CLI more easily. This adds this tag to heatclient
because the package provides some sub-commands.

Also, the heatclient package is required by heat so updating it should
restart the services.

Change-Id: I4e24408d9c377d0b67d87efa07f07913ef997402
This commit is contained in:
Takashi Kajinami 2023-11-20 01:15:56 +09:00
parent 02599a4e70
commit aae84d93b0
2 changed files with 5 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/client.pp
View File

@ -14,10 +14,12 @@ class heat::client (
include heat::deps
include heat::params
# NOTE(tkajinam): heat-package tag is used because heatclient is required
# by heat
package { 'python-heatclient':
ensure => $ensure,
name => $::heat::params::client_package_name,
tag => 'openstack',
tag => ['openstack', 'openstackclient', 'heat-package'],
}
include openstacklib::openstackclient

15
spec/classes/heat_client_spec.rb
View File

@ -2,18 +2,7 @@ require 'spec_helper'
describe 'heat::client' do
let :params do
{}
end
let :default_params do
{ :package_ensure => 'present' }
end
shared_examples_for 'heat client' do
let :p do
default_params.merge(params)
end
it { is_expected.to contain_class('heat::deps') }
it { is_expected.to contain_class('heat::params') }
@ -21,8 +10,8 @@ describe 'heat::client' do
it 'installs heat client package' do
is_expected.to contain_package('python-heatclient').with(
:name => platform_params[:client_package_name],
:ensure => p[:package_ensure],
:tag => 'openstack'
:ensure => 'present',
:tag => ['openstack', 'openstackclient', 'heat-package']
)
end