Skip to content

Commit

Permalink
add secrets field support for ControllerGetVolume Request
Browse files Browse the repository at this point in the history
At present secrets are not part of this request and some
consumers need this field for its operation.: That said,
storage backends like Ceph Cluster require the access to be
achieved with the help of authentication. For such systems this
becomes a requirement to have secrets/credentials be part of
the request CO make for its further operations or fetching the state
from the backend cluster.

Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
  • Loading branch information
humblec committed Jun 16, 2022
1 parent ad238e5 commit 1cf046f
Show file tree
Hide file tree
Showing 3 changed files with 221 additions and 198 deletions.
5 changes: 5 additions & 0 deletions csi.proto
Original file line number Diff line number Diff line change
Expand Up @@ -913,6 +913,11 @@ message ControllerGetVolumeRequest {
// The ID of the volume to fetch current volume information for.
// This field is REQUIRED.
string volume_id = 1;

// Secrets required by plugin to complete controller get volume
// This field is OPTIONAL. Refer to the `Secrets Requirements`
// section on how to use this field.
map<string, string> secrets = 2 [(csi_secret) = true];
}

message ControllerGetVolumeResponse {
Expand Down
Loading

0 comments on commit 1cf046f

Please sign in to comment.