Merge ~waveform/ubuntu-images:mantic into ubuntu-images:mantic

Proposed by Dave Jones
Status: Merged
Merged at revision: 6ecae5f622bf87ac30e891ac64a108789eeefc2e
Proposed branch: ~waveform/ubuntu-images:mantic
Merge into: ubuntu-images:mantic
Diff against target: 175 lines (+25/-35)
3 files modified
ubuntu-pi-arm64.yaml (+13/-21)
ubuntu-server-pi-arm64.yaml (+6/-7)
ubuntu-server-pi.yaml (+6/-7)
Reviewer Review Type Date Requested Status
Steve Langasek Approve
Dimitri John Ledkov (community) test-build Approve
Review via email: mp+449865@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Dimitri John Ledkov (xnox) wrote :

build fails on gadget build, on line 254 of Mafile with
"recipe commences before first target.

It seems like https://github.com/snapcore/pi-gadget/commit/5d45b91b8fddee7c98df52754b15ce2ebc5274e8#r128100528 there is missing '\' on the multiline setting.

review: Needs Fixing
Revision history for this message
Dimitri John Ledkov (xnox) wrote :

The pi-gadget got updated, and I managed to build correct images for both armhf and arm64 server. The destkop one is good as well but requires a further fix up proposed at https://code.launchpad.net/~xnox/ubuntu-images/+git/ubuntu-images/+merge/451941

Revision history for this message
Dimitri John Ledkov (xnox) :
review: Approve (test-build)
Revision history for this message
Steve Langasek (vorlon) wrote :

The biggest effective change here is:

$ git diff origin/classic{,-redesign} | diffstat | tail -n1
 37 files changed, 642 insertions(+), 461 deletions(-)
$

So rather than being just a fix of the branch name, this seems to introduce quite a few functional changes vs the branch we're currently using for building! It would have been nice to have harmonized those two branches so that we could just make the branch name switch separately from any behavioral changes, especially now that we're well past feature freeze.

We know that the Pi images are going to get a lot of testing shortly so if there are any problems we can iterate. But it's still a risky approach.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/ubuntu-pi-arm64.yaml b/ubuntu-pi-arm64.yaml
index 0da4798..8bf665d 100644
--- a/ubuntu-pi-arm64.yaml
+++ b/ubuntu-pi-arm64.yaml
@@ -7,7 +7,7 @@ class: preinstalled
7kernel: linux-image-raspi7kernel: linux-image-raspi
8gadget:8gadget:
9 url: "https://git.launchpad.net/snap-pi"9 url: "https://git.launchpad.net/snap-pi"
10 branch: "classic-redesign"10 branch: "classic"
11 type: "git"11 type: "git"
12 target: "desktop"12 target: "desktop"
13rootfs:13rootfs:
@@ -15,6 +15,8 @@ rootfs:
15 components:15 components:
16 - main16 - main
17 - restricted17 - restricted
18 - universe
19 - multiverse
18 mirror: "http://ports.ubuntu.com/ubuntu-ports/"20 mirror: "http://ports.ubuntu.com/ubuntu-ports/"
19 pocket: updates21 pocket: updates
20 seed:22 seed:
@@ -31,24 +33,17 @@ rootfs:
31 - supported-raspi-common33 - supported-raspi-common
32customization:34customization:
33 extra-snaps:35 extra-snaps:
34 -36 - name: snapd
35 name: snapd37 - name: bare
36 -38 - name: gnome-42-2204
37 name: bare
38 -
39 name: gnome-42-2204
40 channel: stable/ubuntu-23.1039 channel: stable/ubuntu-23.10
41 - 40 - name: gtk-common-themes
42 name: gtk-common-themes
43 channel: stable/ubuntu-23.1041 channel: stable/ubuntu-23.10
44 - 42 - name: firefox
45 name: firefox
46 channel: stable/ubuntu-23.1043 channel: stable/ubuntu-23.10
47 - 44 - name: snap-store
48 name: snap-store
49 channel: stable/ubuntu-23.1045 channel: stable/ubuntu-23.10
50 - 46 - name: snapd-desktop-integration
51 name: snapd-desktop-integration
52 channel: stable/ubuntu-23.1047 channel: stable/ubuntu-23.10
53 extra-packages:48 extra-packages:
54 - name: oem-config49 - name: oem-config
@@ -57,15 +52,13 @@ customization:
57 - name: oem-config-slideshow-ubuntu52 - name: oem-config-slideshow-ubuntu
58 - name: ubiquity-slideshow-ubuntu53 - name: ubiquity-slideshow-ubuntu
59 fstab:54 fstab:
60 -55 - label: "writable"
61 label: "writable"
62 mountpoint: "/"56 mountpoint: "/"
63 filesystem-type: "ext4"57 filesystem-type: "ext4"
64 mount-options: "discard"58 mount-options: "discard"
65 dump: false59 dump: false
66 fsck-order: 160 fsck-order: 1
67 -61 - label: "system-boot"
68 label: "system-boot"
69 mountpoint: "/boot/firmware"62 mountpoint: "/boot/firmware"
70 filesystem-type: "vfat"63 filesystem-type: "vfat"
71 mount-options: "defaults"64 mount-options: "defaults"
@@ -81,7 +74,6 @@ customization:
81 - path: "/usr/sbin/oem-config-prepare"74 - path: "/usr/sbin/oem-config-prepare"
82artifacts:75artifacts:
83 img:76 img:
84 -77 - name: ubuntu-23.10-preinstalled-desktop-arm64+raspi.img
85 name: ubuntu-23.10-preinstalled-desktop-arm64+raspi.img
86 manifest:78 manifest:
87 name: ubuntu-23.10-preinstalled-desktop-arm64+raspi.manifest79 name: ubuntu-23.10-preinstalled-desktop-arm64+raspi.manifest
diff --git a/ubuntu-server-pi-arm64.yaml b/ubuntu-server-pi-arm64.yaml
index 4530056..1634376 100644
--- a/ubuntu-server-pi-arm64.yaml
+++ b/ubuntu-server-pi-arm64.yaml
@@ -7,13 +7,15 @@ class: preinstalled
7kernel: linux-image-raspi7kernel: linux-image-raspi
8gadget:8gadget:
9 url: "https://git.launchpad.net/snap-pi"9 url: "https://git.launchpad.net/snap-pi"
10 branch: "classic-redesign"10 branch: "classic"
11 type: "git"11 type: "git"
12rootfs:12rootfs:
13 archive: ubuntu13 archive: ubuntu
14 components:14 components:
15 - main15 - main
16 - restricted16 - restricted
17 - universe
18 - multiverse
17 mirror: "http://ports.ubuntu.com/ubuntu-ports/"19 mirror: "http://ports.ubuntu.com/ubuntu-ports/"
18 pocket: updates20 pocket: updates
19 seed:21 seed:
@@ -32,14 +34,12 @@ customization:
32 extra-snaps:34 extra-snaps:
33 - name: snapd35 - name: snapd
34 fstab:36 fstab:
35 -37 - label: "writable"
36 label: "writable"
37 mountpoint: "/"38 mountpoint: "/"
38 filesystem-type: "ext4"39 filesystem-type: "ext4"
39 dump: false40 dump: false
40 fsck-order: 141 fsck-order: 1
41 -42 - label: "system-boot"
42 label: "system-boot"
43 mountpoint: "/boot/firmware"43 mountpoint: "/boot/firmware"
44 filesystem-type: "vfat"44 filesystem-type: "vfat"
45 mount-options: "defaults"45 mount-options: "defaults"
@@ -47,7 +47,6 @@ customization:
47 fsck-order: 147 fsck-order: 1
48artifacts:48artifacts:
49 img:49 img:
50 -50 - name: ubuntu-23.10-preinstalled-server-arm64+raspi.img
51 name: ubuntu-23.10-preinstalled-server-arm64+raspi.img
52 manifest:51 manifest:
53 name: ubuntu-23.10-preinstalled-server-arm64+raspi.manifest52 name: ubuntu-23.10-preinstalled-server-arm64+raspi.manifest
diff --git a/ubuntu-server-pi.yaml b/ubuntu-server-pi.yaml
index 81fd593..083a106 100644
--- a/ubuntu-server-pi.yaml
+++ b/ubuntu-server-pi.yaml
@@ -7,13 +7,15 @@ class: preinstalled
7kernel: linux-image-raspi7kernel: linux-image-raspi
8gadget:8gadget:
9 url: "https://git.launchpad.net/snap-pi"9 url: "https://git.launchpad.net/snap-pi"
10 branch: "classic-redesign"10 branch: "classic"
11 type: "git"11 type: "git"
12rootfs:12rootfs:
13 archive: ubuntu13 archive: ubuntu
14 components:14 components:
15 - main15 - main
16 - restricted16 - restricted
17 - universe
18 - multiverse
17 mirror: "http://ports.ubuntu.com/ubuntu-ports/"19 mirror: "http://ports.ubuntu.com/ubuntu-ports/"
18 pocket: updates20 pocket: updates
19 seed:21 seed:
@@ -32,14 +34,12 @@ customization:
32 extra-snaps:34 extra-snaps:
33 - name: snapd35 - name: snapd
34 fstab:36 fstab:
35 -37 - label: "writable"
36 label: "writable"
37 mountpoint: "/"38 mountpoint: "/"
38 filesystem-type: "ext4"39 filesystem-type: "ext4"
39 dump: false40 dump: false
40 fsck-order: 141 fsck-order: 1
41 -42 - label: "system-boot"
42 label: "system-boot"
43 mountpoint: "/boot/firmware"43 mountpoint: "/boot/firmware"
44 filesystem-type: "vfat"44 filesystem-type: "vfat"
45 mount-options: "defaults"45 mount-options: "defaults"
@@ -47,7 +47,6 @@ customization:
47 fsck-order: 147 fsck-order: 1
48artifacts:48artifacts:
49 img:49 img:
50 -50 - name: ubuntu-23.10-preinstalled-server-armhf+raspi.img
51 name: ubuntu-23.10-preinstalled-server-armhf+raspi.img
52 manifest:51 manifest:
53 name: ubuntu-23.10-preinstalled-server-armhf+raspi.manifest52 name: ubuntu-23.10-preinstalled-server-armhf+raspi.manifest

Subscribers

People subscribed via source and target branches