Skip to content

Commit da3002f

Browse files
Xin Wangjdef
Xin Wang
authored andcommitted
Clarify idempotent for ControllerExpandVolume and NodeExpandVolume
Add explicit documentation for clarity idempotent for ControllerExpandVolume and NodeExpandVolume Signed-off-by: Xin Wang <[email protected]>
1 parent 09fd6f3 commit da3002f

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

spec.md

+6
Original file line numberDiff line numberDiff line change
@@ -1804,6 +1804,9 @@ The CO MUST implement the specified error recovery behavior when it encounters t
18041804
A Controller plugin MUST implement this RPC call if plugin has `EXPAND_VOLUME` controller capability.
18051805
This RPC allows the CO to expand the size of a volume.
18061806

1807+
This operation MUST be idempotent.
1808+
If a volume corresponding to the specified volume ID is already larger than or equal to the target capacity of the expansion request, the plugin SHOULD reply 0 OK.
1809+
18071810
This call MAY be made by the CO during any time in the lifecycle of the volume after creation if plugin has `VolumeExpansion.ONLINE` capability.
18081811
If plugin has `EXPAND_VOLUME` node capability, then `NodeExpandVolume` MUST be called after successful `ControllerExpandVolume` and `node_expansion_required` in `ControllerExpandVolumeResponse` is `true`.
18091812

@@ -2355,6 +2358,9 @@ The CO MUST implement the specified error recovery behavior when it encounters t
23552358
A Node Plugin MUST implement this RPC call if it has `EXPAND_VOLUME` node capability.
23562359
This RPC call allows CO to expand volume on a node.
23572360

2361+
This operation MUST be idempotent.
2362+
If a volume corresponding to the specified volume ID is already larger than or equal to the target capacity of the expansion request, the plugin SHOULD reply 0 OK.
2363+
23582364
`NodeExpandVolume` ONLY supports expansion of already node-published or node-staged volumes on the given `volume_path`.
23592365

23602366
If plugin has `STAGE_UNSTAGE_VOLUME` node capability then:

0 commit comments

Comments
 (0)