-
Notifications
You must be signed in to change notification settings - Fork 62
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
panic: protobuf tag not enough fields in PathExistsResponse.state #159
Comments
Thanks for the info, I was able to reproduce it by following your steps @andyzhangx, from the error it looks like the unmarshalling of the PathExists response is failing, some things I've tried:
Some other things that I'll try:
The logs that I have at the moment:
|
I found out that the protobuf version in SMB driver is somehow not able to unmarshall the response, @andyzhangx please upgrade the github.com/golang/protobuf version to v1.4.3 I also removed the I see in the server logs that the
Client test logs:
|
Maybe some variation of gogo/protobuf#678? |
thanks for the help! issue resolved after migrating to github.com/golang/protobuf v1.4.3 version |
What happened:
related smb code: https://github.com/kubernetes-csi/csi-driver-smb/blob/8ec0d721cf63e93a324689c17868fd433d193afd/pkg/mounter/safe_mounter_windows.go#L221-L224
@mauriciopoppe
cc @jingxu97
What you expected to happen:
How to reproduce it:
.\csi-proxy.exe --kubelet-path C:\Users\xiazhang\go\src\github.com\kubernetes-csi\csi-driver-smb\pkg\smb\ --v 10
TestCreateVolume
which invokes csi-proxyPathExists
Anything else we need to know?:
Environment:
kubectl version
): n/auname -a
):The text was updated successfully, but these errors were encountered: