ubuntu-cpc: Don't build minimized artifacts that won't boot with linux-kvm

Bug #1757223 reported by Dan Watkins
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
livecd-rootfs (Ubuntu)
Fix Released
Undecided
Dan Watkins
Xenial
Fix Released
Undecided
Unassigned

Bug Description

Currently, the ubuntu-cpc hooks execute for all builds, minimized or not. As minimized images use the linux-kvm kernel, we shouldn't produce images for non-KVM platforms. The hooks that need skipping are the ones that produce the VMDK, OVA and Vagrant box.

Given that livecd-rootfs own the definition of minimized images, it should also own defining which images should be produced when a minimized build runs.

[Impact]

* Increased build time for minimized images
* Publication of minimized cloud images that cannot possibly boot

[Test Case]

Run a minimized livefs build; the .vmdk, .ova and .box files should not be produced. Run a regular livefs build, the .vmdk, .ova and .box files should be produced.

[Regression Potential]

The changes required are adding conditional "exit 0"s to the hooks in question, which aren't used by any other hooks. The only potential regression would be either (a) syntax issues that would manifest obviously in a complete failure to build (and therefore cause the Test Case to fail), or (b) accidental exclusion of these image types for non-minimized builds, which is covered in the Test Case section.

Related branches

Revision history for this message
Dan Watkins (oddbloke) wrote :

Attached debdiff that addresses this in xenial.

Revision history for this message
Dan Watkins (oddbloke) wrote :

Nominated for xenial; minimized images are only produced for {xenial,bionic}, so that's sufficient.

Revision history for this message
Dan Watkins (oddbloke) wrote :

I've tested both the attached bionic branch and the attached xenial debdiff from a PPA.

Revision history for this message
Dan Watkins (oddbloke) wrote :

Attached a xenial MP after it was pointed out to me that we have a branch for xenial's livecd-rootfs.

Steve Langasek (vorlon)
Changed in livecd-rootfs (Ubuntu):
status: New → Fix Committed
Changed in livecd-rootfs (Ubuntu Xenial):
status: New → In Progress
tags: added: patch
Revision history for this message
Steve Langasek (vorlon) wrote : Please test proposed package

Hello Dan, or anyone else affected,

Accepted livecd-rootfs into xenial-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/livecd-rootfs/2.408.28 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 and change the tag from verification-needed-xenial to verification-done-xenial. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-xenial. 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!

Changed in livecd-rootfs (Ubuntu Xenial):
status: In Progress → Fix Committed
tags: added: verification-needed verification-needed-xenial
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package livecd-rootfs - 2.512

---------------
livecd-rootfs (2.512) bionic; urgency=medium

  * ubuntu-cpc: When performing a minimized build, don't generate artifacts
    that won't boot with the linux-kvm kernel (LP: #1757223).

 -- Daniel Watkins <email address hidden> Tue, 20 Mar 2018 13:42:50 -0400

Changed in livecd-rootfs (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Dan Watkins (oddbloke) wrote :

I've built images from -proposed and confirmed that the behaviour as as expected.

tags: added: verification-done verification-done-xenial
removed: verification-needed verification-needed-xenial
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package livecd-rootfs - 2.408.28

---------------
livecd-rootfs (2.408.28) xenial; urgency=medium

  * ubuntu-cpc: When performing a minimized build, don't generate artifacts
    that won't boot with the linux-kvm kernel (LP: #1757223).

 -- Daniel Watkins <email address hidden> Tue, 20 Mar 2018 12:42:25 -0400

Changed in livecd-rootfs (Ubuntu Xenial):
status: Fix Committed → Fix Released
Revision history for this message
Steve Langasek (vorlon) wrote : Update Released

The verification of the Stable Release Update for livecd-rootfs has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

tags: added: id-5ab10f6f7858201e0301532d
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.