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: I3e075edf9e97995ff9d014f41bea72ad758c72e2 Closes-Bug: #1904962
This commit is contained in:
parent
bd04fe0a99
commit
a90ce49015
|
@ -12,9 +12,14 @@
|
|||
# (Optional) User to run dbsync command.
|
||||
# Defaults to 'tacker'
|
||||
#
|
||||
# [*db_sync_timeout*]
|
||||
# (Optional) Timeout for the execution of the db_sync
|
||||
# Defaults to 300
|
||||
#
|
||||
class tacker::db::sync(
|
||||
$extra_params = '--config-file /etc/tacker/tacker.conf',
|
||||
$user = 'tacker',
|
||||
$db_sync_timeout = 300,
|
||||
) {
|
||||
|
||||
include tacker::deps
|
||||
|
@ -26,6 +31,7 @@ class tacker::db::sync(
|
|||
refreshonly => true,
|
||||
try_sleep => 5,
|
||||
tries => 10,
|
||||
timeout => $db_sync_timeout,
|
||||
logoutput => on_failure,
|
||||
subscribe => [
|
||||
Anchor['tacker::install::end'],
|
||||
|
|
|
@ -0,0 +1,3 @@
|
|||
---
|
||||
features:
|
||||
- Adds db_sync_timeout parameter to db sync.
|
|
@ -14,6 +14,7 @@ describe 'tacker::db::sync' do
|
|||
:refreshonly => 'true',
|
||||
:try_sleep => 5,
|
||||
:tries => 10,
|
||||
:timeout => 300,
|
||||
:logoutput => 'on_failure',
|
||||
:subscribe => ['Anchor[tacker::install::end]',
|
||||
'Anchor[tacker::config::end]',
|
||||
|
@ -27,6 +28,7 @@ describe 'tacker::db::sync' do
|
|||
let :params do
|
||||
{
|
||||
:extra_params => '--config-file /etc/tacker/tacker.conf',
|
||||
:db_sync_timeout => 750,
|
||||
}
|
||||
end
|
||||
|
||||
|
@ -38,6 +40,7 @@ describe 'tacker::db::sync' do
|
|||
:refreshonly => 'true',
|
||||
:try_sleep => 5,
|
||||
:tries => 10,
|
||||
:timeout => 750,
|
||||
:logoutput => 'on_failure',
|
||||
:subscribe => ['Anchor[tacker::install::end]',
|
||||
'Anchor[tacker::config::end]',
|
||||
|
|
Loading…
Reference in New Issue