Skip to content
This repository has been archived by the owner on Jan 22, 2024. It is now read-only.

Add support for cross-device volumes #48

Closed
wants to merge 1 commit into from

Conversation

mvollrath
Copy link
Contributor

Hard-linking will fail when Docker graph is on an alternate mount point. Fallback to copy whenever it fails.

Fixes #47

@seanbell
Copy link

Why is a symlink not possible? Why must it be hardlink or copy?

@3XX0
Copy link
Member

3XX0 commented Feb 11, 2016

Symlinks are relative to the new container namespace and won't cross its boundaries.
We are actually using them to create the sonames in the container (https://github.com/NVIDIA/nvidia-docker/blob/master/tools/src/nvidia/volumes.go#L189).

@mvollrath mvollrath force-pushed the cross_volume_support branch 2 times, most recently from d076226 to 28efb78 Compare February 16, 2016 19:20
@mvollrath
Copy link
Contributor Author

I've updated this based on feedback in #47

Copy fallback is now only used for volume setup.

@mvollrath mvollrath force-pushed the cross_volume_support branch from 28efb78 to 4bd5e1c Compare February 16, 2016 20:17
Hard-linking will fail when Docker is on an alternate mount point.

Fixes NVIDIA#47
@3XX0
Copy link
Member

3XX0 commented Feb 18, 2016

Merged! Thanks for the PR @mvollrath

@3XX0 3XX0 closed this Feb 18, 2016
@mvollrath
Copy link
Contributor Author

Thanks, but you should remove the vim .swp file and add *.swp to .gitignore to prevent more accidental swapfile commits.

flx42 added a commit that referenced this pull request Feb 24, 2016
@flx42
Copy link
Member

flx42 commented Feb 24, 2016

Ouch, thank you for noticing!

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

Successfully merging this pull request may close these issues.

4 participants