-
Notifications
You must be signed in to change notification settings - Fork 73
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
WiP - CSI Spec 51e48d38 (gRPC Errors) #32
Conversation
541cbb6
to
b356efe
Compare
@akutz gocsi/mock returns error |
@akutz also, the following happened when I run the mock and tries to nodepublishvolume a non-existent volume: Edit : I actually don't know what's causing it.. i thought it was above, but no. Re-checking to make sure i m not sending bad data causing mock to crash. Edit @akutz you are assuming volume has attributes, but it doesn't.
|
Thanks Vlad. I’ll take a look. Pretty sure I know what the problem is. |
For your testing, find the Mock SP’s interceptor list and add the option to the spec validator that requires volumes to exist. |
This patch introduces support for the CSI specification, revision 51e48d38. This is a very important update as it shutters the old CSI error model in favor of the new, standard gRPC error codes.
1a8cfba
to
cf5b8f7
Compare
This patch removes all of the previous CSI helper functions contained in GoCSI. With the new error model the helper functions were largely vestigial. There is one remaining/new function -- PageAllVolumes. This function uses the ListVolumes RPC and paging to return a channel on which all volumes may be received by issuing paged requests to the SP until all volumes have been returned from the server.
cf5b8f7
to
a17c5e0
Compare
This patch introduces support for the CSI specification, revision 51e48d38. This is a very important update as it shutters the old CSI error model in favor of the new, standard gRPC error codes.
This patch also includes new, unified client/server interceptors for validating requests/responses against the CSI specification, handling logging, and transmitting a unique request ID from the client to the server.
Updatecsc