From e5ba54102bee4a56d89722907db608adab34eade Mon Sep 17 00:00:00 2001
From: Brent Eagles <beagles@redhat.com>
Date: Wed, 21 Jul 2021 15:13:12 -0230
Subject: [PATCH] Set stop grace periods for designate services

Many of the designate services perform background syncrhonization
tasks and can lag a bit on shutdown, especially on heavily loaded or
large scale systems. Increasing the stop grace periods reduces the
likelihood that the system might get into a non-consistent state or a
state where a great deal of reconciling needs to occur on restart.

Change-Id: If521662eb4c47ada45cf0a9edd0a58db8a1d038d
---
 deployment/designate/designate-api-container-puppet.yaml      | 1 +
 deployment/designate/designate-bind-container.yaml            | 1 +
 deployment/designate/designate-central-container-puppet.yaml  | 1 +
 deployment/designate/designate-mdns-container-puppet.yaml     | 1 +
 deployment/designate/designate-producer-container-puppet.yaml | 1 +
 deployment/designate/designate-sink-container-puppet.yaml     | 1 +
 deployment/designate/designate-worker-container-puppet.yaml   | 1 +
 7 files changed, 7 insertions(+)

diff --git a/deployment/designate/designate-api-container-puppet.yaml b/deployment/designate/designate-api-container-puppet.yaml
index c869a42f65..469035af78 100644
--- a/deployment/designate/designate-api-container-puppet.yaml
+++ b/deployment/designate/designate-api-container-puppet.yaml
@@ -150,6 +150,7 @@ outputs:
             start_order: 15
             image: &designate_api_image {get_param: ContainerDesignateApiImage}
             net: host
+            stop_grace_period: 300
             privileged: false
             restart: always
             volumes:
diff --git a/deployment/designate/designate-bind-container.yaml b/deployment/designate/designate-bind-container.yaml
index 65c07a924e..68c58a4b08 100644
--- a/deployment/designate/designate-bind-container.yaml
+++ b/deployment/designate/designate-bind-container.yaml
@@ -128,6 +128,7 @@ outputs:
             image: {get_param: ContainerDesignateBackendBIND9Image}
             net: host
             privileged: true
+            stop_grace_period: 300
             restart: always
             volumes:
               list_concat:
diff --git a/deployment/designate/designate-central-container-puppet.yaml b/deployment/designate/designate-central-container-puppet.yaml
index dc23109224..90dc2055f4 100644
--- a/deployment/designate/designate-central-container-puppet.yaml
+++ b/deployment/designate/designate-central-container-puppet.yaml
@@ -190,6 +190,7 @@ outputs:
             image: *designate_central_image
             net: host
             privileged: false
+            stop_grace_period: 300
             restart: always
             healthcheck:
               test: /openstack/healthcheck
diff --git a/deployment/designate/designate-mdns-container-puppet.yaml b/deployment/designate/designate-mdns-container-puppet.yaml
index 6814145ea5..d81935e905 100644
--- a/deployment/designate/designate-mdns-container-puppet.yaml
+++ b/deployment/designate/designate-mdns-container-puppet.yaml
@@ -155,6 +155,7 @@ outputs:
             net: host
             privileged: false
             restart: always
+            stop_grace_period: 300
             healthcheck:
               test: /openstack/healthcheck
             volumes:
diff --git a/deployment/designate/designate-producer-container-puppet.yaml b/deployment/designate/designate-producer-container-puppet.yaml
index b75c1fd3d0..a3adc1723d 100644
--- a/deployment/designate/designate-producer-container-puppet.yaml
+++ b/deployment/designate/designate-producer-container-puppet.yaml
@@ -115,6 +115,7 @@ outputs:
             net: host
             privileged: false
             restart: always
+            stop_grace_period: 300
             healthcheck:
               test: /openstack/healthcheck
             volumes:
diff --git a/deployment/designate/designate-sink-container-puppet.yaml b/deployment/designate/designate-sink-container-puppet.yaml
index ccd7c173f1..12bd61d064 100644
--- a/deployment/designate/designate-sink-container-puppet.yaml
+++ b/deployment/designate/designate-sink-container-puppet.yaml
@@ -102,6 +102,7 @@ outputs:
             net: host
             privileged: false
             restart: always
+            stop_grace_period: 300
             healthcheck:
               test: /openstack/healthcheck
             volumes:
diff --git a/deployment/designate/designate-worker-container-puppet.yaml b/deployment/designate/designate-worker-container-puppet.yaml
index bc4c108d52..195904c231 100644
--- a/deployment/designate/designate-worker-container-puppet.yaml
+++ b/deployment/designate/designate-worker-container-puppet.yaml
@@ -132,6 +132,7 @@ outputs:
             image: &designate_worker_image {get_param: ContainerDesignateWorkerImage}
             net: host
             privileged: false
+            stop_grace_period: 300
             restart: always
             healthcheck:
               test: /openstack/healthcheck