⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

[kustomize_deploy] Add retry for wait conditions#3634

Merged
hjensas merged 1 commit intoopenstack-k8s-operators:mainfrom
hjensas:retry-wait-cmd
Feb 3, 2026
Merged

[kustomize_deploy] Add retry for wait conditions#3634
hjensas merged 1 commit intoopenstack-k8s-operators:mainfrom
hjensas:retry-wait-cmd

Conversation

@hjensas
Copy link
Contributor

@hjensas hjensas commented Jan 28, 2026

Resources may not be immediately available in the API after oc apply completes, causing wait commands to fail with NotFound errors. This adds retry logic with 5 attempts and 3-second delays to handle transient errors during resource registration.

Assisted-By: Claude Code/claude-4.5-sonnet

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 28, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign eshulman2 for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

@hjensas hjensas force-pushed the retry-wait-cmd branch 2 times, most recently from c4ad46a to cd9109c Compare January 28, 2026 13:12
evallesp
evallesp previously approved these changes Jan 29, 2026
Resources may not be immediately available in the API after `oc apply`
completes, causing wait commands to fail with NotFound errors. This adds
retry logic with 5 attempts and 3-second delays to handle transient errors
during resource registration.

Assisted-By: Claude Code/claude-4.5-sonnet
Signed-off-by: Harald Jensås <hjensas@redhat.com>
@hjensas hjensas merged commit 0947ae7 into openstack-k8s-operators:main Feb 3, 2026
6 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.

4 participants