openstack
/
puppet-glance
Code Issues Proposed changes

Revert "glance provider: pick os_region_name from DEFAULT" 

This reverts commit b76fbcbea3.

With 268370457c now merged, we need to go
back to looking for os_region_name in [glance_store] instead of
[DEFAULT].

Change-Id: I197d012387d630ca6fb3a38c48f0603add6c6de1
This commit is contained in:
Colleen Murphy 2015-06-22 17:17:14 -07:00
parent 370af254e4
commit 378523be43
2 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
lib/puppet/provider/glance.rb
View File

@ -16,7 +16,7 @@ class Puppet::Provider::Glance < Puppet::Provider
glance_file['keystone_authtoken']['admin_tenant_name'] and
glance_file['keystone_authtoken']['admin_user'] and
glance_file['keystone_authtoken']['admin_password'] and
glance_file['DEFAULT']['os_region_name']
glance_file['glance_store']['os_region_name']
g = {}
g['auth_host'] = glance_file['keystone_authtoken']['auth_host'].strip
@ -25,7 +25,7 @@ class Puppet::Provider::Glance < Puppet::Provider
g['admin_tenant_name'] = glance_file['keystone_authtoken']['admin_tenant_name'].strip
g['admin_user'] = glance_file['keystone_authtoken']['admin_user'].strip
g['admin_password'] = glance_file['keystone_authtoken']['admin_password'].strip
g['os_region_name'] = glance_file['DEFAULT']['os_region_name'].strip
g['os_region_name'] = glance_file['glance_store']['os_region_name'].strip
# auth_admin_prefix not required to be set.
g['auth_admin_prefix'] = (glance_file['keystone_authtoken']['auth_admin_prefix'] || '').strip
@ -36,14 +36,14 @@ class Puppet::Provider::Glance < Puppet::Provider
glance_file['keystone_authtoken']['admin_tenant_name'] and
glance_file['keystone_authtoken']['admin_user'] and
glance_file['keystone_authtoken']['admin_password'] and
glance_file['DEFAULT']['os_region_name']
glance_file['glance_store']['os_region_name']
g = {}
g['identity_uri'] = glance_file['keystone_authtoken']['identity_uri'].strip
g['admin_tenant_name'] = glance_file['keystone_authtoken']['admin_tenant_name'].strip
g['admin_user'] = glance_file['keystone_authtoken']['admin_user'].strip
g['admin_password'] = glance_file['keystone_authtoken']['admin_password'].strip
g['os_region_name'] = glance_file['DEFAULT']['os_region_name'].strip
g['os_region_name'] = glance_file['glance_store']['os_region_name'].strip
return g
else

2
spec/unit/provider/glance_spec.rb
View File

@ -36,7 +36,7 @@ describe Puppet::Provider::Glance do
'admin_user' => 'user',
'admin_password' => 'pass'
},
'DEFAULT' =>
'glance_store' =>
{
'os_region_name' => 'SomeRegion',
}