Skip to content

Commit e9147b7

Browse files
committed
Clarify that plugin may return OK for ControllerUnpublish if node or volume not found
1 parent c3707a6 commit e9147b7

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

spec.md

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1257,6 +1257,9 @@ This RPC is typically called by the CO when the workload using the volume is bei
12571257

12581258
This operation MUST be idempotent.
12591259
If the volume corresponding to the `volume_id` is not attached to the node corresponding to the `node_id`, the Plugin MUST reply `0 OK`.
1260+
If the volume corresponding to `volume_id` or the node corresponding to
1261+
`node_id` cannot be found by the Plugin and the volume can be safely regarded as
1262+
ControllerUnpublished from the node, the plugin MAY return `0 OK`.
12601263
If this operation failed, or the CO does not know if the operation failed or not, it can choose to call `ControllerUnpublishVolume` again.
12611264

12621265
```protobuf
@@ -1292,8 +1295,8 @@ The CO MUST implement the specified error recovery behavior when it encounters t
12921295

12931296
| Condition | gRPC Code | Description | Recovery Behavior |
12941297
|-----------|-----------|-------------|-------------------|
1295-
| 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. |
1296-
| Node does not exist | 5 NOT_FOUND | Indicates that a node corresponding to the specified `node_id` does not exist. | Caller MUST verify that the `node_id` is correct and that the node is available and has not been terminated or deleted before retrying with exponential backoff. |
1298+
| Volume does not exist and volume not assumed ControllerUnpublished from node | 5 NOT_FOUND | Indicates that a volume corresponding to the specified `volume_id` does not exist and volume not assumed ControllerUnpublished from node. | 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. |
1299+
| Node does not exist and volume not assumed ControllerUnpublished from node | 5 NOT_FOUND | Indicates that a node corresponding to the specified `node_id` does not exist and volume not assumed ControllerUnpublished from node. | Caller MUST verify that the `node_id` is correct and that the node is available and has not been terminated or deleted before retrying with exponential backoff. |
12971300

12981301

12991302
#### `ValidateVolumeCapabilities`

0 commit comments

Comments
 (0)