Skip to content

Commit

Permalink
ceph: update rook-ceph-mgr-cluster role rules to include PV and SC
Browse files Browse the repository at this point in the history
Since we changed the Rook orchestrator module for Ceph, it now has
to access Storage Classes and Persistent Volumes in the cluster to
gather inventory and create OSDs so we have to make changes to the
rook-ceph-mgr-cluster role so the orchestrator has permission to
access these resources.

Signed-off-by: Joseph Sawaya <jsawaya@redhat.com>
  • Loading branch information
Joseph Sawaya committed Jul 26, 2021
1 parent ce5777e commit 1ddc390
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 0 deletions.
9 changes: 9 additions & 0 deletions cluster/charts/rook-ceph/templates/clusterrole.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -164,6 +164,7 @@ rules:
- configmaps
- nodes
- nodes/proxy
- persistentvolumes
verbs:
- get
- list
Expand All @@ -178,6 +179,14 @@ rules:
- list
- get
- watch
- apiGroups:
- storage.k8s.io
resources:
- storageclasses
verbs:
- get
- list
- watch
---
# Aspects of ceph-mgr that require access to the system namespace
kind: ClusterRole
Expand Down
9 changes: 9 additions & 0 deletions cluster/examples/kubernetes/ceph/common.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -290,6 +290,7 @@ rules:
- configmaps
- nodes
- nodes/proxy
- persistentvolumes
verbs:
- get
- list
Expand All @@ -304,6 +305,14 @@ rules:
- list
- get
- watch
- apiGroups:
- storage.k8s.io
resources:
- storageclasses
verbs:
- get
- list
- watch
---
kind: ClusterRole
apiVersion: rbac.authorization.k8s.io/v1
Expand Down

0 comments on commit 1ddc390

Please sign in to comment.