Skip to content

[skmo] Resolve Skupper Listener race condition with Certificate creation - #4139

Draft
srac0 wants to merge 3 commits into
openstack-k8s-operators:mainfrom
srac0:fix/skmo-skupper-race-condition
Draft

[skmo] Resolve Skupper Listener race condition with Certificate creation#4139
srac0 wants to merge 3 commits into
openstack-k8s-operators:mainfrom
srac0:fix/skmo-skupper-race-condition

Conversation

@srac0

@srac0 srac0 commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

The Skupper Listener playbook creates a cert-manager Certificate and immediately attempts to create a Listener referencing the TLS secret. In practice, cert-manager can take 5-10 minutes to issue certificates, causing the Listener to enter an Error state when the secret doesn't exist yet. The Skupper controller does not re-reconcile automatically, leaving the Listener permanently broken.

@openshift-ci

openshift-ci Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

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

@openshift-ci

openshift-ci Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

[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 dasm 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

@srac0
srac0 force-pushed the fix/skmo-skupper-race-condition branch from cb965ab to c6a786e Compare August 31, 2026 10:06
srac0 added 2 commits August 31, 2026 17:23
The Skupper Listener playbook creates a cert-manager Certificate and
immediately attempts to create a Listener referencing the TLS secret.
In practice, cert-manager can take 5-10 minutes to issue certificates,
causing the Listener to enter an Error state when the secret doesn't
exist yet. The Skupper controller does not re-reconcile automatically,
leaving the Listener permanently broken.

Signed-off-by: Samuel Rac <srac@redhat.com>
@srac0
srac0 force-pushed the fix/skmo-skupper-race-condition branch 2 times, most recently from b01ee71 to 8ca54aa Compare August 31, 2026 19:43
@centosinfra-prod-github-app

Copy link
Copy Markdown

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://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/7934b24859a2450ca78c0b4f780bcc7c

openstack-k8s-operators-content-provider FAILURE in 7m 30s
⚠️ podified-multinode-edpm-deployment-crc SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ cifmw-crc-podified-edpm-baremetal SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ cifmw-crc-podified-edpm-baremetal-minor-update SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
✔️ cifmw-pod-zuul-files SUCCESS in 5m 06s
✔️ openstack-k8s-operators-content-provider-bootc SUCCESS in 2h 16m 49s
✔️ cifmw-crc-podified-edpm-baremetal-bootc SUCCESS in 1h 38m 05s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 10m 01s
✔️ cifmw-pod-pre-commit SUCCESS in 9m 46s
✔️ cifmw-architecture-validate-hci SUCCESS in 5m 46s
✔️ cifmw-molecule-kustomize_deploy SUCCESS in 6m 01s

@srac0
srac0 force-pushed the fix/skmo-skupper-race-condition branch from 8ca54aa to d167503 Compare September 1, 2026 07:37
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.

1 participant