From 2c0c9d0749f4112997e29d88259de87a37a7e55f Mon Sep 17 00:00:00 2001 From: Evan Mahoney Date: Wed, 22 Jul 2026 16:39:00 -0400 Subject: [PATCH] Restore registry-server startup probe FailureThreshold to 15 Restores StartupProbe.FailureThreshold from 10 back to 15, returning the startup budget from 100s to 150s. This reverts the threshold portion of 401bfff4df20c97bef08db2dc9fc75bc1879be7a. The 100s budget can be too tight when there are performance constraints or large custom CatalogSources (OCPBUGS-78095). The commit message in 401bfff states the intent was to maintain the ~150s startup time as originally designed, suggesting the reduction was unintentional. Test fixtures updated to reflect recomputed olm.pod-spec-hash values. Co-Authored-By: Claude Sonnet 4.6 (1M context) Signed-off-by: Evan Mahoney --- .../registry/reconciler/reconciler.go | 2 +- .../registry/reconciler/reconciler_test.go | 32 +++++++++---------- 2 files changed, 17 insertions(+), 17 deletions(-) diff --git a/pkg/controller/registry/reconciler/reconciler.go b/pkg/controller/registry/reconciler/reconciler.go index 46081bbb4a..d10d5992e5 100644 --- a/pkg/controller/registry/reconciler/reconciler.go +++ b/pkg/controller/registry/reconciler/reconciler.go @@ -180,7 +180,7 @@ func Pod(source *operatorsv1alpha1.CatalogSource, name, opmImg, utilImage, img s Port: 50051, }, }, - FailureThreshold: 10, + FailureThreshold: 15, PeriodSeconds: 10, TimeoutSeconds: 5, }, diff --git a/pkg/controller/registry/reconciler/reconciler_test.go b/pkg/controller/registry/reconciler/reconciler_test.go index c527842fb6..fc6187a684 100644 --- a/pkg/controller/registry/reconciler/reconciler_test.go +++ b/pkg/controller/registry/reconciler/reconciler_test.go @@ -38,7 +38,7 @@ func TestPodMemoryTarget(t *testing.T) { ObjectMeta: metav1.ObjectMeta{ GenerateName: "test-", Namespace: "testns", - Labels: map[string]string{"olm.pod-spec-hash": "b5nr2RkG6B45bx5Cn63j7B5ybVGjvneXC0AGu7", "olm.managed": "true"}, + Labels: map[string]string{"olm.pod-spec-hash": "cBESV2Miwqi3TX1KELgXItd7j206e4q6b21BnP", "olm.managed": "true"}, Annotations: map[string]string{"cluster-autoscaler.kubernetes.io/safe-to-evict": "true"}, }, Spec: corev1.PodSpec{ @@ -71,7 +71,7 @@ func TestPodMemoryTarget(t *testing.T) { Port: 50051, }, }, - FailureThreshold: 10, + FailureThreshold: 15, PeriodSeconds: 10, TimeoutSeconds: 5, }, @@ -110,7 +110,7 @@ func TestPodMemoryTarget(t *testing.T) { ObjectMeta: metav1.ObjectMeta{ GenerateName: "test-", Namespace: "testns", - Labels: map[string]string{"olm.pod-spec-hash": "62b6r57eNVnmLZo0glbfT9JJHn98HcJzBhZN94", "olm.managed": "true"}, + Labels: map[string]string{"olm.pod-spec-hash": "22OBIOEoFxFjed27eJvvYzqL0TfdXsuapKoxsE", "olm.managed": "true"}, Annotations: map[string]string{"cluster-autoscaler.kubernetes.io/safe-to-evict": "true"}, }, Spec: corev1.PodSpec{ @@ -144,7 +144,7 @@ func TestPodMemoryTarget(t *testing.T) { Port: 50051, }, }, - FailureThreshold: 10, + FailureThreshold: 15, PeriodSeconds: 10, TimeoutSeconds: 5, }, @@ -209,7 +209,7 @@ func TestPodExtractContent(t *testing.T) { ObjectMeta: metav1.ObjectMeta{ GenerateName: "test-", Namespace: "testns", - Labels: map[string]string{"olm.pod-spec-hash": "b5nr2RkG6B45bx5Cn63j7B5ybVGjvneXC0AGu7", "olm.managed": "true"}, + Labels: map[string]string{"olm.pod-spec-hash": "cBESV2Miwqi3TX1KELgXItd7j206e4q6b21BnP", "olm.managed": "true"}, Annotations: map[string]string{"cluster-autoscaler.kubernetes.io/safe-to-evict": "true"}, }, Spec: corev1.PodSpec{ @@ -242,7 +242,7 @@ func TestPodExtractContent(t *testing.T) { Port: 50051, }, }, - FailureThreshold: 10, + FailureThreshold: 15, PeriodSeconds: 10, TimeoutSeconds: 5, }, @@ -285,7 +285,7 @@ func TestPodExtractContent(t *testing.T) { ObjectMeta: metav1.ObjectMeta{ GenerateName: "test-", Namespace: "testns", - Labels: map[string]string{"olm.pod-spec-hash": "7RONLG0R9WhWbcfGQvcwjexvGmZhN0Hyl16mHH", "olm.managed": "true"}, + Labels: map[string]string{"olm.pod-spec-hash": "30fzEky6rWPSaUHUtTN3mbl0SvE2PryXPIr090", "olm.managed": "true"}, Annotations: map[string]string{"cluster-autoscaler.kubernetes.io/safe-to-evict": "true"}, }, Spec: corev1.PodSpec{ @@ -363,7 +363,7 @@ func TestPodExtractContent(t *testing.T) { Port: 50051, }, }, - FailureThreshold: 10, + FailureThreshold: 15, PeriodSeconds: 10, TimeoutSeconds: 5, }, @@ -406,7 +406,7 @@ func TestPodExtractContent(t *testing.T) { ObjectMeta: metav1.ObjectMeta{ GenerateName: "test-", Namespace: "testns", - Labels: map[string]string{"olm.pod-spec-hash": "8GtO2FTxu9llB1icRoWQIinKIVmNcrtivDihGG", "olm.managed": "true"}, + Labels: map[string]string{"olm.pod-spec-hash": "57eMKS6v8du5evWOTeBJEFYx7r5udVxvhEQpLB", "olm.managed": "true"}, Annotations: map[string]string{"cluster-autoscaler.kubernetes.io/safe-to-evict": "true"}, }, Spec: corev1.PodSpec{ @@ -486,7 +486,7 @@ func TestPodExtractContent(t *testing.T) { Port: 50051, }, }, - FailureThreshold: 10, + FailureThreshold: 15, PeriodSeconds: 10, TimeoutSeconds: 5, }, @@ -522,7 +522,7 @@ func TestPodExtractContent(t *testing.T) { ObjectMeta: metav1.ObjectMeta{ GenerateName: "test-", Namespace: "testns", - Labels: map[string]string{"olm.pod-spec-hash": "8MrvWgiEdqXHHX9wiYJTEe8NgdSQYTVQtOnqml", "olm.managed": "true"}, + Labels: map[string]string{"olm.pod-spec-hash": "bfNQa9OTbkeciTpFoCgpKhNENGjVjY61PYAoYA", "olm.managed": "true"}, Annotations: map[string]string{"cluster-autoscaler.kubernetes.io/safe-to-evict": "true"}, }, Spec: corev1.PodSpec{ @@ -555,7 +555,7 @@ func TestPodExtractContent(t *testing.T) { Port: 50051, }, }, - FailureThreshold: 10, + FailureThreshold: 15, PeriodSeconds: 10, TimeoutSeconds: 5, }, @@ -605,7 +605,7 @@ func TestPodExtractContent(t *testing.T) { ObjectMeta: metav1.ObjectMeta{ GenerateName: "test-", Namespace: "testns", - Labels: map[string]string{"olm.pod-spec-hash": "30zwNEAtF3RcZ94bM4MzA64LwQ8ikpEyrQHTd6", "olm.managed": "true"}, + Labels: map[string]string{"olm.pod-spec-hash": "5CPn3qG7EdDS0mOX7AazM9Tt1RBtR6bjjMmmep", "olm.managed": "true"}, Annotations: map[string]string{"cluster-autoscaler.kubernetes.io/safe-to-evict": "true"}, }, Spec: corev1.PodSpec{ @@ -687,7 +687,7 @@ func TestPodExtractContent(t *testing.T) { Port: 50051, }, }, - FailureThreshold: 10, + FailureThreshold: 15, PeriodSeconds: 10, TimeoutSeconds: 5, }, @@ -737,7 +737,7 @@ func TestPodExtractContent(t *testing.T) { ObjectMeta: metav1.ObjectMeta{ GenerateName: "test-", Namespace: "testns", - Labels: map[string]string{"olm.pod-spec-hash": "G21gfJ9Iz3wyOmy5eBn7kFuqBv93zGJP5lFEM", "olm.managed": "true"}, + Labels: map[string]string{"olm.pod-spec-hash": "adSLBZw1o1geKStnSKaiQEubBnN4eqkHabCyHs", "olm.managed": "true"}, Annotations: map[string]string{"cluster-autoscaler.kubernetes.io/safe-to-evict": "true"}, }, Spec: corev1.PodSpec{ @@ -821,7 +821,7 @@ func TestPodExtractContent(t *testing.T) { Port: 50051, }, }, - FailureThreshold: 10, + FailureThreshold: 15, PeriodSeconds: 10, TimeoutSeconds: 5, },