From 3b3f3115cbf16024b657946f53998432444a239b Mon Sep 17 00:00:00 2001 From: Emilien Macchi Date: Sun, 2 Feb 2014 11:07:29 +0100 Subject: [PATCH] Ensure port binding is correct for both API & registry - use bind_port for both services (Glance API & Registry) - Ensure we use the right port for registry connection from API service - unit tests Close #136 Signed-off-by: Emilien Macchi --- manifests/image.pp | 3 +++ spec/classes/cloud_image_spec.rb | 34 ++++++++++++++++++-------------- 2 files changed, 22 insertions(+), 15 deletions(-) diff --git a/manifests/image.pp b/manifests/image.pp index 5cc0f50d..a61d0e92 100644 --- a/manifests/image.pp +++ b/manifests/image.pp @@ -85,6 +85,7 @@ class cloud::image( class { 'glance::api': sql_connection => "mysql://${encoded_glance_user}:${encoded_glance_password}@${glance_db_host}/glance", registry_host => $openstack_vip, + registry_port => $ks_glance_registry_internal_port, verbose => $verbose, debug => $debug, auth_host => $ks_keystone_internal_host, @@ -93,6 +94,7 @@ class cloud::image( keystone_user => 'glance', log_facility => 'LOG_LOCAL0', bind_host => $api_eth, + bind_port => $ks_glance_api_internal_port, use_syslog => true } @@ -106,6 +108,7 @@ class cloud::image( keystone_user => 'glance', log_facility => 'LOG_LOCAL0', bind_host => $api_eth, + bind_port => $ks_glance_registry_internal_port, use_syslog => true } diff --git a/spec/classes/cloud_image_spec.rb b/spec/classes/cloud_image_spec.rb index ad01eb94..9dce90f0 100644 --- a/spec/classes/cloud_image_spec.rb +++ b/spec/classes/cloud_image_spec.rb @@ -23,21 +23,22 @@ describe 'cloud::image' do shared_examples_for 'openstack image' do let :params do - { :glance_db_host => '10.0.0.1', - :glance_db_user => 'glance', - :glance_db_password => 'secrete', - :ks_keystone_internal_host => '10.0.0.1', - :ks_glance_internal_host => '10.0.0.1', - :openstack_vip => '10.0.0.42', - :ks_glance_api_internal_port => '9292', - :ks_glance_password => 'secrete', - :rabbit_host => '10.0.0.1', - :rabbit_password => 'secrete', - :rbd_store_user => 'glance', - :rbd_store_pool => 'images', - :debug => true, - :verbose => true, - :api_eth => '10.0.0.1' } + { :glance_db_host => '10.0.0.1', + :glance_db_user => 'glance', + :glance_db_password => 'secrete', + :ks_keystone_internal_host => '10.0.0.1', + :ks_glance_internal_host => '10.0.0.1', + :openstack_vip => '10.0.0.42', + :ks_glance_api_internal_port => '9292', + :ks_glance_registry_internal_port => '9191', + :ks_glance_password => 'secrete', + :rabbit_host => '10.0.0.1', + :rabbit_password => 'secrete', + :rbd_store_user => 'glance', + :rbd_store_pool => 'images', + :debug => true, + :verbose => true, + :api_eth => '10.0.0.1' } end it 'configure glance-api' do @@ -45,6 +46,7 @@ describe 'cloud::image' do :sql_connection => 'mysql://glance:secrete@10.0.0.1/glance', :keystone_password => 'secrete', :registry_host => '10.0.0.42', + :registry_port => '9191', :keystone_tenant => 'services', :keystone_user => 'glance', :verbose => true, @@ -52,6 +54,7 @@ describe 'cloud::image' do :auth_host => '10.0.0.1', :log_facility => 'LOG_LOCAL0', :bind_host => '10.0.0.1', + :bind_port => '9292', :use_syslog => true ) end @@ -67,6 +70,7 @@ describe 'cloud::image' do :auth_host => '10.0.0.1', :log_facility => 'LOG_LOCAL0', :bind_host => '10.0.0.1', + :bind_port => '9191', :use_syslog => true ) end