Clarify ControllerPublishVolume readonly field

container-storage-interface · Nov 8, 2018 · bb04428 · bb04428
1 parent c62adb3
commit bb04428
Show file tree

Hide file tree

Showing 3 changed files with 83 additions and 77 deletions.
diff --git a/csi.proto b/csi.proto
@@ -588,7 +588,9 @@ message ControllerPublishVolumeRequest {
   VolumeCapability volume_capability = 3;
 
   // Whether to publish the volume in readonly mode. This field is
-  // REQUIRED.
+  // REQUIRED. If an SP does not support this field, SP MUST ignore
+  // the field and handle the request normally (SP SHOULD not error if
+  // SP does not support this field).
   bool readonly = 4;
 
   // Secrets required by plugin to complete controller publish volume