Skip to content

[kustomize_deploy] wait for all CatalogSources to be READY after applying CRs#3656

Merged
openshift-merge-bot[bot] merged 1 commit intoopenstack-k8s-operators:mainfrom
rebtoor:install-operators-fix
Feb 11, 2026
Merged

[kustomize_deploy] wait for all CatalogSources to be READY after applying CRs#3656
openshift-merge-bot[bot] merged 1 commit intoopenstack-k8s-operators:mainfrom
rebtoor:install-operators-fix

Conversation

@rebtoor
Copy link
Copy Markdown
Contributor

@rebtoor rebtoor commented Feb 4, 2026 •
edited
Loading

Replace pre-apply CatalogSource checks with post-apply approach that
waits for ALL CatalogSources to reach READY state. This prevents
subscription resolution failures caused by transient connectivity
issues with any CatalogSource (including redhat-operators).

Changes:

  • Apply kustomized CRs first, then verify CatalogSource health
  • Dynamically discover and wait for all CatalogSources cluster-wide
  • Use configurable retry/delay variables for consistency
  • Remove redundant installPlanRef assert (until loop already verifies)

@rebtoor rebtoor requested review from a team and holser February 4, 2026 09:38
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Feb 4, 2026

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@rebtoor
Copy link
Copy Markdown
Contributor Author

rebtoor commented Feb 4, 2026

/test all

@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul Bot commented Feb 4, 2026

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/e518df1dd85e48bd8013e1cc19b35f00

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 34m 02s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 19m 07s
❌ cifmw-crc-podified-edpm-baremetal RETRY_LIMIT in 26m 47s
❌ cifmw-crc-podified-edpm-baremetal-minor-update FAILURE in 26m 46s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 36s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 32s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 10s
✔️ cifmw-architecture-validate-hci SUCCESS in 5m 03s
✔️ cifmw-molecule-kustomize_deploy SUCCESS in 4m 21s

@rebtoor
Copy link
Copy Markdown
Contributor Author

rebtoor commented Feb 4, 2026

recheck

holser
holser requested changes Feb 4, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@holser holser left a comment

Choose a reason for hiding this comment

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

Can we set retries\delays to what we have in other tasks? Thank you in advance

Comment thread roles/kustomize_deploy/tasks/install_operators.yml Outdated
@rebtoor
[kustomize_deploy] wait for all CatalogSources to be READY after appl…
28df7f4 
…ying CRs

Replace pre-apply CatalogSource checks with post-apply approach that
waits for ALL CatalogSources to reach READY state. This prevents
subscription resolution failures caused by transient connectivity
issues with any CatalogSource (including redhat-operators).

Changes:
- Apply kustomized CRs first, then verify CatalogSource health
- Dynamically discover and wait for all CatalogSources cluster-wide
- Use configurable retry/delay variables for consistency
- Remove redundant installPlanRef assert (until loop already verifies)

Signed-off-by: Roberto Alfieri <ralfieri@redhat.com>
@rebtoor rebtoor force-pushed the install-operators-fix branch from 240662c to 28df7f4 Compare February 4, 2026 13:31
@rebtoor rebtoor changed the title [kustomize_deploy] wait for openstack-operator-index CatalogSource before subscription [kustomize_deploy] wait for all CatalogSources to be READY after applying CRs Feb 4, 2026
elfiesmelfie
elfiesmelfie reviewed Feb 6, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@elfiesmelfie elfiesmelfie left a comment

Choose a reason for hiding this comment

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

I hit the same issue in a job deploying the uni-gamma VA with day2 scenario.
This PR appears to have resolved the issue for me.

@rebtoor rebtoor marked this pull request as ready for review February 9, 2026 09:34
holser
holser approved these changes Feb 11, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@holser holser left a comment

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 added the lgtm label Feb 11, 2026
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Feb 11, 2026

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by:

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 7c093ac into openstack-k8s-operators:main Feb 11, 2026
13 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@elfiesmelfie elfiesmelfie elfiesmelfie left review comments

@holser holser holser approved these changes

@evallesp evallesp evallesp approved these changes

Assignees

@holser holser

Labels

approved lgtm

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@rebtoor @holser @elfiesmelfie @evallesp