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

FlexVolume does not work on Windows node #56875

Closed
andyzhangx opened this Issue Dec 6, 2017 · 2 comments

Comments

Projects
None yet
3 participants
@andyzhangx
Member

andyzhangx commented Dec 6, 2017

Is this a BUG REPORT or FEATURE REQUEST?:
/kind bug

What happened:
I have tried FlexVolume on a Windows node, it does not work. I am now working on this feature now.
Below is the FlexVolume initialization error

E1205 09:27:58.705853     644 driver-call.go:225] Failed to unmarshal output for command: init, output: , error: unexpected end of JSON input
W1205 09:27:58.705853     644 driver-call.go:147] FlexVolume: driver call failed: executable: c:\volumeplugins/foo~cifs/cifs, args: [init], error: exec: "c:\\volumeplugins/foo~cifs/cifs": file does not exist, output: 

What you expected to happen:

How to reproduce it (as minimally and precisely as possible):

Anything else we need to know?:

Environment:

  • Kubernetes version (use kubectl version): v1.7.7
  • Cloud provider or hardware configuration:
  • OS (e.g. from /etc/os-release):
  • Kernel (e.g. uname -a):
  • Install tools:
  • Others:

/sig windows
/assign
@kubernetes/sig-windows-feature-requests

@andyzhangx

This comment has been minimized.

Show comment
Hide comment
@andyzhangx

andyzhangx Dec 7, 2017

Member

Detailed steps about how to use flexvolume on Windows with this PR:
https://github.com/andyzhangx/Demo/tree/master/windows/flexvolume

Member

andyzhangx commented Dec 7, 2017

Detailed steps about how to use flexvolume on Windows with this PR:
https://github.com/andyzhangx/Demo/tree/master/windows/flexvolume

@michmike

This comment has been minimized.

Show comment
Hide comment
@michmike

michmike commented Dec 13, 2017

cc: @bsteciuk

k8s-merge-robot added a commit that referenced this issue Dec 16, 2017

Merge pull request #56921 from andyzhangx/flexvolume-windows
Automatic merge from submit-queue (batch tested with PRs 56650, 55813, 56911, 56921, 56871). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

enable flexvolume on Windows node

**What this PR does / why we need it**:
enable flexvolume on Windows node:
current PodVolumeDir is like 
`\var\lib\kubelet\pods\f54c5a74-da63-11e7-b71a-000d3a02c330\volumes\test~hostpath.cmd\flextest`
which is a unix path, with this PR, PodVolumeDir in **windows** kubelet  will be converted into like
`c:\var\lib\kubelet\pods\f54c5a74-da63-11e7-b71a-000d3a02c330\volumes\test~hostpath.cmd\flextest` 

**Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
Fixes #56875

**Special notes for your reviewer**:
Detailed steps about how to use flexvolume on Windows with this PR:
https://github.com/andyzhangx/Demo/tree/master/windows/flexvolume

**Release note**:

```
enable flexvolume on Windows node
```

/sig windows
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment