diff --git a/attributes/default.rb b/attributes/default.rb index 9ea9c08..b1475d6 100644 --- a/attributes/default.rb +++ b/attributes/default.rb @@ -42,7 +42,7 @@ default['openstack']['block-storage']['lock_path'] = '/var/lock/cinder' # Default notification_driver default['openstack']['block-storage']['notification_driver'] = 'cinder.openstack.common.notifier.rpc_notifier' # Availability zone/region for the OpenStack Block-Storage service -default['openstack']['block-storage']['region'] = 'RegionOne' +default['openstack']['block-storage']['region'] = node['openstack']['region'] default['openstack']['block-storage']['scheduler_role'] = 'os-block-storage-scheduler' # Template strings to be used to generate resource names diff --git a/spec/identity_registration_spec.rb b/spec/identity_registration_spec.rb index 48d9743..fdf9dfd 100644 --- a/spec/identity_registration_spec.rb +++ b/spec/identity_registration_spec.rb @@ -31,6 +31,22 @@ describe 'openstack-block-storage::identity_registration' do ) end + it 'overrides cinder volume service region' do + @chef_run = ::ChefSpec::Runner.new ::UBUNTU_OPTS do |n| + n.set['openstack']['block-storage']['region'] = 'serviceRegion' + end + @chef_run.converge 'openstack-block-storage::identity_registration' + resource = @chef_run.find_resource( + 'openstack-identity_register', + 'Register Cinder Volume Service' + ).to_hash + + expect(resource).to include( + endpoint_region: 'serviceRegion', + action: [:create_service] + ) + end + it 'registers cinder volume endpoint' do resource = @chef_run.find_resource( 'openstack-identity_register', @@ -51,6 +67,22 @@ describe 'openstack-block-storage::identity_registration' do ) end + it 'overrides cinder volume endpoint region' do + @chef_run = ::ChefSpec::Runner.new ::UBUNTU_OPTS do |n| + n.set['openstack']['block-storage']['region'] = 'volumeRegion' + end + @chef_run.converge 'openstack-block-storage::identity_registration' + resource = @chef_run.find_resource( + 'openstack-identity_register', + 'Register Cinder Volume Endpoint' + ).to_hash + + expect(resource).to include( + endpoint_region: 'volumeRegion', + action: [:create_endpoint] + ) + end + it 'registers service user' do resource = @chef_run.find_resource( 'openstack-identity_register',