Skip to content

Commit

Permalink
Propose a new role for session data.
Browse files Browse the repository at this point in the history
  • Loading branch information
carlbraganza committed Jul 3, 2023
1 parent 2dee59e commit 690f801
Show file tree
Hide file tree
Showing 2 changed files with 47 additions and 1 deletion.
4 changes: 4 additions & 0 deletions keps/sig-storage/3314-csi-changed-block-tracking/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -341,6 +341,7 @@ individual driver Namespace.
The creation and use of a snapshot session is illustrated in the figure below,
with additional information available in the [Design Details](#design-details) section.

> @TODO Eventually use a PNG - SVG does not render text spaces properly.
![Snapshot Session](./session.drawio.svg)


Expand Down Expand Up @@ -492,7 +493,10 @@ to implement the necessary RBAC policy as illustrated in the following figure:
> in a RoleBinding and in a ClusterRoleBinding?
> I think the ClusterRoleBinding would result in the SA getting access to
> SnapshotSessionData in other CSI driver namespaces.
> I propose a **SnapshotSessionData** ClusterRole that the sidecar's SA can
> use with a RoleBinding to get access to the SnapshotSessionData CR.
> @TODO Eventually use a PNGs - SVG does not render text spaces properly.
![CSI Snapshot Session Roles](./roles.drawio.svg)

- The **SnapshotSessionClient** ClusterRole should be used in a
Expand Down
Loading

0 comments on commit 690f801

Please sign in to comment.