Allow db sync timeouts to be configurable
As Openstack projects continue to have longer database migration chains, the Puppet default timeout of 300 seconds for an execution is becoming too short a duration on some hardware, leading to timeouts. As projects continue to add more migration scripts without pruning the base, timeouts will continue to become more frequent unless this time can be expanded. Change-Id: I0479e0ef6262d6753d199c12fb0dc98069dd32fe Closes-Bug: #1904962
This commit is contained in:
parent
982102a718
commit
405eac4390
|
@ -10,8 +10,13 @@
|
|||
# 'database sync'.
|
||||
# Defaults to undef
|
||||
#
|
||||
# [*db_sync_timeout*]
|
||||
# (Optional) Timeout for the execution of the db_sync
|
||||
# Defaults to 300
|
||||
#
|
||||
class designate::db::sync(
|
||||
$extra_params = undef,
|
||||
$extra_params = undef,
|
||||
$db_sync_timeout = 300,
|
||||
) {
|
||||
|
||||
include designate::deps
|
||||
|
@ -23,6 +28,7 @@ class designate::db::sync(
|
|||
refreshonly => true,
|
||||
try_sleep => 5,
|
||||
tries => 10,
|
||||
timeout => $db_sync_timeout,
|
||||
logoutput => on_failure,
|
||||
subscribe => [
|
||||
Anchor['designate::install::end'],
|
||||
|
|
|
@ -0,0 +1,3 @@
|
|||
---
|
||||
features:
|
||||
- Adds db_sync_timeout parameter to db sync.
|
|
@ -18,6 +18,7 @@ describe 'designate::db::sync' do
|
|||
:refreshonly => 'true',
|
||||
:try_sleep => 5,
|
||||
:tries => 10,
|
||||
:timeout => 300,
|
||||
:logoutput => 'on_failure',
|
||||
:subscribe => ['Anchor[designate::install::end]',
|
||||
'Anchor[designate::config::end]',
|
||||
|
@ -31,7 +32,8 @@ describe 'designate::db::sync' do
|
|||
context 'with parameter overrides' do
|
||||
let :params do
|
||||
{
|
||||
:extra_params => '--config-file /etc/designate/designate.conf'
|
||||
:extra_params => '--config-file /etc/designate/designate.conf',
|
||||
:db_sync_timeout => 750,
|
||||
}
|
||||
end
|
||||
it 'runs designate manage with diffent config' do
|
||||
|
@ -42,6 +44,7 @@ describe 'designate::db::sync' do
|
|||
:refreshonly => 'true',
|
||||
:try_sleep => 5,
|
||||
:tries => 10,
|
||||
:timeout => 750,
|
||||
:logoutput => 'on_failure',
|
||||
:subscribe => ['Anchor[designate::install::end]',
|
||||
'Anchor[designate::config::end]',
|
||||
|
|
Loading…
Reference in New Issue