openstack/puppet-cinder
Code Issues Proposed changes

Use common default value for auth_url

Browse Source 
The auth_url option needs to be set correctly so that the plugin can
connect to identity API. Although this may need to be set in every
actual production deployment, using the consistent default for single
node set up may be useful, instead of setting the option partially.

Change-Id: I7b8108ab1947241078aaf56de587d8eccacb99fe
Signed-off-by: Takashi Kajinami <kajinamit@oss.nttdata.com>
This commit is contained in:
Takashi Kajinami
2025-09-14 20:28:09 +09:00
parent b48d97c26f
commit 20710186ba
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/nova.pp
View File

@@ -55,7 +55,7 @@
#
# [*auth_url*]
# (Optional) Identity service url.
# Defaults to $facts['os_service_default']
# Defaults to 'http://127.0.0.1:5000'
#
# [*username*]
# (Optional) Nova admin username.
@@ -90,7 +90,7 @@ class cinder::nova (
$collect_timing = $facts['os_service_default'],
$split_loggers = $facts['os_service_default'],
$auth_type = 'password',
$auth_url = $facts['os_service_default'],
$auth_url = 'http://127.0.0.1:5000',
$username = 'nova',
$user_domain_name = 'Default',
$project_name = 'services',

2
spec/classes/cinder_nova_spec.rb
View File

@@ -19,7 +19,7 @@ describe 'cinder::nova' do
is_expected.to contain_cinder_config('nova/collect_timing').with_value('<SERVICE DEFAULT>')
is_expected.to contain_cinder_config('nova/split_loggers').with_value('<SERVICE DEFAULT>')
is_expected.to contain_cinder_config('nova/auth_type').with_value('password')
is_expected.to contain_cinder_config('nova/auth_url').with_value('<SERVICE DEFAULT>')
is_expected.to contain_cinder_config('nova/auth_url').with_value('http://127.0.0.1:5000')
is_expected.to contain_cinder_config('nova/username').with_value('nova')
is_expected.to contain_cinder_config('nova/user_domain_name').with_value('Default')
is_expected.to contain_cinder_config('nova/password').with_value('novapass').with_secret(true)