Skip to content

Commit 3664c5c

Browse files
author
James DeFelice
committed
spec: remove topology from validate-volume.
Current thinking is that topology of a volume can be discovered via the ListVolumes RPC of the controller and is not strictly required here. We can always add it back later if needed.
1 parent ad053af commit 3664c5c

File tree

3 files changed

+284
-321
lines changed

3 files changed

+284
-321
lines changed

csi.proto

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -644,14 +644,10 @@ message ValidateVolumeCapabilitiesRequest {
644644
// This field is OPTIONAL.
645645
map<string, string> parameters = 4;
646646

647-
// See CreateVolumeRequest.accessibility_requirements.
648-
// This field is OPTIONAL.
649-
TopologyRequirement accessibility_requirements = 5;
650-
651647
// Secrets required by plugin to complete volume validation request.
652648
// This field is OPTIONAL. Refer to the `Secrets Requirements`
653649
// section on how to use this field.
654-
map<string, string> controller_validate_secrets = 6;
650+
map<string, string> controller_validate_secrets = 5;
655651
}
656652

657653
message ValidateVolumeCapabilitiesResponse {
@@ -664,13 +660,9 @@ message ValidateVolumeCapabilitiesResponse {
664660
// This field is REQUIRED.
665661
repeated VolumeCapability volume_capabilities = 2;
666662

667-
// Topology requirements supported by the plugin.
668-
// This field is OPTIONAL.
669-
TopologyRequirement accessibility_requirements = 3;
670-
671663
// The volume creation parameters validated by the plugin.
672664
// This field is OPTIONAL.
673-
map<string, string> parameters = 4;
665+
map<string, string> parameters = 3;
674666
}
675667

676668
// Confirmed indicates to the CO the set of capabilities that the

0 commit comments

Comments
 (0)