Skip to content

Commit ac7df2d

Browse files
srirooparclaude
andcommitted
OLS-3221 Reduce app server liveness probe failureThreshold to 3
Lowers the liveness probe failureThreshold from 15 to 3, giving the pod 90 seconds to self-heal before Kubernetes restarts it. This enables auto-recovery after PostgreSQL restart once the service-side liveness endpoint checks DB health. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 275d33d commit ac7df2d

3 files changed

Lines changed: 5 additions & 1 deletion

File tree

internal/controller/appserver/assets_test.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1623,6 +1623,8 @@ user_data_collector_config: {}
16231623
Expect(dep.Spec.Template.Spec.Containers[0].LivenessProbe).ToNot(BeNil())
16241624
Expect(dep.Spec.Template.Spec.Containers[0].LivenessProbe.HTTPGet.Port).To(Equal(intstr.FromString("https")))
16251625
Expect(dep.Spec.Template.Spec.Containers[0].LivenessProbe.HTTPGet.Path).To(Equal("/liveness"))
1626+
Expect(dep.Spec.Template.Spec.Containers[0].LivenessProbe.FailureThreshold).To(Equal(int32(3)))
1627+
Expect(dep.Spec.Template.Spec.Containers[0].LivenessProbe.PeriodSeconds).To(Equal(int32(30)))
16261628
Expect(dep.Spec.Template.Spec.Containers[0].ReadinessProbe).ToNot(BeNil())
16271629
Expect(dep.Spec.Template.Spec.Containers[0].ReadinessProbe.HTTPGet.Port).To(Equal(intstr.FromString("https")))
16281630
Expect(dep.Spec.Template.Spec.Containers[0].ReadinessProbe.HTTPGet.Path).To(Equal("/readiness"))

internal/controller/appserver/deployment.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -438,7 +438,7 @@ func GenerateOLSDeployment(r reconciler.Reconciler, cr *olsv1alpha1.OLSConfig) (
438438
InitialDelaySeconds: 30,
439439
PeriodSeconds: 30,
440440
TimeoutSeconds: 30,
441-
FailureThreshold: 15,
441+
FailureThreshold: 3,
442442
},
443443
},
444444
},

internal/controller/appserver/deployment_test.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -273,6 +273,8 @@ var _ = Describe("App server deployment generation", func() {
273273
Expect(dep.Spec.Selector.MatchLabels).To(Equal(utils.GenerateAppServerSelectorLabels()))
274274
Expect(dep.Spec.Template.Spec.Containers[0].LivenessProbe).ToNot(BeNil())
275275
Expect(dep.Spec.Template.Spec.Containers[0].LivenessProbe.HTTPGet.Port).To(Equal(intstr.FromString("https")))
276+
Expect(dep.Spec.Template.Spec.Containers[0].LivenessProbe.FailureThreshold).To(Equal(int32(3)))
277+
Expect(dep.Spec.Template.Spec.Containers[0].LivenessProbe.PeriodSeconds).To(Equal(int32(30)))
276278
Expect(dep.Spec.Template.Spec.Containers[0].ReadinessProbe).ToNot(BeNil())
277279
Expect(dep.Spec.Template.Spec.Containers[0].ReadinessProbe.HTTPGet.Port).To(Equal(intstr.FromString("https")))
278280
})

0 commit comments

Comments
 (0)