stackforge/puppet-openstack-cloud
Code Issues Proposed changes

Merge pull request #706 from enovance/heat_role

Browse Source 
heat: ensure admin has heat_stack_owner role
This commit is contained in:
Yanis Guenane 2014-10-30 08:24:22 -04:00
commit ee5d018a46
2 changed files with 34 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
manifests/orchestration/engine.pp
View File

@ -23,7 +23,8 @@ class cloud::orchestration::engine(
$ks_heat_password = 'heatpassword', $ks_heat_password = 'heatpassword',
$ks_heat_cfn_public_port = 8000, $ks_heat_cfn_public_port = 8000,
$ks_heat_cloudwatch_public_port = 8003, $ks_heat_cloudwatch_public_port = 8003,
$auth_encryption_key = 'secrete' $auth_encryption_key = 'secrete',
$ks_admin_tenant = 'admin',
) { ) {
include 'cloud::orchestration' include 'cloud::orchestration'
@ -36,4 +37,10 @@ class cloud::orchestration::engine(
heat_watch_server_url => "${ks_heat_public_proto}://${ks_heat_public_host}:${ks_heat_cloudwatch_public_port}" heat_watch_server_url => "${ks_heat_public_proto}://${ks_heat_public_host}:${ks_heat_cloudwatch_public_port}"
} }
# to avoid bug https://bugs.launchpad.net/heat/+bug/1306665
keystone_user_role { "admin@${ks_admin_tenant}":
ensure => present,
roles => 'heat_stack_owner',
}
} }

48
spec/classes/cloud_orchestration_engine_spec.rb
View File

@ -57,33 +57,37 @@ describe 'cloud::orchestration::engine' do
it 'configure heat common' do it 'configure heat common' do
is_expected.to contain_class('heat').with( is_expected.to contain_class('heat').with(
:verbose => true, :verbose => true,
:debug => true, :debug => true,
:log_facility => 'LOG_LOCAL0', :log_facility => 'LOG_LOCAL0',
:use_syslog => true, :use_syslog => true,
:rabbit_userid => 'heat', :rabbit_userid => 'heat',
:rabbit_hosts => ['10.0.0.1'], :rabbit_hosts => ['10.0.0.1'],
:rabbit_password => 'secrete', :rabbit_password => 'secrete',
:keystone_host => '10.0.0.1', :keystone_host => '10.0.0.1',
:keystone_port => '5000', :keystone_port => '5000',
:keystone_protocol => 'http', :keystone_protocol => 'http',
:keystone_password => 'secrete', :keystone_password => 'secrete',
:auth_uri => 'http://10.0.0.1:5000/v2.0', :auth_uri => 'http://10.0.0.1:5000/v2.0',
:keystone_ec2_uri => 'http://10.0.0.1:5000/v2.0/ec2tokens', :keystone_ec2_uri => 'http://10.0.0.1:5000/v2.0/ec2tokens',
:sql_connection => 'mysql://heat:secrete@10.0.0.1/heat?charset=utf8', :sql_connection => 'mysql://heat:secrete@10.0.0.1/heat?charset=utf8',
:log_dir => false :log_dir => false
) )
is_expected.to contain_heat_config('clients/endpoint_type').with('value' => 'internalURL') is_expected.to contain_heat_config('clients/endpoint_type').with('value' => 'internalURL')
end end
it 'configure heat engine' do it 'configure heat engine' do
is_expected.to contain_class('heat::engine').with( is_expected.to contain_class('heat::engine').with(
:enabled => true, :enabled => true,
:auth_encryption_key => 'secrete', :auth_encryption_key => 'secrete',
:heat_metadata_server_url => 'http://10.0.0.1:8000', :heat_metadata_server_url => 'http://10.0.0.1:8000',
:heat_waitcondition_server_url => 'http://10.0.0.1:8000/v1/waitcondition', :heat_waitcondition_server_url => 'http://10.0.0.1:8000/v1/waitcondition',
:heat_watch_server_url => 'http://10.0.0.1:8003' :heat_watch_server_url => 'http://10.0.0.1:8003'
) )
is_expected.to contain_keystone_user_role('admin@admin').with(
:ensure => 'present',
:roles => 'heat_stack_owner',
)
end end
end end