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
1diff --git a/ubuntu-pi-arm64.yaml b/ubuntu-pi-arm64.yaml
2index 0da4798..8bf665d 100644
3--- a/ubuntu-pi-arm64.yaml
4+++ b/ubuntu-pi-arm64.yaml
5@@ -7,7 +7,7 @@ class: preinstalled
6 kernel: linux-image-raspi
7 gadget:
8 url: "https://git.launchpad.net/snap-pi"
9- branch: "classic-redesign"
10+ branch: "classic"
11 type: "git"
12 target: "desktop"
13 rootfs:
14@@ -15,6 +15,8 @@ rootfs:
15 components:
16 - main
17 - restricted
18+ - universe
19+ - multiverse
20 mirror: "http://ports.ubuntu.com/ubuntu-ports/"
21 pocket: updates
22 seed:
23@@ -31,24 +33,17 @@ rootfs:
24 - supported-raspi-common
25 customization:
26 extra-snaps:
27- -
28- name: snapd
29- -
30- name: bare
31- -
32- name: gnome-42-2204
33+ - name: snapd
34+ - name: bare
35+ - name: gnome-42-2204
36 channel: stable/ubuntu-23.10
37- -
38- name: gtk-common-themes
39+ - name: gtk-common-themes
40 channel: stable/ubuntu-23.10
41- -
42- name: firefox
43+ - name: firefox
44 channel: stable/ubuntu-23.10
45- -
46- name: snap-store
47+ - name: snap-store
48 channel: stable/ubuntu-23.10
49- -
50- name: snapd-desktop-integration
51+ - name: snapd-desktop-integration
52 channel: stable/ubuntu-23.10
53 extra-packages:
54 - name: oem-config
55@@ -57,15 +52,13 @@ customization:
56 - name: oem-config-slideshow-ubuntu
57 - name: ubiquity-slideshow-ubuntu
58 fstab:
59- -
60- label: "writable"
61+ - label: "writable"
62 mountpoint: "/"
63 filesystem-type: "ext4"
64 mount-options: "discard"
65 dump: false
66 fsck-order: 1
67- -
68- label: "system-boot"
69+ - label: "system-boot"
70 mountpoint: "/boot/firmware"
71 filesystem-type: "vfat"
72 mount-options: "defaults"
73@@ -81,7 +74,6 @@ customization:
74 - path: "/usr/sbin/oem-config-prepare"
75 artifacts:
76 img:
77- -
78- name: ubuntu-23.10-preinstalled-desktop-arm64+raspi.img
79+ - name: ubuntu-23.10-preinstalled-desktop-arm64+raspi.img
80 manifest:
81 name: ubuntu-23.10-preinstalled-desktop-arm64+raspi.manifest
82diff --git a/ubuntu-server-pi-arm64.yaml b/ubuntu-server-pi-arm64.yaml
83index 4530056..1634376 100644
84--- a/ubuntu-server-pi-arm64.yaml
85+++ b/ubuntu-server-pi-arm64.yaml
86@@ -7,13 +7,15 @@ class: preinstalled
87 kernel: linux-image-raspi
88 gadget:
89 url: "https://git.launchpad.net/snap-pi"
90- branch: "classic-redesign"
91+ branch: "classic"
92 type: "git"
93 rootfs:
94 archive: ubuntu
95 components:
96 - main
97 - restricted
98+ - universe
99+ - multiverse
100 mirror: "http://ports.ubuntu.com/ubuntu-ports/"
101 pocket: updates
102 seed:
103@@ -32,14 +34,12 @@ customization:
104 extra-snaps:
105 - name: snapd
106 fstab:
107- -
108- label: "writable"
109+ - label: "writable"
110 mountpoint: "/"
111 filesystem-type: "ext4"
112 dump: false
113 fsck-order: 1
114- -
115- label: "system-boot"
116+ - label: "system-boot"
117 mountpoint: "/boot/firmware"
118 filesystem-type: "vfat"
119 mount-options: "defaults"
120@@ -47,7 +47,6 @@ customization:
121 fsck-order: 1
122 artifacts:
123 img:
124- -
125- name: ubuntu-23.10-preinstalled-server-arm64+raspi.img
126+ - name: ubuntu-23.10-preinstalled-server-arm64+raspi.img
127 manifest:
128 name: ubuntu-23.10-preinstalled-server-arm64+raspi.manifest
129diff --git a/ubuntu-server-pi.yaml b/ubuntu-server-pi.yaml
130index 81fd593..083a106 100644
131--- a/ubuntu-server-pi.yaml
132+++ b/ubuntu-server-pi.yaml
133@@ -7,13 +7,15 @@ class: preinstalled
134 kernel: linux-image-raspi
135 gadget:
136 url: "https://git.launchpad.net/snap-pi"
137- branch: "classic-redesign"
138+ branch: "classic"
139 type: "git"
140 rootfs:
141 archive: ubuntu
142 components:
143 - main
144 - restricted
145+ - universe
146+ - multiverse
147 mirror: "http://ports.ubuntu.com/ubuntu-ports/"
148 pocket: updates
149 seed:
150@@ -32,14 +34,12 @@ customization:
151 extra-snaps:
152 - name: snapd
153 fstab:
154- -
155- label: "writable"
156+ - label: "writable"
157 mountpoint: "/"
158 filesystem-type: "ext4"
159 dump: false
160 fsck-order: 1
161- -
162- label: "system-boot"
163+ - label: "system-boot"
164 mountpoint: "/boot/firmware"
165 filesystem-type: "vfat"
166 mount-options: "defaults"
167@@ -47,7 +47,6 @@ customization:
168 fsck-order: 1
169 artifacts:
170 img:
171- -
172- name: ubuntu-23.10-preinstalled-server-armhf+raspi.img
173+ - name: ubuntu-23.10-preinstalled-server-armhf+raspi.img
174 manifest:
175 name: ubuntu-23.10-preinstalled-server-armhf+raspi.manifest

Subscribers

People subscribed via source and target branches