Merge lp:~allenap/maas/update-acceptance-for-trusty into lp:~maas-committers/maas/trunk

Proposed by Gavin Panella
Status: Merged
Approved by: Gavin Panella
Approved revision: no longer in the source branch.
Merged at revision: 1949
Proposed branch: lp:~allenap/maas/update-acceptance-for-trusty
Merge into: lp:~maas-committers/maas/trunk
Diff against target: 48 lines (+8/-9)
1 file modified
acceptance/Makefile (+8/-9)
To merge this branch: bzr merge lp:~allenap/maas/update-acceptance-for-trusty
Reviewer Review Type Date Requested Status
Gavin Panella (community) Approve
Review via email: mp+206607@code.launchpad.net

Commit message

Update acceptance/Makefile to work with newer versions of the LXC tools.

Description of the change

The way LXC works has changed a bit in Trusty, so I needed to change acceptance/Makefile to work with it.

To post a comment you must log in.
Revision history for this message
Gavin Panella (allenap) wrote :

I'm going to self-approve this. I don't think anyone else cares :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'acceptance/Makefile'
--- acceptance/Makefile 2013-09-03 15:47:06 +0000
+++ acceptance/Makefile 2014-02-16 09:53:49 +0000
@@ -33,14 +33,13 @@
3333
34# The container on which to base ephemerals.34# The container on which to base ephemerals.
35container := maas-$(series)35container := maas-$(series)
36containerdir := /var/lib/lxc/$(container)
3736
38## Convenience definitions.37## Convenience definitions.
3938
40define ephexec39define ephexec
41sudo LC_ALL=C SSH_ASKPASS=$(abspath ubuntupass) setsid \40sudo LC_ALL=C SSH_ASKPASS=$(abspath ubuntupass) setsid \
42 lxc-start-ephemeral -b $(upstream) -o $(container) -u ubuntu -- \41 lxc-start-ephemeral -b $(upstream) -o $(container) -u ubuntu -- \
43 DEBIAN_FRONTEND=noninteractive SUDO_ASKPASS=$(abspath ubuntupass)42 env DEBIAN_FRONTEND=noninteractive SUDO_ASKPASS=$(abspath ubuntupass)
44endef43endef
4544
46define ephexec-make45define ephexec-make
@@ -65,9 +64,14 @@
65 @ls -1 *.deb64 @ls -1 *.deb
66 @touch $@65 @touch $@
6766
68container: $(containerdir)67container: | services sudo
68ifneq ($(shell sudo lxc-ls -1 $(container)),$(container))
69 sudo lxc-create -n $(container) \
70 -f /etc/lxc/default.conf -t ubuntu -- --release $(series)
71 $(abspath update-container) $(container)
72endif
6973
70container-update: $(containerdir) | services sudo74container-update: container | services sudo
71 $(abspath update-container) $(container)75 $(abspath update-container) $(container)
7276
73services:77services:
@@ -124,11 +128,6 @@
124source/debian: | source128source/debian: | source
125 bzr export $@ $(packaging)/debian129 bzr export $@ $(packaging)/debian
126130
127$(containerdir): | services sudo
128 sudo lxc-create -n $(container) \
129 -f /etc/lxc/default.conf -t ubuntu -- --release $(series)
130 $(abspath update-container) $(container)
131
132## Miscellaneous.131## Miscellaneous.
133132
134sudo:133sudo: