Merge lp:~vorlon/livecd-rootfs/ubuntu-image-improvements into lp:livecd-rootfs

Proposed by Steve Langasek
Status: Merged
Merged at revision: 1578
Proposed branch: lp:~vorlon/livecd-rootfs/ubuntu-image-improvements
Merge into: lp:livecd-rootfs
Diff against target: 85 lines (+30/-7)
3 files modified
debian/changelog (+12/-0)
live-build/auto/build (+8/-6)
live-build/auto/config (+10/-1)
To merge this branch: bzr merge lp:~vorlon/livecd-rootfs/ubuntu-image-improvements
Reviewer Review Type Date Requested Status
Adam Conrad (community) Approve
Canonical Foundations Team Pending
Review via email: mp+333411@code.launchpad.net
To post a comment you must log in.
1580. By Steve Langasek

Name the manifest the way ubuntu-cdimage expects it for subarchs

Revision history for this message
Adam Conrad (adconrad) wrote :

Approve of the general concept, need-fixing on implementation nitpickery, see inline.

review: Needs Fixing
1581. By Steve Langasek

move configuration bits into live-build/auto/config from live-build/auto/build, per Adam

1582. By Steve Langasek

Change ubuntu-core image builds to use the standard "$PREFIX" naming for
artifacts instead of being gratuitously different; requires a matching
change to ubuntu-cdimage.

1583. By Steve Langasek

fix wrong use of PREFIX literal

Revision history for this message
Adam Conrad (adconrad) wrote :

Two random discussion points inline that shouldn't block the MP. +1

review: Approve
Revision history for this message
Adam Conrad (adconrad) :
Revision history for this message
Steve Langasek (vorlon) :

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2017-11-06 19:12:58 +0000
3+++ debian/changelog 2017-11-10 00:13:40 +0000
4@@ -1,3 +1,15 @@
5+livecd-rootfs (2.481) UNRELEASED; urgency=medium
6+
7+ * The target for x86 ubuntu-core images is VMs, so create them with
8+ a buffer of free space in order to work if booted in place.
9+ * ubuntu-image now produces a manifest file listing the snap revisions
10+ used; publish this so that it's picked up by launchpad-buildd.
11+ * Change ubuntu-core image builds to use the standard "$PREFIX" naming for
12+ artifacts instead of being gratuitously different; requires a matching
13+ change to ubuntu-cdimage.
14+
15+ -- Steve Langasek <steve.langasek@ubuntu.com> Wed, 08 Nov 2017 12:01:25 -0800
16+
17 livecd-rootfs (2.480) bionic; urgency=medium
18
19 [ Gary Wang ]
20
21=== modified file 'live-build/auto/build'
22--- live-build/auto/build 2017-11-06 12:25:57 +0000
23+++ live-build/auto/build 2017-11-10 00:13:40 +0000
24@@ -15,12 +15,17 @@
25 exit 1
26 fi
27
28+# Link output files somewhere BuildLiveCD will be able to find them.
29+PREFIX="livecd.$PROJECT${SUBARCH:+-$SUBARCH}"
30+
31 if [ "${IMAGEFORMAT:-}" = "ubuntu-image" ]; then
32 # Use ubuntu-image instead of live-build
33+
34 env SNAPPY_STORE_NO_CDN=1 \
35- ubuntu-image -c edge \
36- -o livecd."$PROJECT".img livecd."$PROJECT".model-assertion
37- xz -0 -T4 livecd."$PROJECT".img
38+ ubuntu-image -c edge $UBUNTU_IMAGE_ARGS \
39+ -o "$PREFIX".img "$PREFIX".model-assertion
40+ xz -0 -T4 "$PREFIX".img
41+ mv seed.manifest "$PREFIX".manifest
42 exit 0
43 fi
44
45@@ -443,9 +448,6 @@
46 exit 1
47 fi
48
49-# Link output files somewhere BuildLiveCD will be able to find them.
50-PREFIX="livecd.$PROJECT${SUBARCH:+-$SUBARCH}"
51-
52 case $LB_INITRAMFS in
53 casper)
54 INITFS="casper"
55
56=== modified file 'live-build/auto/config'
57--- live-build/auto/config 2017-11-06 12:25:57 +0000
58+++ live-build/auto/config 2017-11-10 00:13:40 +0000
59@@ -90,6 +90,7 @@
60 PREINSTALLED=false
61 PREINSTALL_POOL=
62 PREINSTALL_POOL_SEEDS=
63+PREFIX="livecd.$PROJECT${SUBARCH:+-$SUBARCH}"
64
65 CHROOT_HOOKS=
66 BINARY_HOOKS=
67@@ -171,9 +172,17 @@
68 exit 1
69 ;;
70 esac
71+ case $MODEL in
72+ pc-amd64|pc-i386)
73+ UBUNTU_IMAGE_ARGS="--image-size 3700M" ;;
74+ *)
75+ UBUNTU_IMAGE_ARGS="" ;;
76+ esac
77+
78 echo "IMAGEFORMAT=$IMAGEFORMAT" >> config/common
79+ echo "UBUNTU_IMAGE_ARGS=\"$UBUNTU_IMAGE_ARGS\"" >> config/common
80 # Store model assertion in top dir to get it picked up later as a build artifact
81- env SNAPPY_STORE_NO_CDN=1 snap known --remote model series=16 model="$MODEL" brand-id=canonical > livecd."$PROJECT".model-assertion
82+ env SNAPPY_STORE_NO_CDN=1 snap known --remote model series=16 model="$MODEL" brand-id=canonical > "$PREFIX".model-assertion
83 echo "Configured ubuntu-image for the following model assertion:"
84 cat livecd."$PROJECT".model-assertion
85 echo "----------------------------------------------------------"

Subscribers

People subscribed via source and target branches