[SRU] Allow to bypass container exit test
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
flash-kernel (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Jammy |
Fix Released
|
Undecided
|
Unassigned | ||
Kinetic |
Fix Released
|
Undecided
|
Unassigned | ||
livecd-rootfs (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Jammy |
Fix Released
|
Undecided
|
Unassigned | ||
Kinetic |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
https:/
So introduce a new environment variable that allows to bypass this test in such circumstances: FK_FORCE_CONTAINER must be equal to "yes" to skip the test and allow flash-kernel to run.
[Impact]
RISC-V images build needs to run flash-kernel to install the dtb for update-grub to find. Currently, Ubuntu has a delta that prevents flash-kernel to run in a container but our infrastructure runs the images build inside a container which breaks the installation of the dtb on RISC-V images.
We fix this by adding a new environment variable which will be set to "yes" only when building a RISC-V image.
[Test Plan]
To reproduce this bug, we need to launch livecd-rootfs inside a LXC container.
To test this fix, I'll patch livecd-rootfs in order to set the new environment variable and we'll trigger a new build of a RISC-V image from launchpad.
Additionally, an Ubuntu Server image should be spun up in a LXC container, and flash-kernel from the -proposed pocket should be executed to ensure it does *not* run:
* Launch an Ubuntu Jammy image in LXD: lxc launch ubuntu:j test-fk
* Shell into the instance: lxc exec test-fk bash
* Enable proposed (https:/
* Update flash-kernel to the proposed version: apt install -t jammy-proposed flash-kernel
* Run flash-kernel if the update did not already: flash-kernel
* Ensure it exited cleanly without doing anything (echo $? and check /boot looks the same?)
[Where problems could occur]
We must take care at not breaking other images builds, or boot on the cloud images (the reason for initial introduction of the container test in LP: #1863145), so great attention to the patch must be paid so that it only changes the current behaviour when we set the environment variable. The chosen variable name is specific enough as it likely won't be used (i.e.: FK_FORCE_
[Other Info]
This patch is needed in Jammy as we intend to publish new RISC-V images for 22.04.1.
Related branches
- Łukasz Zemczak: Approve
-
Diff: 264 lines (+172/-34)3 files modifieddebian/changelog (+8/-0)
live-build/auto/config (+12/-2)
live-build/ubuntu-cpc/hooks.d/base/disk-image-uefi.binary (+152/-32)
- Łukasz Zemczak: Approve
-
Diff: 35 lines (+4/-2)2 files modifieddebian/changelog (+2/-0)
live-build/ubuntu-cpc/hooks.d/base/disk-image-uefi.binary (+2/-2)
description: | updated |
description: | updated |
Please find attached the debdiff for kinetic