Skip to content

Conversation

@rabi
Copy link
Contributor

@rabi rabi commented Feb 3, 2026

When a node is removed from a DataPlane NodeSet, the TLS certificates created for that node were not being cleaned up, leaving orphan Certificate resources in the cluster.

This adds a cleanupStaleCertificates function that runs at the beginning of EnsureTLSCerts to delete certificates for nodes that no longer exist in the NodeSet. It uses the allHostnames map (which reflects current nodes) to identify stale certificates by comparing against the hostname label on existing Certificate CRs.

Jira: OSPRH-26077

@openshift-ci openshift-ci bot requested review from slagle and stuggi February 3, 2026 06:02
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 3, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rabi

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-ci openshift-ci bot added the approved label Feb 3, 2026
When a node is removed from a DataPlane NodeSet, the TLS certificates
created for that node were not being cleaned up, leaving orphan
Certificate resources in the cluster.

This adds a cleanupStaleCertificates function that runs at the beginning
of EnsureTLSCerts to delete certificates for nodes that no longer exist
in the NodeSet. It uses the allHostnames map (which reflects current
nodes) to identify stale certificates by comparing against the hostname
label on existing Certificate CRs.

Jira: OSPRH-26077
Co-authored-by: Cursor <[email protected]>
Signed-off-by: rabi <[email protected]>
@rabi
Copy link
Contributor Author

rabi commented Feb 3, 2026

/test functional

@softwarefactory-project-zuul
Copy link

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/a92c233d21144e86ad9d6b51329980ab

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 19m 38s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 21m 23s
cifmw-crc-podified-edpm-baremetal RETRY_LIMIT in 31m 51s
✔️ openstack-operator-tempest-multinode SUCCESS in 1h 43m 57s

@rabi
Copy link
Contributor Author

rabi commented Feb 4, 2026

/retest-required

@rabi
Copy link
Contributor Author

rabi commented Feb 4, 2026

recheck

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants