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
=== modified file 'debian/changelog'
--- debian/changelog 2017-11-06 19:12:58 +0000
+++ debian/changelog 2017-11-10 00:13:40 +0000
@@ -1,3 +1,15 @@
1livecd-rootfs (2.481) UNRELEASED; urgency=medium
2
3 * The target for x86 ubuntu-core images is VMs, so create them with
4 a buffer of free space in order to work if booted in place.
5 * ubuntu-image now produces a manifest file listing the snap revisions
6 used; publish this so that it's picked up by launchpad-buildd.
7 * Change ubuntu-core image builds to use the standard "$PREFIX" naming for
8 artifacts instead of being gratuitously different; requires a matching
9 change to ubuntu-cdimage.
10
11 -- Steve Langasek <steve.langasek@ubuntu.com> Wed, 08 Nov 2017 12:01:25 -0800
12
1livecd-rootfs (2.480) bionic; urgency=medium13livecd-rootfs (2.480) bionic; urgency=medium
214
3 [ Gary Wang ]15 [ Gary Wang ]
416
=== modified file 'live-build/auto/build'
--- live-build/auto/build 2017-11-06 12:25:57 +0000
+++ live-build/auto/build 2017-11-10 00:13:40 +0000
@@ -15,12 +15,17 @@
15 exit 115 exit 1
16fi16fi
1717
18# Link output files somewhere BuildLiveCD will be able to find them.
19PREFIX="livecd.$PROJECT${SUBARCH:+-$SUBARCH}"
20
18if [ "${IMAGEFORMAT:-}" = "ubuntu-image" ]; then21if [ "${IMAGEFORMAT:-}" = "ubuntu-image" ]; then
19 # Use ubuntu-image instead of live-build22 # Use ubuntu-image instead of live-build
23
20 env SNAPPY_STORE_NO_CDN=1 \24 env SNAPPY_STORE_NO_CDN=1 \
21 ubuntu-image -c edge \25 ubuntu-image -c edge $UBUNTU_IMAGE_ARGS \
22 -o livecd."$PROJECT".img livecd."$PROJECT".model-assertion26 -o "$PREFIX".img "$PREFIX".model-assertion
23 xz -0 -T4 livecd."$PROJECT".img27 xz -0 -T4 "$PREFIX".img
28 mv seed.manifest "$PREFIX".manifest
24 exit 029 exit 0
25fi30fi
2631
@@ -443,9 +448,6 @@
443 exit 1448 exit 1
444fi449fi
445450
446# Link output files somewhere BuildLiveCD will be able to find them.
447PREFIX="livecd.$PROJECT${SUBARCH:+-$SUBARCH}"
448
449case $LB_INITRAMFS in451case $LB_INITRAMFS in
450 casper)452 casper)
451 INITFS="casper"453 INITFS="casper"
452454
=== modified file 'live-build/auto/config'
--- live-build/auto/config 2017-11-06 12:25:57 +0000
+++ live-build/auto/config 2017-11-10 00:13:40 +0000
@@ -90,6 +90,7 @@
90PREINSTALLED=false90PREINSTALLED=false
91PREINSTALL_POOL=91PREINSTALL_POOL=
92PREINSTALL_POOL_SEEDS=92PREINSTALL_POOL_SEEDS=
93PREFIX="livecd.$PROJECT${SUBARCH:+-$SUBARCH}"
9394
94CHROOT_HOOKS=95CHROOT_HOOKS=
95BINARY_HOOKS=96BINARY_HOOKS=
@@ -171,9 +172,17 @@
171 exit 1172 exit 1
172 ;;173 ;;
173 esac174 esac
175 case $MODEL in
176 pc-amd64|pc-i386)
177 UBUNTU_IMAGE_ARGS="--image-size 3700M" ;;
178 *)
179 UBUNTU_IMAGE_ARGS="" ;;
180 esac
181
174 echo "IMAGEFORMAT=$IMAGEFORMAT" >> config/common182 echo "IMAGEFORMAT=$IMAGEFORMAT" >> config/common
183 echo "UBUNTU_IMAGE_ARGS=\"$UBUNTU_IMAGE_ARGS\"" >> config/common
175 # Store model assertion in top dir to get it picked up later as a build artifact184 # Store model assertion in top dir to get it picked up later as a build artifact
176 env SNAPPY_STORE_NO_CDN=1 snap known --remote model series=16 model="$MODEL" brand-id=canonical > livecd."$PROJECT".model-assertion185 env SNAPPY_STORE_NO_CDN=1 snap known --remote model series=16 model="$MODEL" brand-id=canonical > "$PREFIX".model-assertion
177 echo "Configured ubuntu-image for the following model assertion:"186 echo "Configured ubuntu-image for the following model assertion:"
178 cat livecd."$PROJECT".model-assertion187 cat livecd."$PROJECT".model-assertion
179 echo "----------------------------------------------------------"188 echo "----------------------------------------------------------"

Subscribers

People subscribed via source and target branches