Skip to content

fix(rbac): Restore listeners/finalizers update permission#384

Merged
NickLarsenNZ merged 2 commits intomainfrom
fix/rbac-finalizers
Apr 16, 2026
Merged

fix(rbac): Restore listeners/finalizers update permission#384
NickLarsenNZ merged 2 commits intomainfrom
fix/rbac-finalizers

Conversation

@NickLarsenNZ
Copy link
Copy Markdown
Member

@NickLarsenNZ NickLarsenNZ commented Apr 16, 2026

Fixes a rule accidentally removed in #380

Without the listeners/finalizers update rule, the listener operator is unable to set metadata.ownerReferences[].blockOwnerDeletion on the Service (or other) resource.

This appears to have only affected OpenShift clusters in our testing because OpenShift enables the OwnerReferencesPermissionEnforcement admission plugin by default.

Without the `listeners/finalizers` `update`  rule, the listener operator is unable to set `metadata.ownerReferences[].blockOwnerDeletion` on the Service (or other) resource.
@NickLarsenNZ NickLarsenNZ self-assigned this Apr 16, 2026
Comment thread deploy/helm/listener-operator/templates/roles.yaml Outdated
Co-authored-by: Nick <10092581+NickLarsenNZ@users.noreply.github.com>
@NickLarsenNZ NickLarsenNZ moved this to Development: Waiting for Review in Stackable Engineering Apr 16, 2026
@NickLarsenNZ NickLarsenNZ requested a review from sbernauer April 16, 2026 12:02
Copy link
Copy Markdown
Member

@sbernauer sbernauer left a comment

Choose a reason for hiding this comment

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

I could imaging there are more OwnerReferencesPermissionEnforcement cases, but who knows

@NickLarsenNZ
Copy link
Copy Markdown
Member Author

I could imaging there are more OwnerReferencesPermissionEnforcement cases, but who knows

I went through the other's, but they didn't have finalizer permissions in the first place.
The closest I found was ZooKeeper (Patch is required to manage a finalizer for znode cleanup before deletion).

@NickLarsenNZ NickLarsenNZ added this pull request to the merge queue Apr 16, 2026
Merged via the queue into main with commit a94d734 Apr 16, 2026
12 checks passed
@NickLarsenNZ NickLarsenNZ deleted the fix/rbac-finalizers branch April 16, 2026 12:36
@sbernauer sbernauer moved this from Development: Waiting for Review to Development: Done in Stackable Engineering Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: Done

Development

Successfully merging this pull request may close these issues.

2 participants