Skip to content

Commit 5dc29fa

Browse files
committed
Fix section for secure_proxy_ssl_header
In Newton nova uses HTTPtoWSGI middleware and secure_proxy_ssl_header is deprecated, however in Mitaka secure_proxy_ssl_header is still used, so its section is fixed by this commit. Change-Id: Ic6ac9f61778f989ebbe5408b7e47176673ced427 Related-Bug: #1576740 Closes-Bug: #1588360
1 parent 373cd5f commit 5dc29fa

File tree

2 files changed

+17
-17
lines changed

2 files changed

+17
-17
lines changed

manifests/api.pp

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -299,21 +299,21 @@
299299
}
300300

301301
nova_config {
302-
'DEFAULT/api_paste_config': value => $api_paste_config;
303-
'DEFAULT/enabled_apis': value => $enabled_apis_real;
304-
'DEFAULT/volume_api_class': value => $volume_api_class;
305-
'DEFAULT/osapi_compute_listen': value => $api_bind_address;
306-
'DEFAULT/metadata_listen': value => $metadata_listen;
307-
'DEFAULT/metadata_listen_port': value => $metadata_listen_port;
308-
'DEFAULT/osapi_compute_listen_port': value => $osapi_compute_listen_port;
309-
'DEFAULT/osapi_volume_listen': value => $api_bind_address;
310-
'DEFAULT/osapi_compute_workers': value => $osapi_compute_workers;
311-
'DEFAULT/metadata_workers': value => $metadata_workers;
312-
'DEFAULT/use_forwarded_for': value => $use_forwarded_for;
313-
'DEFAULT/default_floating_pool': value => $default_floating_pool;
314-
'DEFAULT/fping_path': value => $fping_path;
315-
'osapi_v3/enabled': value => $osapi_v3;
316-
'oslo_middleware/secure_proxy_ssl_header': value => $secure_proxy_ssl_header;
302+
'DEFAULT/api_paste_config': value => $api_paste_config;
303+
'DEFAULT/enabled_apis': value => $enabled_apis_real;
304+
'DEFAULT/volume_api_class': value => $volume_api_class;
305+
'DEFAULT/osapi_compute_listen': value => $api_bind_address;
306+
'DEFAULT/metadata_listen': value => $metadata_listen;
307+
'DEFAULT/metadata_listen_port': value => $metadata_listen_port;
308+
'DEFAULT/osapi_compute_listen_port': value => $osapi_compute_listen_port;
309+
'DEFAULT/osapi_volume_listen': value => $api_bind_address;
310+
'DEFAULT/osapi_compute_workers': value => $osapi_compute_workers;
311+
'DEFAULT/metadata_workers': value => $metadata_workers;
312+
'DEFAULT/use_forwarded_for': value => $use_forwarded_for;
313+
'DEFAULT/default_floating_pool': value => $default_floating_pool;
314+
'DEFAULT/fping_path': value => $fping_path;
315+
'DEFAULT/secure_proxy_ssl_header': value => $secure_proxy_ssl_header;
316+
'osapi_v3/enabled': value => $osapi_v3;
317317
}
318318

319319
if ($neutron_metadata_proxy_shared_secret){

spec/classes/nova_api_spec.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@
6666
is_expected.to contain_nova_config('DEFAULT/metadata_workers').with('value' => '5')
6767
is_expected.to contain_nova_config('DEFAULT/default_floating_pool').with('value' => 'nova')
6868
is_expected.to contain_nova_config('DEFAULT/fping_path').with('value' => '/usr/sbin/fping')
69-
is_expected.to contain_nova_config('oslo_middleware/secure_proxy_ssl_header').with('value' => '<SERVICE DEFAULT>')
69+
is_expected.to contain_nova_config('DEFAULT/secure_proxy_ssl_header').with('value' => '<SERVICE DEFAULT>')
7070
end
7171

7272
it 'do not configure v3 api' do
@@ -146,9 +146,9 @@
146146
is_expected.to contain_nova_config('DEFAULT/osapi_compute_workers').with('value' => '1')
147147
is_expected.to contain_nova_config('DEFAULT/metadata_workers').with('value' => '2')
148148
is_expected.to contain_nova_config('DEFAULT/default_floating_pool').with('value' => 'public')
149+
is_expected.to contain_nova_config('DEFAULT/secure_proxy_ssl_header').with('value' => 'HTTP-X-Forwarded-Proto')
149150
is_expected.to contain_nova_config('neutron/service_metadata_proxy').with('value' => true)
150151
is_expected.to contain_nova_config('neutron/metadata_proxy_shared_secret').with('value' => 'secrete')
151-
is_expected.to contain_nova_config('oslo_middleware/secure_proxy_ssl_header').with('value' => 'HTTP-X-Forwarded-Proto')
152152
end
153153

154154
it 'configure nova api v3' do

0 commit comments

Comments
 (0)