openstack/puppet-octavia
Code Issues Proposed changes

Use correct section for host and port settings

Browse Source 
The module configured the bind_host and bind_port as
part of the api section, which does not exist in config
reference. The correct section should be DEFAULT. With
the setting being in api, the api service starts up with
binding to host 127.0.0.1 instead of 0.0.0.0.

Change-Id: I90218f65cc60c546c7ef43d9f5a464d9e7031431
Closes-Bug: 1650837
This commit is contained in:
Matthew J. Black
2016-12-17 21:38:43 -05:00
parent 1d799fcf00
commit d953d25fe6
2 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
spec/classes/octavia_api_spec.rb
View File

@@ -30,9 +30,9 @@ describe 'octavia::api' do
)
end
it 'configures api' do
is_expected.to contain_octavia_config('api/host').with_value( params[:host] )
is_expected.to contain_octavia_config('api/port').with_value( params[:port] )
it 'configures bind_host and bind_port' do
is_expected.to contain_octavia_config('DEFAULT/host').with_value( params[:host] )
is_expected.to contain_octavia_config('DEFAULT/port').with_value( params[:port] )
end
[{:enabled => true}, {:enabled => false}].each do |param_hash|