diff options
| author |   Michele Baldessari <michele@acksyn.org> | 2016-10-01 00:10:18 +0200 |
|---|---|---|
| committer |   Emilien Macchi <emilien@redhat.com> | 2016-10-03 18:27:36 +0000 |
| commit | 0bc59a7362ea5c9a01fe8bf521e19a4014577079 (patch) | |
| tree | a4676cbe8167b8244b23ab5b97b0dd2a5f10f75d /manifests/profile/pacemaker/cinder | |
| parent | c08431d41aa5a447ebef669a152143e1274a227d (diff) | |
Fix the timeout for pacemaker systemd resources
Back in the Mitaka cycle via the change If6b43982c958f63bc78ad997400bf1279c23df7e
we made sure that the default start and stop timeouts for pacemaker
systemd resources is 200s (>= twice the default 90s DefaultTimeoutStopSec
in systemd). We did this change by setting puppet resource defaults for
the Pacemaker::Resource::Service class:
Pacemaker::Resource::Service {
op_params => 'start timeout=200s stop timeout=200s',
}
The problem is that after the composable services rework, this does not
work anymore and the pacemaker systemd resources that still exist do not
have these timeouts set.
We want to move away from resource defaults for this because its results
are dependent on the inclusion order which in tripleo is not guaranteed
any longer (https://docs.puppet.com/puppet/latest/reference/lang_scope.html#scope-lookup-rules)
The only services affected in Newton are: cinder-volume,
cinder-backup, manila-share, haproxy. I preferred fixing all the
pacemaker resources because it seems the cleanest and most logical
commit.
Change-Id: If89a95706514e536a7a2949871a0002c79b6046e
Closes-Bug: #1629366
Diffstat (limited to 'manifests/profile/pacemaker/cinder')
| -rw-r--r-- | manifests/profile/pacemaker/cinder/api.pp | 1 | ||||
| -rw-r--r-- | manifests/profile/pacemaker/cinder/backup.pp | 4 | ||||
| -rw-r--r-- | manifests/profile/pacemaker/cinder/scheduler.pp | 1 | ||||
| -rw-r--r-- | manifests/profile/pacemaker/cinder/volume.pp | 4 |
4 files changed, 8 insertions, 2 deletions
diff --git a/manifests/profile/pacemaker/cinder/api.pp b/manifests/profile/pacemaker/cinder/api.pp index d18942d..f25a050 100644 --- a/manifests/profile/pacemaker/cinder/api.pp +++ b/manifests/profile/pacemaker/cinder/api.pp @@ -48,6 +48,7 @@ class tripleo::profile::pacemaker::cinder::api ( if $step >= 5 and $pacemaker_master { pacemaker::resource::service { $::cinder::params::api_service : + op_params => 'start timeout=200s stop timeout=200s', clone_params => 'interleave=true', require => Pacemaker::Resource::Ocf['openstack-core'], } diff --git a/manifests/profile/pacemaker/cinder/backup.pp b/manifests/profile/pacemaker/cinder/backup.pp index 72ec456..63988d6 100644 --- a/manifests/profile/pacemaker/cinder/backup.pp +++ b/manifests/profile/pacemaker/cinder/backup.pp @@ -54,7 +54,9 @@ class tripleo::profile::pacemaker::cinder::backup ( } if $step >= 5 and $pacemaker_master { - pacemaker::resource::service { $::cinder::params::backup_service : } + pacemaker::resource::service { $::cinder::params::backup_service : + op_params => 'start timeout=200s stop timeout=200s', + } } } diff --git a/manifests/profile/pacemaker/cinder/scheduler.pp b/manifests/profile/pacemaker/cinder/scheduler.pp index e25ef54..4bfb89a 100644 --- a/manifests/profile/pacemaker/cinder/scheduler.pp +++ b/manifests/profile/pacemaker/cinder/scheduler.pp @@ -48,6 +48,7 @@ class tripleo::profile::pacemaker::cinder::scheduler ( if $step >= 5 and $pacemaker_master { pacemaker::resource::service { $::cinder::params::scheduler_service : + op_params => 'start timeout=200s stop timeout=200s', clone_params => 'interleave=true', } pacemaker::constraint::base { 'cinder-api-then-cinder-scheduler-constraint': diff --git a/manifests/profile/pacemaker/cinder/volume.pp b/manifests/profile/pacemaker/cinder/volume.pp index 82e2522..46e8b79 100644 --- a/manifests/profile/pacemaker/cinder/volume.pp +++ b/manifests/profile/pacemaker/cinder/volume.pp @@ -53,7 +53,9 @@ class tripleo::profile::pacemaker::cinder::volume ( } if $step >= 5 and $pacemaker_master { - pacemaker::resource::service { $::cinder::params::volume_service : } + pacemaker::resource::service { $::cinder::params::volume_service : + op_params => 'start timeout=200s stop timeout=200s', + } } } |