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

CSI-VFS Managed Docker Plug-in #1076

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

akutz
Copy link
Member

@akutz akutz commented Oct 16, 2017

This patch adds a Docker managed plug-in for CSI-VFS.

@akutz akutz added the docker label Oct 16, 2017
@akutz akutz self-assigned this Oct 16, 2017
@akutz akutz force-pushed the feature/docker-plug-in-csi-vfs branch from 14263a1 to 6c23076 Compare October 16, 2017 03:13
@codecov-io
Copy link

codecov-io commented Oct 16, 2017

Codecov Report

Merging #1076 into master will increase coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1076       /-   ##
==========================================
  Coverage   34.16%   34.19%    0.03%     
==========================================
  Files          36       36              
  Lines        2901     2901              
==========================================
  Hits          991      992        1     
  Misses       1807     1806       -1     
  Partials      103      103
Impacted Files Coverage Δ
libstorage/api/types/types_localdevices.go 86.3% <0%> ( 1.36%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1a39fd7...a2c233f. Read the comment docs.

@akutz akutz force-pushed the feature/docker-plug-in-csi-vfs branch from 6c23076 to e736937 Compare October 16, 2017 04:26
This patch adds a Docker managed plug-in for CSI-VFS.
@akutz akutz force-pushed the feature/docker-plug-in-csi-vfs branch from e736937 to a2c233f Compare October 16, 2017 06:24
@hpbieker
Copy link

Any idea when this will be merged?


---

<a name="csi-nfs"></a>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix this link?

@clintkitson
Copy link
Member

Hello @hpbieker,

What was your intended use of the VFS plugin? For the most part we are using it for comprehensive end-to-end testing of the CSI plugins without a dependency on a backend storage platform. Any more info here would be great to here.

@hpbieker
Copy link

@clintkitson. I am looking for a way to utilize a cifs network file system for my Docker volumes. I know I can use the local volume driver and mount the cifs file system in each container. However, this means that I will have to include the username and password in the docker-stack.yml file for each service. I was hoping to just define a volume driver that could have predefined the authentication to my cifs file store.

I guess this would be possible by using the vfs driver. Preferably, there is a cifs driver similar to the nfs driver so that the mounting of the cifs drive is handled automatically. I guess it is just a matter of generalization of the nfs driver.

@clintkitson
Copy link
Member

What does "predefined the authentication to my cifs file store" mean?

@hpbieker
Copy link

@clintkitson Sorry for the bad wording. I mean that I do not want to specify a username, password, hostname and path each volume I create. I think this should be a part of the configuration of the volume driver.

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

Successfully merging this pull request may close these issues.

4 participants