Skip to content

Commit

Permalink
ceph: add missing rights rook-ceph-purge-osd account
Browse files Browse the repository at this point in the history
The purge osd job need to read and update persistentvolumeclaims
While the job doesn't failed with osd and deployments well deleted, the associated pvc is not deleted by the job as it failed to read it

Signed-off-by: n.fraison <n.fraison@criteo.com>
  • Loading branch information
n.fraison committed Sep 27, 2021
1 parent 01bc8ab commit 2b94cee
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion cluster/charts/rook-ceph-cluster/templates/role.yaml
Expand Up @@ -82,7 +82,7 @@ rules:
verbs: ["get", "list", "delete" ]
- apiGroups: [""]
resources: ["persistentvolumeclaims"]
verbs: ["delete"]
verbs: ["get", "update", "delete"]

{{- if .Values.monitoring.enabled }}
---
Expand Down
2 changes: 1 addition & 1 deletion cluster/charts/rook-ceph/templates/role.yaml
Expand Up @@ -188,5 +188,5 @@ rules:
verbs: ["get", "list", "delete" ]
- apiGroups: [""]
resources: ["persistentvolumeclaims"]
verbs: ["delete"]
verbs: ["get", "update", "delete"]
{{- end }}
Expand Up @@ -145,7 +145,7 @@ rules:
verbs: ["get", "list", "delete"]
- apiGroups: [""]
resources: ["persistentvolumeclaims"]
verbs: ["delete"]
verbs: ["get", "update", "delete"]
---
# Allow the osd purge job to run in this namespace
kind: RoleBinding
Expand Down
2 changes: 1 addition & 1 deletion cluster/examples/kubernetes/ceph/common.yaml
Expand Up @@ -1195,7 +1195,7 @@ rules:
verbs: ["get", "list", "delete"]
- apiGroups: [""]
resources: ["persistentvolumeclaims"]
verbs: ["delete"]
verbs: ["get", "update", "delete"]
---
# Allow the osd purge job to run in this namespace
kind: RoleBinding
Expand Down

0 comments on commit 2b94cee

Please sign in to comment.