Merge "Allow customization of db sync command line"

This commit is contained in:
Jenkins 2015-09-23 19:49:33 +00:00 committed by Gerrit Code Review
commit 240301fe01
3 changed files with 49 additions and 1 deletions

View File

@ -1,8 +1,20 @@
#
# Class to execute "keystone-manage db_sync
#
class keystone::db::sync {
# == Parameters
#
# [*extra_params*]
# (optional) String of extra command line parameters to append
# to the keystone-manage db_sync command. These will be
# inserted in the command line between 'keystone-manage' and
# 'db_sync' in the command line.
# Defaults to ''
#
class keystone::db::sync(
$extra_params = undef,
) {
exec { 'keystone-manage db_sync':
command => "keystone-manage ${extra_params} db_sync",
path => '/usr/bin',
user => 'keystone',
refreshonly => true,

View File

@ -0,0 +1,35 @@
require 'spec_helper'
describe 'keystone::db::sync' do
describe 'with only required params' do
it {
is_expected.to contain_exec('keystone-manage db_sync').with(
:command => 'keystone-manage db_sync',
:user => 'keystone',
:refreshonly => true,
:subscribe => ['Package[keystone]', 'Keystone_config[database/connection]'],
:require => 'User[keystone]'
)
}
end
describe "overriding extra_params" do
let :params do
{
:extra_params => '--config-file /etc/keystone/keystone.conf',
}
end
it {
is_expected.to contain_exec('keystone-manage db_sync').with(
:command => 'keystone-manage --config-file /etc/keystone/keystone.conf db_sync',
:user => 'keystone',
:refreshonly => true,
:subscribe => ['Package[keystone]', 'Keystone_config[database/connection]'],
:require => 'User[keystone]'
)
}
end
end

View File

@ -141,6 +141,7 @@ describe 'keystone' do
it 'should synchronize the db if $sync_db is true' do
if param_hash['sync_db']
is_expected.to contain_exec('keystone-manage db_sync').with(
:command => 'keystone-manage db_sync',
:user => 'keystone',
:refreshonly => true,
:subscribe => ['Package[keystone]', 'Keystone_config[database/connection]'],