Replace var approach with config_template approach

Instead of having specific variables do adjust the init
behaviour, the more generalised approach of hard setting
sensible defaults in the template and allowing the use
of config_template to override those is being used.

Change-Id: Id74d06cdfa2cf1e4a6270566d67083a50d82f6ac

defaults/main.yml

@@ -311,12 +311,4 @@ cinder_rootwrap_conf_overrides: {}
 cinder_api_paste_ini_overrides: {}
 cinder_cinder_conf_overrides: {}
 
-# Timeout (in seconds) for cinder to wait between SIGTERM and SIGKILL
-# Applies to TimeoutSec in systemd, affecting start and stop times.
-# 'sigkill_timeout' is a global option should one desired.
-cinder_sigkill_timeout: 120
-
-# Number of seconds to wait before restarting the service
-cinder_restart_wait: 5
-
 _UUID_regex: "[0-9a-f]{8}-([0-9a-f]{4}-){3}[0-9a-f]{12}"

releasenotes/notes/cinder_init_time_settings-1ef46f6b9d4fc1df.yaml

@@ -1,19 +1,18 @@
 ---
 features:
-  - The ``cinder_sigkill_timeout`` was introduced to control
+  - For the ``os_cinder`` role, the systemd unit ``TimeoutSec`` value which
-    the time between sending a SIGTERM signal and a SIGKILL signal when
+    controls the time between sending a SIGTERM signal and a SIGKILL signal
-    stopping or restarting services. This can help in letting long-lived
+    when stopping or restarting the service has been reduced from 300 seconds
-    sessions drain while preventing new ones from starting before a
+    to 120 seconds. This provides 2 minutes for long-lived sessions to drain
-    restart. Default is 120 seconds (was harcoded to 300 seconds).
+    while preventing new ones from starting before a restart or a stop. The
-  - The ``cinder_restart_wait`` variable has been added to control the time
+    ``RestartSec`` value which controls the time between the service stop and
-    between restarts of cinder services. Default is 5 seconds (was
+    start when restarting has been reduced from 150 seconds to 2 seconds to
-    hardcoded to 150 seconds).
+    make the restart happen faster. These values can be adjusted by using the
+    ``cinder_*_init_config_overrides`` variables which use the
+    ``config_template`` task to change template defaults.
 upgrade:
-  - The ``cinder_sigkill_timeout`` was introduced to control
+  - The variables ``cinder_sigkill_timeout`` and ``cinder_restart_wait`` have
-    the time between sending a SIGTERM signal and a SIGKILL signal when
+    been removed. The previous default values have now been set in the
-    stopping or restarting services. This can help in letting long-lived
+    template directly and can be adjusted by using the
-    sessions drain while preventing new ones from starting before a
+    ``cinder_*_init_overrides`` variables which use the ``config_template``
-    restart. Default is 120 seconds (was harcoded to 300 seconds).
+    task to change template defaults.
-  - The ``cinder_restart_wait`` variable has been added to control the time
-    between restarts of cinder services. Default is 5 seconds (was
-    hardcoded to 150 seconds).

templates/cinder-systemd-init.j2

@@ -17,9 +17,9 @@ ExecStart={{ cinder_bin }}/{{ item.value.service_name }} {{ program_config_optio
 {% endif %}
 
 # Give a reasonable amount of time for the server to start up/shut down
-TimeoutSec={{ cinder_sigkill_timeout }}
+TimeoutSec=120
 Restart=on-failure
-RestartSec={{ cinder_restart_wait }}
+RestartSec=2
 
 # This creates a specific slice which all services will operate from
 #  The accounting options give us the ability to see resource usage through