From c775184bddffdb90856a9f53e1613a0a8897c689 Mon Sep 17 00:00:00 2001
From: Gage Hugo <gagehugo@gmail.com>
Date: Fri, 18 Jun 2021 19:12:15 -0500
Subject: [PATCH] Set reasonable default probe timeouts

This change modifies the default values for all of the readiness
and liveness probes to something a bit less aggressive, namely
the default timeout of 1 second.

Change-Id: Ib389aebb2450f8ed134ef8f75110b559d1a4f2ee
---
 nova/Chart.yaml              |  2 +-
 nova/values.yaml             | 24 ++++++++++++++++++++++--
 releasenotes/notes/nova.yaml |  1 +
 3 files changed, 24 insertions(+), 3 deletions(-)

diff --git a/nova/Chart.yaml b/nova/Chart.yaml
index f80d483364..46846f95ba 100644
--- a/nova/Chart.yaml
+++ b/nova/Chart.yaml
@@ -14,7 +14,7 @@ apiVersion: v1
 appVersion: v1.0.0
 description: OpenStack-Helm Nova
 name: nova
-version: 0.2.4
+version: 0.2.5
 home: https://docs.openstack.org/nova/latest/
 icon: https://www.openstack.org/themes/openstack/images/project-mascots/Nova/OpenStack_Project_Nova_vertical.png
 sources:
diff --git a/nova/values.yaml b/nova/values.yaml
index 7f51758a17..53e86126df 100644
--- a/nova/values.yaml
+++ b/nova/values.yaml
@@ -2252,18 +2252,28 @@ pod:
           enabled: True
           params:
             initialDelaySeconds: 30
+            periodSeconds: 60
+            timeoutSeconds: 15
         readiness:
           enabled: True
           params:
+            initialDelaySeconds: 30
+            periodSeconds: 60
+            timeoutSeconds: 15
     api-osapi:
       default:
         liveness:
           enabled: True
           params:
             initialDelaySeconds: 30
+            periodSeconds: 60
+            timeoutSeconds: 15
         readiness:
           enabled: True
           params:
+            initialDelaySeconds: 30
+            periodSeconds: 60
+            timeoutSeconds: 15
     conductor:
       default:
         liveness:
@@ -2298,22 +2308,28 @@ pod:
           enabled: True
           params:
             initialDelaySeconds: 30
+            periodSeconds: 60
+            timeoutSeconds: 15
         readiness:
           enabled: True
           params:
             initialDelaySeconds: 30
+            periodSeconds: 60
+            timeoutSeconds: 15
     placement:
       default:
         liveness:
           enabled: True
           params:
             initialDelaySeconds: 50
-            periodSeconds: 10
+            periodSeconds: 30
+            timeoutSeconds: 10
         readiness:
           enabled: True
           params:
             initialDelaySeconds: 15
-            periodSeconds: 10
+            periodSeconds: 30
+            timeoutSeconds: 10
     scheduler:
       default:
         liveness:
@@ -2334,10 +2350,14 @@ pod:
           enabled: True
           params:
             initialDelaySeconds: 30
+            periodSeconds: 60
+            timeoutSeconds: 15
         readiness:
           enabled: True
           params:
             initialDelaySeconds: 30
+            periodSeconds: 60
+            timeoutSeconds: 15
   security_context:
     nova:
       pod:
diff --git a/releasenotes/notes/nova.yaml b/releasenotes/notes/nova.yaml
index daa4de882e..55f05c4b3c 100644
--- a/releasenotes/notes/nova.yaml
+++ b/releasenotes/notes/nova.yaml
@@ -25,4 +25,5 @@ nova:
   - 0.2.2 Adding rabbitmq TLS logic
   - 0.2.3 Replace deprecated configuration ``[vnc]/vncserver_proxyclient_address``
   - 0.2.4 Mount rabbitmq TLS secret
+  - 0.2.5 Set reasonable default probe timeouts
 ...