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 openshift-ci bot added the approved label Feb 3, 2026
@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

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
Signed-off-by: rabi <ramishra@redhat.com>
Copy link
Contributor

@stuggi stuggi 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
Copy link
Contributor

openshift-ci bot commented Feb 5, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rabi, stuggi

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

@rabi
Copy link
Contributor Author

rabi commented Feb 6, 2026

/test openstack-operator-build-deploy-kuttl-4-18

1 similar comment
@rabi
Copy link
Contributor Author

rabi commented Feb 6, 2026

/test openstack-operator-build-deploy-kuttl-4-18

@lmiccini
Copy link
Contributor

lmiccini commented Feb 6, 2026

/test functional

5 similar comments
@lmiccini
Copy link
Contributor

lmiccini commented Feb 6, 2026

/test functional

@stuggi
Copy link
Contributor

stuggi commented Feb 6, 2026

/test functional

@stuggi
Copy link
Contributor

stuggi commented Feb 6, 2026

/test functional

@rabi
Copy link
Contributor Author

rabi commented Feb 6, 2026

/test functional

@viroel
Copy link
Contributor

viroel commented Feb 6, 2026

/test functional

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 6, 2026

@rabi: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/functional 966d037 link true /test functional

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@stuggi
Copy link
Contributor

stuggi commented Feb 6, 2026

the cluster seems to overloaded. tuning envtest #1798 helped. need to rebase when landed

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