From 4040d421feb9d2912fd2983e00f26a352b907684 Mon Sep 17 00:00:00 2001 From: Ben Swartzlander Date: Thu, 24 Jul 2025 13:02:51 -0400 Subject: [PATCH] Clarify ControllerModifyVolume errors Collapse the INVALID_ARGUMENTS error rows. For ControllerModifyVolume, clarify that INVALID_ARGUMENT is an infeasible error and that the SP has not made any changes, so the CO does not need to retry. --- spec.md | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/spec.md b/spec.md index 5489db64..10cf3d55 100644 --- a/spec.md +++ b/spec.md @@ -1711,8 +1711,7 @@ message ControllerModifyVolumeResponse { | Condition | gRPC Code | Description | Recovery Behavior | |-----------|-----------|-------------|-------------------| -| Parameters not supported | 3 INVALID_ARGUMENT | Indicates that the CO has specified mutable parameters not supported by the volume. | Caller MAY verify mutable parameters. | -| Exceeds capabilities | 3 INVALID_ARGUMENT | Indicates that the CO has specified capabilities not supported by the volume. | Caller MAY verify volume capabilities by calling ValidateVolumeCapabilities and retry with matching capabilities. | +| Parameters not supported | 3 INVALID_ARGUMENT | Indicates that the CO has specified invalid mutable parameter keys or values, or the specified volume cannot support the specified parameters. The SP MUST NOT have applied any modification to the volume as part of this specific call. | A caller may verify volume capabilities by calling ValidateVolumeCapabilities and then retry the request with valid mutable parameters. | | Volume does not exist | 5 NOT_FOUND | Indicates that a volume corresponding to the specified volume_id does not exist. | Caller MUST verify that the volume_id is correct and that the volume is accessible and has not been deleted before retrying with exponential back off. | #### `GetCapacity`