Skip to content

Use dedicated startup probe with higher failure threshold#606

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
Deydra71:fix-startup-probe-main
Jun 24, 2026
Merged

Use dedicated startup probe with higher failure threshold#606
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
Deydra71:fix-startup-probe-main

Conversation

@Deydra71

Copy link
Copy Markdown
Contributor

The startup probe shared the same configuration as liveness/readiness probes, giving Horizon only ~40s to start. In resource-constrained environments startup exceeds this window causing CrashLoopBackOff.

Introduce formatStartupProbe() with FailureThreshold: 12 (allowing ~120s for startup), matching the pattern used by Cinder, Glance, and Manila operators.

Assisted-by: Claude Opus 4.6

Note: This is a repeated issue seen in SKMO CI runs that's deploying horizon in main region.

The startup probe shared the same configuration as liveness/readiness
probes, giving Horizon only ~40s to start. In resource-constrained
environments startup exceeds this window causing CrashLoopBackOff.

Introduce formatStartupProbe() with FailureThreshold: 12 (allowing
~120s for startup), matching the pattern used by Cinder, Glance, and
Manila operators.

Signed-off-by: Veronika Fisarova <vfisarov@redhat.com>
Assisted-by: Claude Opus 4.6
@Deydra71 Deydra71 requested review from abays and mcgonago June 24, 2026 07:38
@openshift-ci openshift-ci Bot requested a review from ashu-011 June 24, 2026 07:38

@abays abays left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci

openshift-ci Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abays, Deydra71

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot Bot merged commit 016b918 into openstack-k8s-operators:main Jun 24, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants