Merge pull request #408 from SeanMooney/Probes

Bump keystone probe timeouts
openstack-k8s-operators · Apr 25, 2024 · 07c6570 · 07c6570
2 parents 9546ece + 44422ef
commit 07c6570
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 10 deletions.
diff --git 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
@@ -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"