[Bug 1770093] Re: Ubiquity crashes with mount points that are FAT16/32

[Dan Streetman]

removing ubuntu-sponsors while I try to get feedback and sponsor this.

** Description changed:

+ [impact]
+ 
+ when installing on pi with ubiquity, the /boot/firmware partition gets
+ mounted with extra parameters that causes ubiquity to crash.
+ 
+ [test case]
+ 
+ see original description.
+ 
+ [regression potential]
+ 
+ this removes the additional mount params from (only) the /boot/firmware
+ mount, so any regression would likely involve problems with that
+ partition.  Also since this patches the installation path, regressions
+ may occur during installation.
+ 
+ [scope]
+ 
+ Debian does not include the extra mount parameters, so Debian does not
+ need patching.
+ 
+ This package has been unchanged since bionic.  The code needing patching
+ is unchanged between xenial and bionic.
+ 
+ This needs patching in all releases (Xenial, Bionic, Eoan, Focal).
+ 
+ [original description]
+ 
  Hi, I've used ubiquity as an installer for the Raspberry Pi 2/3.  One of
  the quirks of the Pi is its need for a FAT formatted boot partition.
  The Pi packages (flash-kernel etc) expect this to be mounted at
  /boot/firmware.  However if I give ubiquity this mount point then it
  soon crashes.
  
  I believe the problem is to do with the mount options that are
  automatically given to fat partitions.  I've solved this by applying a
  patch to the partman-basicfilesystems package.  Please see attached.
  
  I'm not sure whether this is the correct fix, or a more generic fix
  should be applied to ubiquity and FAT partitions.

** Also affects: partman-basicfilesystems (Ubuntu Bionic)
   Importance: Undecided
       Status: New

** Also affects: partman-basicfilesystems (Ubuntu Xenial)
   Importance: Undecided
       Status: New

** Also affects: partman-basicfilesystems (Ubuntu Focal)
   Importance: Low
       Status: New

** Also affects: partman-basicfilesystems (Ubuntu Eoan)
   Importance: Undecided
       Status: New

** Changed in: partman-basicfilesystems (Ubuntu Eoan)
   Importance: Undecided => Low

** Changed in: partman-basicfilesystems (Ubuntu Bionic)
   Importance: Undecided => Medium

** Changed in: partman-basicfilesystems (Ubuntu Xenial)
   Importance: Undecided => Medium

** Changed in: partman-basicfilesystems (Ubuntu Eoan)
   Importance: Low => Medium

** Changed in: partman-basicfilesystems (Ubuntu Focal)
   Importance: Low => Medium

** Changed in: partman-basicfilesystems (Ubuntu Focal)
       Status: New => In Progress

** Changed in: partman-basicfilesystems (Ubuntu Eoan)
       Status: New => In Progress

** Changed in: partman-basicfilesystems (Ubuntu Bionic)
       Status: New => In Progress

** Changed in: partman-basicfilesystems (Ubuntu Xenial)
       Status: New => In Progress

** Changed in: partman-basicfilesystems (Ubuntu Focal)
     Assignee: (unassigned) => Dan Streetman (ddstreet)

** Changed in: partman-basicfilesystems (Ubuntu Eoan)
     Assignee: (unassigned) => Dan Streetman (ddstreet)

** Changed in: partman-basicfilesystems (Ubuntu Bionic)
     Assignee: (unassigned) => Dan Streetman (ddstreet)

** Changed in: partman-basicfilesystems (Ubuntu Xenial)
     Assignee: (unassigned) => Dan Streetman (ddstreet)

** Tags added: ubuntu-sponsor-ddstreet

-- 
You received this bug notification because you are a member of Ubuntu
Sponsors Team, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/1770093

Title:
  Ubiquity crashes with mount points that are FAT16/32

Status in partman-basicfilesystems package in Ubuntu:
  In Progress
Status in partman-basicfilesystems source package in Xenial:
  In Progress
Status in partman-basicfilesystems source package in Bionic:
  In Progress
Status in partman-basicfilesystems source package in Eoan:
  In Progress
Status in partman-basicfilesystems source package in Focal:
  In Progress

Bug description:
  [impact]

  when installing on pi with ubiquity, the /boot/firmware partition gets
  mounted with extra parameters that causes ubiquity to crash.

  [test case]

  see original description.

  [regression potential]

  this removes the additional mount params from (only) the
  /boot/firmware mount, so any regression would likely involve problems
  with that partition.  Also since this patches the installation path,
  regressions may occur during installation.

  [scope]

  Debian does not include the extra mount parameters, so Debian does not
  need patching.

  This package has been unchanged since bionic.  The code needing
  patching is unchanged between xenial and bionic.

  This needs patching in all releases (Xenial, Bionic, Eoan, Focal).

  [original description]

  Hi, I've used ubiquity as an installer for the Raspberry Pi 2/3.  One
  of the quirks of the Pi is its need for a FAT formatted boot
  partition.  The Pi packages (flash-kernel etc) expect this to be
  mounted at /boot/firmware.  However if I give ubiquity this mount
  point then it soon crashes.

  I believe the problem is to do with the mount options that are
  automatically given to fat partitions.  I've solved this by applying a
  patch to the partman-basicfilesystems package.  Please see attached.

  I'm not sure whether this is the correct fix, or a more generic fix
  should be applied to ubiquity and FAT partitions.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/partman-basicfilesystems/+bug/1770093/+subscriptions