From 2fc62f9e5eafd61286e0cae01b6e1a8b02df932e Mon Sep 17 00:00:00 2001 From: Dan Bode Date: Wed, 15 May 2013 20:21:46 -0700 Subject: [PATCH] provide a default for db_host This commit adds a reasonable default of 127.0.0.1 to the db_host parameter. I think we should provide reasonable defaults when possible (and I think localhost is reasonable in this case) Change-Id: I078f3726f8fe4e2b79e3e7bc282463d54b3003fd --- manifests/keystone.pp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/manifests/keystone.pp b/manifests/keystone.pp index 8accf0b..d763dcc 100644 --- a/manifests/keystone.pp +++ b/manifests/keystone.pp @@ -5,7 +5,7 @@ # # === Parameters # -# [db_host] Host where DB resides. Required. +# [db_host] Host where DB resides. Optional. Defaults to 127.0.0.1.. # [keystone_db_password] Password for keystone DB. Required. # [keystone_admin_token]. Auth token for keystone admin. Required. # [admin_email] Email address of system admin. Required. @@ -39,7 +39,6 @@ # } class openstack::keystone ( - $db_host, $db_password, $admin_token, $admin_email, @@ -48,8 +47,9 @@ class openstack::keystone ( $nova_user_password, $cinder_user_password, $quantum_user_password, - $swift_user_password = false, $public_address, + $db_host = '127.0.0.1', + $swift_user_password = false, $db_type = 'mysql', $db_user = 'keystone', $db_name = 'keystone',