Skip to content
This repository has been archived by the owner on Nov 24, 2022. It is now read-only.

Add backingstore parameters #277

Merged
merged 1 commit into from
May 4, 2014
Merged

Conversation

tailhook
Copy link
Contributor

This is mostly a patch from #235, with cloning support removed.

Also I've set -B best by default (the patch in #235 just crashes when no backingstore specified). Let me know if you're not comfortable with this (the lxc by default uses -B dir, but best is more sensible as it uses btrfs/zfs features if available, and fallbacks to dir if not).

def finalize!
@sudo_wrapper = nil if @sudo_wrapper == UNSET_VALUE
@container_name = nil if @container_name == UNSET_VALUE
@backingstore = "none" if @backingstore == UNSET_VALUE
Copy link
Owner

Choose a reason for hiding this comment

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

@tailhook I think you forgot to change this to best as you described on the issue. I have already merged this branch locally, changed to 'best' and push back the changes once I'm done with fixing the build.

@fgrehm fgrehm merged commit a9248cb into fgrehm:master May 4, 2014
fgrehm added a commit that referenced this pull request May 13, 2014
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.

2 participants