[Bug 1871919] Re: images with only core18-based snaps fail to build

[Brian Murray]

Hello Robert, or anyone else affected,

Accepted livecd-rootfs into bionic-proposed. The package will build now
and be available at https://launchpad.net/ubuntu/+source/livecd-
rootfs/2.525.45 in a few hours, and then in the -proposed repository.

Please help us by testing this new package.  See
https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how
to enable and use -proposed.  Your feedback will aid us getting this
update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug,
mentioning the version of the package you tested, what testing has been
performed on the package and change the tag from verification-needed-
bionic to verification-done-bionic. If it does not fix the bug for you,
please add a comment stating that, and change the tag to verification-
failed-bionic. In either case, without details of your testing we will
not be able to proceed.

Further information regarding the verification process can be found at
https://wiki.ubuntu.com/QATeam/PerformingSRUVerification .  Thank you in
advance for helping!

N.B. The updated package will be released to -updates after the bug(s)
fixed by this package have been verified and the package has been in
-proposed for a minimum of 7 days.

** Changed in: livecd-rootfs (Ubuntu Bionic)
       Status: New => Fix Committed

** Tags removed: verification-done
** Tags added: verification-needed verification-needed-bionic

-- 
You received this bug notification because you are a member of Ubuntu
Foundations Bugs, which is subscribed to livecd-rootfs in Ubuntu.
https://bugs.launchpad.net/bugs/1871919

Title:
  images with only core18-based snaps fail to build

Status in livecd-rootfs package in Ubuntu:
  Fix Released
Status in livecd-rootfs source package in Bionic:
  Fix Committed
Status in livecd-rootfs source package in Eoan:
  Fix Released

Bug description:
  [Impact]

   * Building an ubuntu-cpc minimal image by default would have no snaps
  installed.  When a binary hook seeds a snap that has a core18 base
  the build will fail when calling 'snap debug validate-seed' because
  the snapd snap is missing.

   * Additionally, an image with snaps that only depend on core18 (which
  isn't in the ubuntu-cpc:minimized project) would still ship 'core'.
  This was found by code inspection and reported in bug #1871912.

  [Test Case]

   * Build with ubuntu-cpc as that seeds lxd (now that it is based on
  core18).  With the proposed fix the image build should succeed with
  lxd, core18, and snapd preseeded.  The build should not fail and
  should not include core.

  [Regression Potential]

   * Image builds fail in some fashion related to snap preseeding.  It
  is unlikely to build an image that fails to pre-seed when booted as we
  call 'snap debug validate-seed' in the build, which is catching the
  current bug of omitting snapd with core18.

  [Other Info]

   * None

  [Original description]

  Building an ubuntu-cpc minimal image by default would have no snaps
  installed.  When a binary hook seeds a snap that has a core18 base
  the build will fail when calling 'snap debug validate-seed' because
  the snapd snap is missing.

  Additionally, an image with snaps that only depend on core18 (which
  isn't in the ubuntu-cpc:minimized project) would still ship 'core'.
  This was found by code inspection and reported in bug #1871912.

  We want to build images that only contain the appropriate core image
  and when that is not core it should have snapd preseeded.

  The _snap_post_process function is meant to install snapd if core18 is the
  only core snap installed or remove snapd if core is installed and snapd
  was not explicitly installed. But the current logic in _snap_preseed
  will never call _snap_post_process. $core_name will never be empty
  with the existing logic, and even if it were that would only be for the
  'core' snap and we'd miss using the 'core18' logic that pulls in snapd.
  Given the case statement in _snap_post_process can handle doing the
  right thing given any snap we can just call it unconditionally.

  Seeding any snap via snap_preseed will evaluate the base for each snap
  and seed the appropriate base. There should be no reason to explicitly
  seed the 'core' snap and with snaps moving to 'core18' this will add
  'core' without need.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/livecd-rootfs/+bug/1871919/+subscriptions