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

Expanduser for shares as well as additional share properties. #145

Merged
merged 2 commits into from May 29, 2018

Conversation

shuhaowu
Copy link
Contributor

@shuhaowu shuhaowu commented Apr 30, 2018

Allows shares to be specified with ~ on the host.

This allows one to share things like .ssh/.gnupg and other stuff easier into the container.

Additionally, I'm adding an entry to allow the specification of additional disk options as documented by https://github.com/lxc/lxd/blob/master/doc/containers.md#type-disk

@robvdl

@shuhaowu shuhaowu force-pushed the expanduser-share-source-path branch 2 times, most recently from 3b8bcdf to d584992 Compare May 5, 2018 21:56
@shuhaowu
Copy link
Contributor Author

shuhaowu commented May 5, 2018

The error here is due to ansible now requiring python 3.5+. Perhaps we should either fix the version of ansible or stop building for 3.4?

@shuhaowu shuhaowu force-pushed the expanduser-share-source-path branch from d584992 to a276214 Compare May 6, 2018 00:30
This allows one to share things like .ssh/.gnupg and other stuff easier
into the container.
@shuhaowu shuhaowu force-pushed the expanduser-share-source-path branch from a276214 to 82815c6 Compare May 6, 2018 00:31
@codecov-io
Copy link

codecov-io commented May 6, 2018

Codecov Report

Merging #145 into master will decrease coverage by 0.39%.
The diff coverage is 57.14%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #145     +/-   ##
=========================================
- Coverage   91.72%   91.33%   -0.4%     
=========================================
  Files          45       45             
  Lines        1293     1304     +11     
=========================================
+ Hits         1186     1191      +5     
- Misses        107      113      +6
Impacted Files Coverage Δ
lxdock/container.py 79.84% <0%> (-1.61%) ⬇️
lxdock/conf/schema.py 100% <100%> (ø) ⬆️
lxdock/conf/config.py 98.91% <100%> (ø) ⬆️
lxdock/conf/validators.py 95.45% <83.33%> (-4.55%) ⬇️

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 e0a3ebc...e1faae2. Read the comment docs.

@shuhaowu shuhaowu force-pushed the expanduser-share-source-path branch 2 times, most recently from b7206e3 to 1154d2e Compare May 6, 2018 00:47
Any options as exposed by LXD [1] is supported. This allows a mount to
be optional/readonly/etc.

[1]: https://github.com/lxc/lxd/blob/65cec7f2c884ea2490fe9376d10abe8aefa97b84/doc/containers.md#type-disk
@shuhaowu shuhaowu force-pushed the expanduser-share-source-path branch from 1154d2e to e1faae2 Compare May 6, 2018 01:15
@shuhaowu shuhaowu changed the title Expanduser for sharing things Expanduser for shares as well as additional share properties. May 6, 2018
@shuhaowu
Copy link
Contributor Author

Any updates on this? I'm working on a number of features for lxdock and this is going make it difficult to maintain my patches to be upstreamable as there are so many of them.

@robvdl robvdl merged commit 55108bc into lxdock:master May 29, 2018
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.

None yet

3 participants