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

Corrected default volumes root directory for Windows #120

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

Conversation

olljanat
Copy link

I noticed that Linux default volumes root directory was /var/lib/docker-volumes instead of /var/lib/docker/volumes so I corrected that one + added support for Windows version.

@thaJeztah
Copy link
Member

@cpuguy83 PTAL 🤗

@cpuguy83
Copy link
Contributor

I don't know what this value is used for even.
Almost certainly nothing to do with the actual daemon volume dir.

@olljanat
Copy link
Author

I found it when after build https://github.com/ContainX/docker-volume-netshare on Windows and noticed that default value for --basedir looked very odd.

@cpuguy83
Copy link
Contributor

Could you switch use the existing dir layout for both windows and linux (i.e use docker-volumes instead of docker/volumes?

@olljanat olljanat force-pushed the correct-default-volume-root branch from 57c6e90 to ffd79ee Compare January 16, 2020 20:58
@olljanat olljanat changed the title Corrected default volumes root directory for Linux and Windows Corrected default volumes root directory for Windows Jan 16, 2020
@olljanat
Copy link
Author

Ok so it on purpose like that. Updated.

Copy link
Contributor

@cpuguy83 cpuguy83 left a comment

Choose a reason for hiding this comment

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

LGTM

@olljanat olljanat force-pushed the correct-default-volume-root branch from ffd79ee to 44eeae9 Compare January 16, 2020 21:17
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

Successfully merging this pull request may close these issues.

3 participants