Merge lp:~rbalint/livecd-rootfs/s390x-skip-missing-initrd-artful into lp:~ubuntu-core-dev/livecd-rootfs/artful-proposed

Proposed by Balint Reczey
Status: Rejected
Rejected by: Steve Langasek
Proposed branch: lp:~rbalint/livecd-rootfs/s390x-skip-missing-initrd-artful
Merge into: lp:~ubuntu-core-dev/livecd-rootfs/artful-proposed
Diff against target: 61 lines (+22/-10)
2 files modified
debian/changelog (+7/-0)
live-build/ubuntu-cpc/hooks/032-disk-image.binary (+15/-10)
To merge this branch: bzr merge lp:~rbalint/livecd-rootfs/s390x-skip-missing-initrd-artful
Reviewer Review Type Date Requested Status
Steve Langasek Disapprove
Review via email: mp+337238@code.launchpad.net

Description of the change

Skip setting up initrd for s390x when it is not generated for the image.
This fixes building minimized s390x images.

I could not test the change, please someone with s390x access give it a try.

To post a comment you must log in.
Revision history for this message
Steve Langasek (vorlon) wrote :

minimized images are generally not useful on artful because we lack kernels there to support initramfsless boot on most targets. s390x might be an exception because there's low variety in the systems and everything should be built into the kernel, but I'm not sure why this would be worth SRUing to a non-LTS release?

review: Needs Information
Revision history for this message
Balint Reczey (rbalint) wrote :

On Wed, Feb 7, 2018 at 8:35 AM, Steve Langasek
<email address hidden> wrote:
> Review: Needs Information
>
> minimized images are generally not useful on artful because we lack kernels there to support initramfsless boot on most targets. s390x might be an exception because there's low variety in the systems and everything should be built into the kernel, but I'm not sure why this would be worth SRUing to a non-LTS release?

I'm getting emails about minimized Artful build failures, thus IMO we
should either fix the build or stop trying building this image.
I agree that this non-LTS release may not worth fixing now and I'm OK
with just stop trying building.

Revision history for this message
Steve Langasek (vorlon) wrote :

+1 for killing whatever is currently trying to build minimal artful images.

review: Disapprove

Unmerged revisions

1575. By Balint Reczey

Update changelog

1574. By Balint Reczey

Skip setting up initrd for s390x when it is not generated for the image

This fixes building minimized s390x images.

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-08 19:29:43 +0000
3+++ debian/changelog 2018-02-06 20:18:56 +0000
4@@ -1,3 +1,10 @@
5+livecd-rootfs (2.478.2) UNRELEASED; urgency=medium
6+
7+ * Skip setting up initrd for s390x when it is not generated for the image.
8+ This fixes building minimized s390x images.
9+
10+ -- Balint Reczey <rbalint@ubuntu.com> Wed, 07 Feb 2018 03:15:16 +0700
11+
12 livecd-rootfs (2.478.1) artful; urgency=medium
13
14 [ Gary Wang ]
15
16=== modified file 'live-build/ubuntu-cpc/hooks/032-disk-image.binary'
17--- live-build/ubuntu-cpc/hooks/032-disk-image.binary 2017-10-05 05:51:30 +0000
18+++ live-build/ubuntu-cpc/hooks/032-disk-image.binary 2018-02-06 20:18:56 +0000
19@@ -144,27 +144,32 @@
20 [ubuntu]
21 target = /boot
22 image = /boot/vmlinuz
23-ramdisk = /boot/initrd.img
24 parameters = root=LABEL=cloudimg-rootfs
25 EOF
26
27- # Kernel initramfs hooks end up creating a copy
28- # rather than a symlink FIXME
29- pushd mountpoint/boot
30- ln -sf initrd.img-* initrd.img
31- popd
32-
33+ ZIPL_EXTRA_PARAMS=
34+ if [ -e mountpoint/boot/initrd.img-* ]; then
35+ # Kernel initramfs hooks end up creating a copy
36+ # rather than a symlink FIXME
37+ pushd mountpoint/boot
38+ ln -sf initrd.img-* initrd.img
39+ popd
40+
41+ echo "ramdisk = /boot/initrd.img" >> mountpoint/etc/zipl.conf
42+
43+ ZIPL_EXTRA_PARAMS=--ramdisk=/boot/initrd.img
44+ fi
45+
46 # Create bootmap file
47 chroot mountpoint /sbin/zipl -V \
48 --image=/boot/vmlinuz \
49- --ramdisk=/boot/initrd.img \
50 --parameters='root=LABEL=cloudimg-rootfs' \
51 --target=/boot/ \
52 --targetbase=$loop_device \
53 --targettype=SCSI \
54 --targetblocksize=512 \
55- --targetoffset=2048
56-
57+ --targetoffset=2048 \
58+ $ZIPL_EXTRA_PARAMS
59 fi
60
61 if [ -n "$BOOT_MOUNTPOINT" ]; then

Subscribers

People subscribed via source and target branches