Make lxd containers ephemeral #578

Merged
merged 1 commit into from Jun 17, 2016

Conversation

Projects
None yet
4 participants

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Member

elopio commented Jun 17, 2016

OK to test

Member

elopio commented Jun 17, 2016

Thanks @evandandrea.

This should have an integration test, but we have an issue pending in there, so I reported https://bugs.launchpad.net/snapcraft/+bug/1593529 for later.

@@ -61,7 +61,7 @@ def _create_container(self):
remote_tmp = petname.Generate(2, '-')
check_call(['lxc', 'remote', 'add', remote_tmp, self._server])
check_call([
- 'lxc', 'launch',
+ 'lxc', 'launch', '-e',
@sergiusens

sergiusens Jun 17, 2016

Collaborator

when did this ever get dropped? :-(

Collaborator

sergiusens commented Jun 17, 2016

Thanks, and terribly sorry for the slow turn around on this one

@elopio elopio merged commit 038bf3f into snapcore:master Jun 17, 2016

4 checks passed

Examples tests Success
Details
autopkgtest Success
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 95.867%
Details

kalikiana pushed a commit to kalikiana/snapcraft that referenced this pull request Apr 6, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment