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

Initial implementation #1

Merged
merged 1 commit into from
Jun 30, 2015
Merged

Initial implementation #1

merged 1 commit into from
Jun 30, 2015

Conversation

lowenna
Copy link
Contributor

@lowenna lowenna commented Jun 30, 2015

Signed-off-by: John Howard jhoward@microsoft.com

Initial implementation.

Signed-off-by: John Howard <jhoward@microsoft.com>
lowenna pushed a commit that referenced this pull request Jun 30, 2015
@lowenna lowenna merged commit 6d2d199 into master Jun 30, 2015
@lowenna lowenna deleted the initial branch July 2, 2015 16:07
dcantah pushed a commit that referenced this pull request May 17, 2021
dcantah added a commit that referenced this pull request Jul 15, 2021
This adds basic directory mount support for job containers. As any path on the host
is already accessible from the container, the concept of volume mounts is a bit funny
for job containers. However, it still makes sense to treat the volume mount point where
the container image is mounted as where most things should be found regarding the container.

The manner in which this is done is by appending the container mount path for the volume to
where the rootfs volume is mounted on the host and then symlinking it.

So:
Container rootfs volume path = "C:\C\123456789abcdefgh\"

Example #1
--------------
{
    "host_path": "C:\mydir"
    "container_path": "\dir\in\container"
}

"C:\mydir" would be symlinked to "C:\C\123456789abcdefgh\dir\in\container"

Example #2
---------------
Drive letters will be stripped
{
    "host_path": "C:\mydir"
    "container_path": "C:\dir\in\container"
}
"C:\mydir" would be symlinked to "C:\C\123456789abcdefgh\dir\in\container"

Signed-off-by: Daniel Canter <dcanter@microsoft.com>
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.

None yet

1 participant