Merge lp:~vorlon/livecd-rootfs/ubuntu-channels-for-snaps into lp:livecd-rootfs

Proposed by Steve Langasek
Status: Merged
Merged at revision: 1634
Proposed branch: lp:~vorlon/livecd-rootfs/ubuntu-channels-for-snaps
Merge into: lp:livecd-rootfs
Diff against target: 45 lines (+11/-2)
2 files modified
debian/changelog (+7/-0)
live-build/auto/build (+4/-2)
To merge this branch: bzr merge lp:~vorlon/livecd-rootfs/ubuntu-channels-for-snaps
Reviewer Review Type Date Requested Status
Mathieu Trudel-Lapierre Needs Information
Iain Lane Needs Information
Review via email: mp+337897@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Iain Lane (laney) wrote :

Yeah, the current implementation isn't compliant with the spec.

Few questions inline, nothing major.

When ready to upload, please check the seeds to make sure nothing will fail to build due to this. For example I know that desktop is seeding snap:gnome-calculator ATM and it'll need to be published to the new channel.

Revision history for this message
Iain Lane (laney) :
review: Needs Information
Revision history for this message
Iain Lane (laney) wrote :

> it'll need to be published to the new channel.

Ken says it has been. :-)

Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

Looks generally good to me, provided the channels are present.

However, I thought stuff on non-existent channels were not falling back automatically?

review: Needs Information
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 2018-02-14 20:54:21 +0000
+++ debian/changelog 2018-02-16 22:09:43 +0000
@@ -1,3 +1,10 @@
1livecd-rootfs (2.504) UNRELEASED; urgency=medium
2
3 * Use Ubuntu-specific branches when seeding snaps, as required in
4 <https://wiki.ubuntu.com/UbuntuSeededSnaps>.
5
6 -- Steve Langasek <steve.langasek@ubuntu.com> Fri, 16 Feb 2018 14:02:49 -0800
7
1livecd-rootfs (2.503) bionic; urgency=medium8livecd-rootfs (2.503) bionic; urgency=medium
29
3 * Install linux-signed-generic in the live-server image. (LP: #1749065)10 * Install linux-signed-generic in the live-server image. (LP: #1749065)
411
=== modified file 'live-build/auto/build'
--- live-build/auto/build 2018-01-30 10:16:36 +0000
+++ live-build/auto/build 2018-02-16 22:09:43 +0000
@@ -35,6 +35,8 @@
3535
36preinstall_snaps() {36preinstall_snaps() {
37 lb chroot_resolv install37 lb chroot_resolv install
38 snap_channel="stable/ubuntu-$(distro-info --series="$LB_DISTRIBUTION" \
39 -r | awk '{ print $1 }')"
38 chroot chroot sh -c '40 chroot chroot sh -c '
39 set -x;41 set -x;
40 cd /var/lib/snapd/seed;42 cd /var/lib/snapd/seed;
@@ -43,7 +45,7 @@
43 chroot chroot sh -c "45 chroot chroot sh -c "
44 set -x;46 set -x;
45 cd /var/lib/snapd/seed;47 cd /var/lib/snapd/seed;
46 SNAPPY_STORE_NO_CDN=1 snap download \"${snap%/*}\""48 SNAPPY_STORE_NO_CDN=1 snap download --channel=$snap_channel \"${snap%/*}\""
47 done49 done
48 lb chroot_resolv remove50 lb chroot_resolv remove
4951
@@ -59,7 +61,7 @@
59 snap_name=${snap%/*}61 snap_name=${snap%/*}
60 cat <<EOF >> chroot/var/lib/snapd/seed/seed.yaml62 cat <<EOF >> chroot/var/lib/snapd/seed/seed.yaml
61 - name: ${snap_name}63 - name: ${snap_name}
62 channel: stable64 channel: $snap_channel
63EOF65EOF
64 case ${snap} in */classic) echo " classic: true" >> chroot/var/lib/snapd/seed/seed.yaml;; esac66 case ${snap} in */classic) echo " classic: true" >> chroot/var/lib/snapd/seed/seed.yaml;; esac
65 echo -n " file: " >> chroot/var/lib/snapd/seed/seed.yaml67 echo -n " file: " >> chroot/var/lib/snapd/seed/seed.yaml

Subscribers

People subscribed via source and target branches