openstack
/
puppet-cinder
Code Issues Proposed changes

Clean up direct dependencies on puppetlabs-apache 

This change removes direct reference to some classes in
puppetlabs-apache. Details are explained below.

- The api class doesn't need access to anything defined in
  apache::params

- The following classes are included by the openstacklib::wsgi::apache
  resource type, and current inclusions are just redundant.

Change-Id: I0a71c4b57aa57bf98ada16893c2b038e79166ba7
This commit is contained in:
Takashi Kajinami 2021-12-08 22:13:02 +09:00
parent 4dfd69c5b3
commit 7db6cff3d5
4 changed files with 0 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
manifests/api.pp
View File

@ -199,7 +199,6 @@ Use cinder::nova::region_name instead')
}
} elsif $service_name == 'httpd' {
include apache::params
service { 'cinder-api':
ensure => 'stopped',
name => $::cinder::params::api_service,

5
manifests/wsgi/apache.pp
View File

@ -152,11 +152,6 @@ class cinder::wsgi::apache (
include cinder::deps
include cinder::params
include apache
include apache::mod::wsgi
if $ssl_real {
include apache::mod::ssl
}
::openstacklib::wsgi::apache { 'cinder_wsgi':
bind_host => $bind_host,

4
metadata.json
View File

@ -20,10 +20,6 @@
{
"name": "openstack/oslo",
"version_requirement": ">=20.0.0 <21.0.0"
},
{
"name": "puppetlabs/apache",
"version_requirement": ">=5.0.0"
}
],
"description": "Installs and configures OpenStack Cinder (Block Storage).",

10
spec/classes/cinder_wsgi_apache_spec.rb
View File

@ -4,9 +4,6 @@ describe 'cinder::wsgi::apache' do
shared_examples 'apache serving cinder with mod_wsgi' do
context 'with default parameters' do
it { is_expected.to contain_class('cinder::params') }
it { is_expected.to contain_class('apache') }
it { is_expected.to contain_class('apache::mod::wsgi') }
it { is_expected.to contain_class('apache::mod::ssl') }
it { is_expected.to contain_openstacklib__wsgi__apache('cinder_wsgi').with(
:bind_port => 8776,
:group => 'cinder',
@ -48,9 +45,6 @@ describe 'cinder::wsgi::apache' do
}
end
it { is_expected.to contain_class('cinder::params') }
it { is_expected.to contain_class('apache') }
it { is_expected.to contain_class('apache::mod::wsgi') }
it { is_expected.to_not contain_class('apache::mod::ssl') }
it { is_expected.to contain_openstacklib__wsgi__apache('cinder_wsgi').with(
:bind_host => '10.42.51.1',
:bind_port => 12345,
@ -155,15 +149,11 @@ describe 'cinder::wsgi::apache' do
case facts[:osfamily]
when 'Debian'
{
:httpd_service_name => 'apache2',
:httpd_ports_file => '/etc/apache2/ports.conf',
:wsgi_script_path => '/usr/lib/cgi-bin/cinder',
:wsgi_script_source => '/usr/bin/cinder-wsgi'
}
when 'RedHat'
{
:httpd_service_name => 'httpd',
:httpd_ports_file => '/etc/httpd/conf/ports.conf',
:wsgi_script_path => '/var/www/cgi-bin/cinder',
:wsgi_script_source => '/usr/bin/cinder-wsgi'
}