From 44422ef6af812e8a69eab2e52ecc6b2a9e8fc455 Mon Sep 17 00:00:00 2001
From: Sean Mooney <work@seanmooney.info>
Date: Wed, 24 Apr 2024 19:49:38 +0100
Subject: [PATCH] Bump keystone probe timeouts

When deploying locally in crc keystone can crashloop
indefintly because the readiness and liveness probe
were set to strictly.

The nova-operator relaxed the probes becuase of tempest failures
in openstack-k8s-operators/nova-operator#381

This change updates the keystone values to match.
---
 pkg/keystone/deployment.go                       | 10 +++++-----
 tests/kuttl/common/assert_sample_deployment.yaml | 10 +++++-----
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/pkg/keystone/deployment.go b/pkg/keystone/deployment.go
index 29897a65..40e98532 100644
--- a/pkg/keystone/deployment.go
+++ b/pkg/keystone/deployment.go
@@ -45,14 +45,14 @@ func Deployment(
 
 	livenessProbe := &corev1.Probe{
 		// TODO might need tuning
-		TimeoutSeconds:      5,
-		PeriodSeconds:       3,
-		InitialDelaySeconds: 3,
+		TimeoutSeconds:      30,
+		PeriodSeconds:       30,
+		InitialDelaySeconds: 5,
 	}
 	readinessProbe := &corev1.Probe{
 		// TODO might need tuning
-		TimeoutSeconds:      5,
-		PeriodSeconds:       5,
+		TimeoutSeconds:      30,
+		PeriodSeconds:       30,
 		InitialDelaySeconds: 5,
 	}
 
diff --git a/tests/kuttl/common/assert_sample_deployment.yaml b/tests/kuttl/common/assert_sample_deployment.yaml
index 50f16cd2..48907f32 100644
--- a/tests/kuttl/common/assert_sample_deployment.yaml
+++ b/tests/kuttl/common/assert_sample_deployment.yaml
@@ -71,10 +71,10 @@ spec:
             path: /v3
             port: 5000
             scheme: HTTP
-          initialDelaySeconds: 3
-          periodSeconds: 3
+          initialDelaySeconds: 5
+          periodSeconds: 30
           successThreshold: 1
-          timeoutSeconds: 5
+          timeoutSeconds: 30
         name: keystone-api
         readinessProbe:
           failureThreshold: 3
@@ -83,9 +83,9 @@ spec:
             port: 5000
             scheme: HTTP
           initialDelaySeconds: 5
-          periodSeconds: 5
+          periodSeconds: 30
           successThreshold: 1
-          timeoutSeconds: 5
+          timeoutSeconds: 30
         resources:
           requests:
             cpu: "1"