Skip to content

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

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

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

Conversation

@NickLarsenNZ

@NickLarsenNZ NickLarsenNZ commented Apr 16, 2026

Copy link
Copy Markdown
Member

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

@sbernauer sbernauer left a comment

Copy link
Copy Markdown
Member

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
@lfrancke lfrancke moved this from Development: Done to Acceptance: In Progress in Stackable Engineering May 6, 2026
@lfrancke lfrancke moved this from Acceptance: In Progress to Done in Stackable Engineering May 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants