Skip to content

fix: add deployments/finalizers RBAC permission#6

Open
nenioscio wants to merge 1 commit into
pdb-operator:mainfrom
nenioscio:fix/add-deployment-finalizers-rbac
Open

fix: add deployments/finalizers RBAC permission#6
nenioscio wants to merge 1 commit into
pdb-operator:mainfrom
nenioscio:fix/add-deployment-finalizers-rbac

Conversation

@nenioscio

Copy link
Copy Markdown

Add permission for pdb-operator to manage finalizers on Deployments. This is required to set blockOwnerDeletion on PodDisruptionBudget owner references.

When setting blockOwnerDeletion: true on an ownerReference pointing to a Deployment, Kubernetes requires the ability to manage finalizers on that resource.

Tested on OpenShift 4.19

Fixes: poddisruptionbudgets.policy is forbidden: cannot set blockOwnerDeletion if an ownerReference refers to a resource you can't set finalizers on

Description

Type of change

  • [ x ] Bug fix
  • New feature
  • Breaking change
  • Documentation update

Checklist

  • [ x ] Chart lints successfully (helm lint charts/pdb-operator)
  • Unit tests pass (helm unittest charts/pdb-operator)
  • [ x ] helm template renders correctly
  • [ x ] Commits are signed off (git commit -s)

Add permission for pdb-operator to manage finalizers on Deployments.
This is required to set blockOwnerDeletion on PodDisruptionBudget owner references.

When setting blockOwnerDeletion: true on an ownerReference pointing to a Deployment,
Kubernetes requires the ability to manage finalizers on that resource.

Tested on OpenShift 4.19

Fixes: poddisruptionbudgets.policy is forbidden: cannot set blockOwnerDeletion
if an ownerReference refers to a resource you can't set finalizers on
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant