Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Loosen up wording for OFFLINE expansion #423

Open
gnufied opened this issue Apr 3, 2020 · 0 comments
Open

Loosen up wording for OFFLINE expansion #423

gnufied opened this issue Apr 3, 2020 · 0 comments

Comments

@gnufied
Copy link
Contributor

gnufied commented Apr 3, 2020

Sometimes a CO may not accurately be able to verify if volume is already published to a node or not when calling ControllerExpandVolume RPC call and in that case an ControllerExpandVolume call might be issued to the driver even though driver only supports offline expansion and underlying is in-use.

Currently there is a explicit error condition to report when this happens:

| Volume in use | 9 FAILED_PRECONDITION | Indicates that the volume corresponding to the 
specified `volume_id` could not be expanded because it is currently published on a node but the 
plugin does not have ONLINE expansion capability. | Caller SHOULD ensure that volume is not 
published and retry with exponential back off. |

However the wording of OFFLINE only expansion uses following language:

OFFLINE indicates that volumes currently published and available on a node SHALL NOT be 
expanded via ControllerExpandVolume

Which is kinda problematic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant