[Merge] ~edubuntu-dev/ubuntu-images:noble into ubuntu-images:noble

Steve Langasek mp+456547 at code.launchpad.net
Sun Dec 3 06:27:29 UTC 2023


Review: Needs Fixing



Diff comments:

> diff --git a/edubuntu-pi-arm64.yaml b/edubuntu-pi-arm64.yaml
> new file mode 100644
> index 0000000..5aa722c
> --- /dev/null
> +++ b/edubuntu-pi-arm64.yaml
> @@ -0,0 +1,85 @@
> +name: edubuntu-desktop-raspi-arm64
> +display-name: Edubuntu Preinstalled Desktop Raspberry Pi arm64
> +revision: 3
> +architecture: arm64
> +series: noble
> +class: preinstalled
> +kernel: linux-image-raspi
> +gadget:
> +  url: "https://git.launchpad.net/snap-pi"
> +  branch: "classic"
> +  type: "git"
> +  target: "desktop"
> +rootfs:
> +  archive: ubuntu
> +  flavor: edubuntu
> +  components:
> +    - main
> +    - restricted
> +    - universe
> +    - multiverse
> +  mirror: "http://ports.ubuntu.com/ubuntu-ports/"
> +  pocket: updates
> +  seed:
> +    urls:
> +      - "git://git.launchpad.net/~edubuntu-dev/ubuntu-seeds/+git/"
> +      - "git://git.launchpad.net/~ubuntu-core-dev/ubuntu-seeds/+git/"
> +    branch: noble
> +    names:
> +      - desktop-gnome-raspi
> +      - raspi-common
> +      - fonts
> +      - ubuntu-edu-preschool
> +      - ubuntu-edu-primary
> +      - ubuntu-edu-secondary
> +      - ubuntu-edu-tertiary

Present in the ubuntu image but missing from the edubuntu image are:
      - minimal
      - standard
      - supported-raspi-common

I don't see anything else here that would pull these in for you.

I think you probably want 'standard' to be included and that it's an oversight if it's not.  But 'minimal' absolutely must be included: it is not optional on Ubuntu systems, and we must not build any flavor images that are missing ubuntu-minimal.

> +customization:
> +  extra-snaps:
> +    - name: snapd
> +    - name: bare
> +    - name: core22

The Ubuntu image doesn't have any core snaps listed; why are you hard-coding this here?

> +    - name: gnome-42-2204
> +      channel: stable/ubuntu-24.04
> +    - name: gtk-common-themes
> +      channel: stable/ubuntu-24.04
> +    - name: firefox
> +      channel: stable/ubuntu-24.04
> +    - name: snap-store
> +      channel: stable/ubuntu-24.04
> +    - name: snapd-desktop-integration
> +      channel: stable/ubuntu-24.04
> +  extra-packages:
> +    - name: snapd

This shouldn't be necessary, snapd is part of both the edubuntu-desktop-gnome and edubuntu-desktop-gnome-raspi tasks via desktop-common

> +    - name: oem-config-gtk
> +    - name: oem-config
> +    - name: ubiquity-frontend-gtk
> +    - name: language-pack-en-base
> +    - name: edubuntu-live-settings
> +    - name: ubiquity-slideshow-edubuntu
> +    - name: squashfs-tools
> +  fstab:
> +    - label: "writable"
> +      mountpoint: "/"
> +      filesystem-type: "ext4"
> +      mount-options: "discard"
> +      dump: false
> +      fsck-order: 1
> +    - label: "system-boot"
> +      mountpoint: "/boot/firmware"
> +      filesystem-type: "vfat"
> +      mount-options: "defaults"
> +      dump: false
> +      fsck-order: 1
> +# FIXME: Temporarily, we also need to make sure that for desktop pi images
> +#  we actually run oem-config-prepare before first boot. Otherwise we won't
> +#  be able to boot into oem-config properly.
> +#  This should be done via a package postinst ideally, so this manual
> +#  customization should be only temporary.
> +  manual:
> +    execute:
> +      - path: "/usr/sbin/oem-config-prepare"
> +artifacts:
> +  img:
> +    - name: edubuntu-24.04-preinstalled-desktop-arm64+raspi.img
> +  manifest:
> +    name: edubuntu-24.04-preinstalled-desktop-arm64+raspi.manifest


-- 
https://code.launchpad.net/~edubuntu-dev/ubuntu-images/+git/ubuntu-images/+merge/456547
Your team Edubuntu Developers is subscribed to branch ~edubuntu-dev/ubuntu-images:noble.




More information about the Ubuntu-reviews mailing list