Files
snapd/bootloader/grub_test.go
Valentin David 2034c7edb2 boot,bootloader: add support for shim fallback and setting EFI boot variables on install (#13511)
* boot: added function to set EFI variables

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

boot: renamed trustedShimFallbackBinary to seedShimPath

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

boot: refactored setting EFI boot variables at install

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

boot: adjusted variable names and fixed variable initialization

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

boot: improve setting Boot#### EFI variable

Notably, splits off the process of reading a Boot#### variable and
extracting its DevicePath into its own function `readBootVariable` which
can be mocked and otherwise simplifies the `setBootNumberVariable`
function.

Also, fixes behavior around the final BootFFFF variable.  Previously, it
was not possible to select the BootFFFF variable if it was unused, due
to overflow concerns on uint16.  Now, the behavior around BootFFFF is
identical to that of any other boot variable, by using an int internally
instead of uint16, which also allows a more robust check for whether
there were no matching variables.

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

boot: added unit tests for setting EFI Boot#### variable

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

boot: refactored setting EFI boot variables

Rewrote EFI boot variable functions to more closely match the behavior
of shim fallback: https://github.com/rhboot/shim/blob/main/fallback.c

In particular, the following have changed:

1. Existing Boot#### variables must fully match the new load option to
   be considered a match.  In particular, the load option attributes,
   label, and device path must all be byte-for-byte identical.
   Previously, only the device paths were compared.
2. Matching Boot#### variables are no longer overwritten.  Since the
   variable data must now byte-for-byte match the new load option, there
   is no need to overwrite the existing variable.
3. Since existing Boot#### variables are no longer overwritten, the
   variable attributes are no longer checked for those variables.
   Instead, it is assumed that the Boot#### variable attributes are
   viable for it to be used as a boot option.  This matches the behavior
   of `rhboot/shim/fallback.c`, for better or for worse.
4. When modifying the BootOrder variable, boot option numbers are no
   longer pruned if there is no matching Boot#### variable.

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

boot,bootloader: introduce UefiBootloader to build EFI load options

Previously, the path of the shim binary relative to the EFI partition
was passed into `SetEfiBootVariables`. However, different bootloaders
may wish to set up `OptionalData` in the load option.

Additionally, not all `TrustedAssetBootloaders` will attempt to set
EFI boot variables, and not all bootloaders which should set EFI boot
variables necessarily support secure boot. Thus, these should be
decoupled.

This commit adds a new `UefiBootloader` interface with the
`ConstructShimEfiLoadOption` method, which builds an EFI load option
from the shim path for the given bootloader.

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

boot,bootloader: fixed linting errors and improved EFI boot variable test clarity

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

bootloader: improved unit test for grub EFI load option creation

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

boot: set EFI boot variables in `MakeRunnableSystem`

Previously, attempted to set boot variables in
`MakeRecoverySystemBootable`, which is called by `MakeBootableImage`,
which is called when building the image file, rather than during install
mode.

`MakeRunnableSystem` is called on first boot during install mode, and
thus should be responsible for setting EFI boot variables.

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

boot: use seed bootloader when setting EFI variables

In install mode, the bootloader located in ubuntu-seed should be used
when setting the EFI boot variables. Previously, the bootloader in
ubuntu-boot was accidentally re-used.

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

tests: added simple test to execute setefibootvar.go code

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

tests: fixed standalone set EFI vars code test to work with different layouts

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

tests: moved simple setefibootvar.go check to nested test

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

tests: added check for idempotence when setting EFI boot variables

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

bootloader: adjust comments, organization, and add TODO

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

boot,bootloader: fix setting EFI boot variables

Make function to search for EFI asset device path and construct load
option common so each UefiBootloader does not have to re-implement it.
Instead, the bootloader returns the description, asset file path, and
optional data, which can then be used to create the EFI load option.

Also, in `makeRunnableSystem`, the bootloader in ubuntu-seed must have
`NoSlashBoot` in order to correctly find the grub.cfg file and thus the
grub bootloader. This commit fixes this bug, and refactors a bit to
account for the changes in responsibilities between the bootloader and
the setefibootvars.go code.

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

bootloader: fixed grub EFI load option test with tmp rootdir

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

go.mod: move golang.org/x/text import next to other golang.org/x/ imports

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

boot: adjust opts to look for recovery bootloader when setting EFI variables

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

boot: do not overwrite BootOrder if unchanged, and unexport EFI variable helper functions

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

boot: unexport `setEfiBootOrderVariable`

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

boot: move code to detect bootloader and set EFI variables accordingly into dedicated function

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

boot: unexport `setUbuntuSeedEfiBootVariables` and accompanying error

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

boot,bootloader: ensure nil optionalData for EFI variable is equivalent to 0-length slice

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

boot: handle empty boot order and other boot var improvements

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

boot: make setefibootvars functions linux-only

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

* tests: add nested spread test for setting EFI vars

The test checks that EFI boot variables exist for the following:
1. A Boot#### variable pointing to the shim file path.
2. A BootOrder variable with the #### from the above Boot#### as first.

Since the layout of EFI assets is dependent on the gadget snap, the test
downloads and unpacks the gadget, then modifies the contents so that one
variant has the shim and grub binaries in `EFI/boot/` and another
variant has the shim and grub binaries in `EFI/ubuntu/` and the fallback
binary in `EFI/boot/`.

After building a core image around that modified gadget, the VM is
booted and the test checks that the EFI variables are set correctly.
Then, the test modifies the gadget to match the other variant's initial
layout, and then installs the newly modified gadget. This should trigger
re-setting EFI boot variables as well.

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

tests: fix problems in spread test for setting EFI boot variables

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

tests: disabled TPM on EFI boot vars test and separated gadget script

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

tests: fixed EFI vars test to use correct toolbox and include all EFI assets

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

tests: modify-gadget.sh re-use existing gadget so edition is incremented

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

tests: fix mangled EFI var search string and other improvements

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

tests: polish tests for setting EFI boot variables

Notably, allow tests/nested/core/core20-set-efi-boot-variables to run on
arm64 as well as amd64, simplify setefivars.go to search for multiple
assets on multiple architectures, and allow
tests/nested/manual/core20-set-efi-boot-vars to run on any ubuntu-2*.

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>

* bootloader/grub.go: only consider new shim asset in boot entry for now

* tests/nested/core/core20-set-efi-boot-variables: fix details

* boot: update uefi variables on gadget update

* tests/nested/manual/core20-set-efi-boot-vars: work-around file not deleted

* tests/nested/manual/core20-set-efi-boot-vars: use fb.efi like other tests

* tests/nested/manual/core20-set-efi-boot-vars: drop use of toolbox snap

* tests/nested/manual/core20-set-efi-boot-vars: drop work-around for not deleted files

* tests/nested/manual/core20-set-efi-boot-vars: verify install does add a boot entry

* tests/nested/manual/core20-set-efi-boot-vars: run only on version that have UC

* tests/nested/manual/core20-set-efi-boot-vars: obey GADGET_CHANNEL

* tests/nested/manual/core20-set-efi-boot-vars: move get_boot_entry.py to libs

* tests/nested/manual/core20-set-efi-boot-vars: factorize copy of variables

... so we can reuse the script in other tests

* tests/nested/core/core20-set-efi-boot-variables: stop using toolbox snap

* tests/nested/core/core20-set-efi-boot-variables: only run on versions with UC available

* overlord/devicestate: test using EfiLoadOptionParameters

* boot: test that variables are set

* boot: test observers' UpdateBootEntry

* tests/nested/manual/core20-set-efi-boot-vars: also test without secure boot

* many: use trusted install observer when UEFI variables are supported

* boot/makebootable.go: rename sealer to observer

* boot/grub.go: fix function name in doc

* cmd/snap-bootstrap: verify that ObserveExistingTrustedRecoveryAssets is called

* boot: add tests for SetEfiBootVariables

* many: comment on calls to ObserveExistingTrustedRecoveryAssets

---------

Signed-off-by: Oliver Calder <oliver.calder@canonical.com>
Co-authored-by: Oliver Calder <oliver.calder@canonical.com>
2024-06-03 10:03:50 +02:00

1450 lines
50 KiB
Go

// -*- Mode: Go; indent-tabs-mode: t -*-
/*
* Copyright (C) 2014-2024 Canonical Ltd
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU General Public License version 3 as
* published by the Free Software Foundation.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program. If not, see <http://www.gnu.org/licenses/>.
*
*/
package bootloader_test
import (
"fmt"
"os"
"os/exec"
"path/filepath"
"github.com/mvo5/goconfigparser"
. "gopkg.in/check.v1"
"github.com/snapcore/snapd/arch"
"github.com/snapcore/snapd/arch/archtest"
"github.com/snapcore/snapd/bootloader"
"github.com/snapcore/snapd/bootloader/assets"
"github.com/snapcore/snapd/bootloader/grubenv"
"github.com/snapcore/snapd/dirs"
"github.com/snapcore/snapd/osutil"
"github.com/snapcore/snapd/snap"
"github.com/snapcore/snapd/snap/snapfile"
"github.com/snapcore/snapd/snap/snaptest"
"github.com/snapcore/snapd/testutil"
)
type grubTestSuite struct {
baseBootenvTestSuite
bootdir string
}
var _ = Suite(&grubTestSuite{})
func (s *grubTestSuite) SetUpTest(c *C) {
s.baseBootenvTestSuite.SetUpTest(c)
bootloader.MockGrubFiles(c, s.rootdir)
s.bootdir = filepath.Join(s.rootdir, "boot")
// By default assume amd64 in the tests: there are specialized
// tests for other arches
s.AddCleanup(archtest.MockArchitecture("amd64"))
snippets := []assets.ForEditions{
{FirstEdition: 1, Snippet: []byte("console=ttyS0 console=tty1 panic=-1")},
}
s.AddCleanup(assets.MockSnippetsForEdition("grub.cfg:static-cmdline", snippets))
s.AddCleanup(assets.MockSnippetsForEdition("grub-recovery.cfg:static-cmdline", snippets))
}
// grubEditenvCmd finds the right grub{,2}-editenv command
func grubEditenvCmd() string {
for _, exe := range []string{"grub2-editenv", "grub-editenv"} {
if osutil.ExecutableExists(exe) {
return exe
}
}
return ""
}
func grubEnvPath(rootdir string) string {
return filepath.Join(rootdir, "boot/grub/grubenv")
}
func (s *grubTestSuite) grubEditenvSet(c *C, key, value string) {
if grubEditenvCmd() == "" {
c.Skip("grub{,2}-editenv is not available")
}
output, err := exec.Command(grubEditenvCmd(), grubEnvPath(s.rootdir), "set", fmt.Sprintf("%s=%s", key, value)).CombinedOutput()
c.Check(err, IsNil)
c.Check(string(output), Equals, "")
}
func (s *grubTestSuite) grubEditenvGet(c *C, key string) string {
if grubEditenvCmd() == "" {
c.Skip("grub{,2}-editenv is not available")
}
output, err := exec.Command(grubEditenvCmd(), grubEnvPath(s.rootdir), "list").CombinedOutput()
c.Assert(err, IsNil)
cfg := goconfigparser.New()
cfg.AllowNoSectionHeader = true
err = cfg.ReadString(string(output))
c.Assert(err, IsNil)
v, err := cfg.Get("", key)
c.Assert(err, IsNil)
return v
}
func (s *grubTestSuite) makeFakeGrubEnv(c *C) {
s.grubEditenvSet(c, "k", "v")
}
func (s *grubTestSuite) TestNewGrub(c *C) {
// no files means bl is not present, but we can still create the bl object
c.Assert(os.RemoveAll(s.rootdir), IsNil)
g := bootloader.NewGrub(s.rootdir, nil)
c.Assert(g, NotNil)
c.Assert(g.Name(), Equals, "grub")
present, err := g.Present()
c.Assert(err, IsNil)
c.Assert(present, Equals, false)
// now with files present, the bl is present
bootloader.MockGrubFiles(c, s.rootdir)
s.makeFakeGrubEnv(c)
present, err = g.Present()
c.Assert(err, IsNil)
c.Assert(present, Equals, true)
}
func (s *grubTestSuite) TestGetBootloaderWithGrub(c *C) {
s.makeFakeGrubEnv(c)
bootloader, err := bootloader.Find(s.rootdir, nil)
c.Assert(err, IsNil)
c.Assert(bootloader.Name(), Equals, "grub")
}
func (s *grubTestSuite) TestGetBootloaderWithGrubWithDefaultRoot(c *C) {
s.makeFakeGrubEnv(c)
dirs.SetRootDir(s.rootdir)
defer func() { dirs.SetRootDir("") }()
bootloader, err := bootloader.Find("", nil)
c.Assert(err, IsNil)
c.Assert(bootloader.Name(), Equals, "grub")
}
func (s *grubTestSuite) TestGetBootVer(c *C) {
s.makeFakeGrubEnv(c)
s.grubEditenvSet(c, "snap_mode", "regular")
g := bootloader.NewGrub(s.rootdir, nil)
v, err := g.GetBootVars("snap_mode")
c.Assert(err, IsNil)
c.Check(v, HasLen, 1)
c.Check(v["snap_mode"], Equals, "regular")
}
func (s *grubTestSuite) TestSetBootVer(c *C) {
s.makeFakeGrubEnv(c)
g := bootloader.NewGrub(s.rootdir, nil)
err := g.SetBootVars(map[string]string{
"k1": "v1",
"k2": "v2",
})
c.Assert(err, IsNil)
c.Check(s.grubEditenvGet(c, "k1"), Equals, "v1")
c.Check(s.grubEditenvGet(c, "k2"), Equals, "v2")
}
func (s *grubTestSuite) TestExtractKernelAssetsNoUnpacksKernelForGrub(c *C) {
s.makeFakeGrubEnv(c)
g := bootloader.NewGrub(s.rootdir, nil)
files := [][]string{
{"kernel.img", "I'm a kernel"},
{"initrd.img", "...and I'm an initrd"},
{"meta/kernel.yaml", "version: 4.2"},
}
si := &snap.SideInfo{
RealName: "ubuntu-kernel",
Revision: snap.R(42),
}
fn := snaptest.MakeTestSnapWithFiles(c, packageKernel, files)
snapf, err := snapfile.Open(fn)
c.Assert(err, IsNil)
info, err := snap.ReadInfoFromSnapFile(snapf, si)
c.Assert(err, IsNil)
err = g.ExtractKernelAssets(info, snapf)
c.Assert(err, IsNil)
// kernel is *not* here
kernimg := filepath.Join(s.bootdir, "grub", "ubuntu-kernel_42.snap", "kernel.img")
c.Assert(osutil.FileExists(kernimg), Equals, false)
}
func (s *grubTestSuite) TestExtractKernelForceWorks(c *C) {
s.makeFakeGrubEnv(c)
g := bootloader.NewGrub(s.rootdir, nil)
c.Assert(g, NotNil)
files := [][]string{
{"kernel.img", "I'm a kernel"},
{"initrd.img", "...and I'm an initrd"},
{"meta/force-kernel-extraction", ""},
{"meta/kernel.yaml", "version: 4.2"},
}
si := &snap.SideInfo{
RealName: "ubuntu-kernel",
Revision: snap.R(42),
}
fn := snaptest.MakeTestSnapWithFiles(c, packageKernel, files)
snapf, err := snapfile.Open(fn)
c.Assert(err, IsNil)
info, err := snap.ReadInfoFromSnapFile(snapf, si)
c.Assert(err, IsNil)
err = g.ExtractKernelAssets(info, snapf)
c.Assert(err, IsNil)
// kernel is extracted
kernimg := filepath.Join(s.bootdir, "grub", "ubuntu-kernel_42.snap", "kernel.img")
c.Assert(osutil.FileExists(kernimg), Equals, true)
// initrd
initrdimg := filepath.Join(s.bootdir, "grub", "ubuntu-kernel_42.snap", "initrd.img")
c.Assert(osutil.FileExists(initrdimg), Equals, true)
// ensure that removal of assets also works
err = g.RemoveKernelAssets(info)
c.Assert(err, IsNil)
exists, _, err := osutil.DirExists(filepath.Dir(kernimg))
c.Assert(err, IsNil)
c.Check(exists, Equals, false)
}
func (s *grubTestSuite) grubDir() string {
return filepath.Join(s.bootdir, "grub")
}
func (s *grubTestSuite) grubEFINativeDir() string {
return filepath.Join(s.rootdir, "EFI/ubuntu")
}
func (s *grubTestSuite) makeFakeGrubEFINativeEnv(c *C, content []byte) {
err := os.MkdirAll(s.grubEFINativeDir(), 0755)
c.Assert(err, IsNil)
err = os.WriteFile(filepath.Join(s.grubEFINativeDir(), "grub.cfg"), content, 0644)
c.Assert(err, IsNil)
}
func (s *grubTestSuite) makeFakeShimFallback(c *C) {
err := os.MkdirAll(filepath.Join(s.rootdir, "/EFI/boot"), 0755)
c.Assert(err, IsNil)
_, err = os.Create(filepath.Join(s.rootdir, "/EFI/boot/fbx64.efi"))
c.Assert(err, IsNil)
_, err = os.Create(filepath.Join(s.rootdir, "/EFI/boot/fbaa64.efi"))
c.Assert(err, IsNil)
}
func (s *grubTestSuite) TestNewGrubWithOptionRecovery(c *C) {
s.makeFakeGrubEFINativeEnv(c, nil)
g := bootloader.NewGrub(s.rootdir, &bootloader.Options{Role: bootloader.RoleRecovery})
c.Assert(g, NotNil)
c.Assert(g.Name(), Equals, "grub")
}
func (s *grubTestSuite) TestNewGrubWithOptionRecoveryBootEnv(c *C) {
s.makeFakeGrubEFINativeEnv(c, nil)
g := bootloader.NewGrub(s.rootdir, &bootloader.Options{Role: bootloader.RoleRecovery})
// check that setting vars goes to the right place
c.Check(filepath.Join(s.grubEFINativeDir(), "grubenv"), testutil.FileAbsent)
err := g.SetBootVars(map[string]string{
"k1": "v1",
"k2": "v2",
})
c.Assert(err, IsNil)
c.Check(filepath.Join(s.grubEFINativeDir(), "grubenv"), testutil.FilePresent)
env, err := g.GetBootVars("k1", "k2")
c.Assert(err, IsNil)
c.Check(env, DeepEquals, map[string]string{
"k1": "v1",
"k2": "v2",
})
}
func (s *grubTestSuite) TestNewGrubWithOptionRecoveryNoEnv(c *C) {
// fake a *regular* grub env
s.makeFakeGrubEnv(c)
// we can't create a recovery grub with that
g, err := bootloader.Find(s.rootdir, &bootloader.Options{Role: bootloader.RoleRecovery})
c.Assert(g, IsNil)
c.Assert(err, Equals, bootloader.ErrBootloader)
}
func (s *grubTestSuite) TestGrubSetRecoverySystemEnv(c *C) {
s.makeFakeGrubEFINativeEnv(c, nil)
g := bootloader.NewGrub(s.rootdir, &bootloader.Options{Role: bootloader.RoleRecovery})
// check that we can set a recovery system specific bootenv
bvars := map[string]string{
"snapd_recovery_kernel": "/snaps/pc-kernel_1.snap",
"other_options": "are-supported",
}
err := g.SetRecoverySystemEnv("/systems/20191209", bvars)
c.Assert(err, IsNil)
recoverySystemGrubenv := filepath.Join(s.rootdir, "/systems/20191209/grubenv")
c.Assert(recoverySystemGrubenv, testutil.FilePresent)
genv := grubenv.NewEnv(recoverySystemGrubenv)
err = genv.Load()
c.Assert(err, IsNil)
c.Check(genv.Get("snapd_recovery_kernel"), Equals, "/snaps/pc-kernel_1.snap")
c.Check(genv.Get("other_options"), Equals, "are-supported")
}
func (s *grubTestSuite) TestGetRecoverySystemEnv(c *C) {
s.makeFakeGrubEFINativeEnv(c, nil)
g := bootloader.NewGrub(s.rootdir, &bootloader.Options{Role: bootloader.RoleRecovery})
err := os.MkdirAll(filepath.Join(s.rootdir, "/systems/20191209"), 0755)
c.Assert(err, IsNil)
recoverySystemGrubenv := filepath.Join(s.rootdir, "/systems/20191209/grubenv")
// does not fail when there is no recovery env
value, err := g.GetRecoverySystemEnv("/systems/20191209", "no_file")
c.Assert(err, IsNil)
c.Check(value, Equals, "")
genv := grubenv.NewEnv(recoverySystemGrubenv)
genv.Set("snapd_extra_cmdline_args", "foo bar baz")
genv.Set("random_option", `has "some spaces"`)
err = genv.Save()
c.Assert(err, IsNil)
value, err = g.GetRecoverySystemEnv("/systems/20191209", "snapd_extra_cmdline_args")
c.Assert(err, IsNil)
c.Check(value, Equals, "foo bar baz")
value, err = g.GetRecoverySystemEnv("/systems/20191209", "random_option")
c.Assert(err, IsNil)
c.Check(value, Equals, `has "some spaces"`)
value, err = g.GetRecoverySystemEnv("/systems/20191209", "not_set")
c.Assert(err, IsNil)
c.Check(value, Equals, ``)
}
func (s *grubTestSuite) makeKernelAssetSnap(c *C, snapFileName string) snap.PlaceInfo {
kernelSnap, err := snap.ParsePlaceInfoFromSnapFileName(snapFileName)
c.Assert(err, IsNil)
// make a kernel.efi snap as it would be by ExtractKernelAssets()
kernelSnapExtractedAssetsDir := filepath.Join(s.grubDir(), snapFileName)
err = os.MkdirAll(kernelSnapExtractedAssetsDir, 0755)
c.Assert(err, IsNil)
err = os.WriteFile(filepath.Join(kernelSnapExtractedAssetsDir, "kernel.efi"), nil, 0644)
c.Assert(err, IsNil)
return kernelSnap
}
func (s *grubTestSuite) makeKernelAssetSnapAndSymlink(c *C, snapFileName, symlinkName string) snap.PlaceInfo {
kernelSnap := s.makeKernelAssetSnap(c, snapFileName)
// make a kernel.efi symlink to the kernel.efi above
err := os.Symlink(
filepath.Join(snapFileName, "kernel.efi"),
filepath.Join(s.grubDir(), symlinkName),
)
c.Assert(err, IsNil)
return kernelSnap
}
func (s *grubTestSuite) TestGrubExtractedRunKernelImageKernel(c *C) {
s.makeFakeGrubEnv(c)
g := bootloader.NewGrub(s.rootdir, nil)
eg, ok := g.(bootloader.ExtractedRunKernelImageBootloader)
c.Assert(ok, Equals, true)
kernel := s.makeKernelAssetSnapAndSymlink(c, "pc-kernel_1.snap", "kernel.efi")
// ensure that the returned kernel is the same as the one we put there
sn, err := eg.Kernel()
c.Assert(err, IsNil)
c.Assert(sn, DeepEquals, kernel)
}
func (s *grubTestSuite) TestGrubExtractedRunKernelImageTryKernel(c *C) {
s.makeFakeGrubEnv(c)
g := bootloader.NewGrub(s.rootdir, nil)
eg, ok := g.(bootloader.ExtractedRunKernelImageBootloader)
c.Assert(ok, Equals, true)
// ensure it doesn't return anything when the symlink doesn't exist
_, err := eg.TryKernel()
c.Assert(err, Equals, bootloader.ErrNoTryKernelRef)
// when a bad kernel snap name is in the extracted path, it will complain
// appropriately
kernelSnapExtractedAssetsDir := filepath.Join(s.grubDir(), "bad_snap_rev_name")
badKernelSnapPath := filepath.Join(kernelSnapExtractedAssetsDir, "kernel.efi")
tryKernelSymlink := filepath.Join(s.grubDir(), "try-kernel.efi")
err = os.MkdirAll(kernelSnapExtractedAssetsDir, 0755)
c.Assert(err, IsNil)
err = os.WriteFile(badKernelSnapPath, nil, 0644)
c.Assert(err, IsNil)
err = os.Symlink("bad_snap_rev_name/kernel.efi", tryKernelSymlink)
c.Assert(err, IsNil)
_, err = eg.TryKernel()
c.Assert(err, ErrorMatches, "cannot parse kernel snap file name from symlink target \"bad_snap_rev_name\": .*")
// remove the bad symlink
err = os.Remove(tryKernelSymlink)
c.Assert(err, IsNil)
// make a real symlink
tryKernel := s.makeKernelAssetSnapAndSymlink(c, "pc-kernel_2.snap", "try-kernel.efi")
// ensure that the returned kernel is the same as the one we put there
sn, err := eg.TryKernel()
c.Assert(err, IsNil)
c.Assert(sn, DeepEquals, tryKernel)
// if the destination of the symlink is removed, we get an error
err = os.Remove(filepath.Join(s.grubDir(), "pc-kernel_2.snap", "kernel.efi"))
c.Assert(err, IsNil)
_, err = eg.TryKernel()
c.Assert(err, ErrorMatches, "cannot read dangling symlink try-kernel.efi")
}
func (s *grubTestSuite) TestGrubExtractedRunKernelImageEnableKernel(c *C) {
s.makeFakeGrubEnv(c)
g := bootloader.NewGrub(s.rootdir, nil)
eg, ok := g.(bootloader.ExtractedRunKernelImageBootloader)
c.Assert(ok, Equals, true)
// ensure we fail to create a dangling symlink to a kernel snap that was not
// actually extracted
nonExistSnap, err := snap.ParsePlaceInfoFromSnapFileName("pc-kernel_12.snap")
c.Assert(err, IsNil)
err = eg.EnableKernel(nonExistSnap)
c.Assert(err, ErrorMatches, "cannot enable kernel.efi at pc-kernel_12.snap/kernel.efi: file does not exist")
kernel := s.makeKernelAssetSnap(c, "pc-kernel_1.snap")
// enable the Kernel we extracted
err = eg.EnableKernel(kernel)
c.Assert(err, IsNil)
// ensure that the symlink was put where we expect it
asset, err := os.Readlink(filepath.Join(s.grubDir(), "kernel.efi"))
c.Assert(err, IsNil)
c.Assert(asset, DeepEquals, filepath.Join("pc-kernel_1.snap", "kernel.efi"))
// create a new kernel snap and ensure that we can safely enable that one
// too
kernel2 := s.makeKernelAssetSnap(c, "pc-kernel_2.snap")
err = eg.EnableKernel(kernel2)
c.Assert(err, IsNil)
// ensure that the symlink was put where we expect it
asset, err = os.Readlink(filepath.Join(s.grubDir(), "kernel.efi"))
c.Assert(err, IsNil)
c.Assert(asset, DeepEquals, filepath.Join("pc-kernel_2.snap", "kernel.efi"))
}
func (s *grubTestSuite) TestGrubExtractedRunKernelImageEnableTryKernel(c *C) {
s.makeFakeGrubEnv(c)
g := bootloader.NewGrub(s.rootdir, nil)
eg, ok := g.(bootloader.ExtractedRunKernelImageBootloader)
c.Assert(ok, Equals, true)
kernel := s.makeKernelAssetSnap(c, "pc-kernel_1.snap")
// enable the Kernel we extracted
err := eg.EnableTryKernel(kernel)
c.Assert(err, IsNil)
// ensure that the symlink was put where we expect it
asset, err := os.Readlink(filepath.Join(s.grubDir(), "try-kernel.efi"))
c.Assert(err, IsNil)
c.Assert(asset, DeepEquals, filepath.Join("pc-kernel_1.snap", "kernel.efi"))
}
func (s *grubTestSuite) TestGrubExtractedRunKernelImageDisableTryKernel(c *C) {
if os.Geteuid() == 0 {
c.Skip("the test cannot be run by the root user")
}
s.makeFakeGrubEnv(c)
g := bootloader.NewGrub(s.rootdir, nil)
eg, ok := g.(bootloader.ExtractedRunKernelImageBootloader)
c.Assert(ok, Equals, true)
// trying to disable when the try-kernel.efi symlink is missing does not
// raise any errors
err := eg.DisableTryKernel()
c.Assert(err, IsNil)
// make the symlink and check that the symlink is missing afterwards
s.makeKernelAssetSnapAndSymlink(c, "pc-kernel_1.snap", "try-kernel.efi")
// make sure symlink is there
c.Assert(filepath.Join(s.grubDir(), "try-kernel.efi"), testutil.FilePresent)
err = eg.DisableTryKernel()
c.Assert(err, IsNil)
// ensure that the symlink is no longer there
c.Assert(filepath.Join(s.grubDir(), "try-kernel.efi"), testutil.FileAbsent)
c.Assert(filepath.Join(s.grubDir(), "pc-kernel_1.snap/kernel.efi"), testutil.FilePresent)
// try again but make sure that the directory cannot be written to
s.makeKernelAssetSnapAndSymlink(c, "pc-kernel_1.snap", "try-kernel.efi")
err = os.Chmod(s.grubDir(), 000)
c.Assert(err, IsNil)
defer os.Chmod(s.grubDir(), 0755)
err = eg.DisableTryKernel()
c.Assert(err, ErrorMatches, "remove .*/grub/try-kernel.efi: permission denied")
}
func (s *grubTestSuite) TestKernelExtractionRunImageKernel(c *C) {
s.makeFakeGrubEnv(c)
g := bootloader.NewGrub(s.rootdir, &bootloader.Options{Role: bootloader.RoleRunMode})
c.Assert(g, NotNil)
files := [][]string{
{"kernel.efi", "I'm a kernel"},
{"another-kernel-file", "another kernel file"},
{"meta/kernel.yaml", "version: 4.2"},
}
si := &snap.SideInfo{
RealName: "ubuntu-kernel",
Revision: snap.R(42),
}
fn := snaptest.MakeTestSnapWithFiles(c, packageKernel, files)
snapf, err := snapfile.Open(fn)
c.Assert(err, IsNil)
info, err := snap.ReadInfoFromSnapFile(snapf, si)
c.Assert(err, IsNil)
err = g.ExtractKernelAssets(info, snapf)
c.Assert(err, IsNil)
// kernel is extracted
kernefi := filepath.Join(s.bootdir, "grub", "ubuntu-kernel_42.snap", "kernel.efi")
c.Assert(kernefi, testutil.FilePresent)
// other file is not extracted
other := filepath.Join(s.bootdir, "grub", "ubuntu-kernel_42.snap", "another-kernel-file")
c.Assert(other, testutil.FileAbsent)
// ensure that removal of assets also works
err = g.RemoveKernelAssets(info)
c.Assert(err, IsNil)
exists, _, err := osutil.DirExists(filepath.Dir(kernefi))
c.Assert(err, IsNil)
c.Check(exists, Equals, false)
}
func (s *grubTestSuite) TestKernelExtractionRunImageKernelNoSlashBoot(c *C) {
// this is ubuntu-boot but during install we use the native EFI/ubuntu
// layout, same as Recovery, without the /boot mount
s.makeFakeGrubEFINativeEnv(c, nil)
g := bootloader.NewGrub(s.rootdir, &bootloader.Options{Role: bootloader.RoleRunMode, NoSlashBoot: true})
c.Assert(g, NotNil)
files := [][]string{
{"kernel.efi", "I'm a kernel"},
{"another-kernel-file", "another kernel file"},
{"meta/kernel.yaml", "version: 4.2"},
}
si := &snap.SideInfo{
RealName: "ubuntu-kernel",
Revision: snap.R(42),
}
fn := snaptest.MakeTestSnapWithFiles(c, packageKernel, files)
snapf, err := snapfile.Open(fn)
c.Assert(err, IsNil)
info, err := snap.ReadInfoFromSnapFile(snapf, si)
c.Assert(err, IsNil)
err = g.ExtractKernelAssets(info, snapf)
c.Assert(err, IsNil)
// kernel is extracted
kernefi := filepath.Join(s.rootdir, "EFI/ubuntu", "ubuntu-kernel_42.snap", "kernel.efi")
c.Assert(kernefi, testutil.FilePresent)
// other file is not extracted
other := filepath.Join(s.rootdir, "EFI/ubuntu", "ubuntu-kernel_42.snap", "another-kernel-file")
c.Assert(other, testutil.FileAbsent)
// enable the Kernel we extracted
eg, ok := g.(bootloader.ExtractedRunKernelImageBootloader)
c.Assert(ok, Equals, true)
err = eg.EnableKernel(info)
c.Assert(err, IsNil)
// ensure that the symlink was put where we expect it
asset, err := os.Readlink(filepath.Join(s.rootdir, "EFI/ubuntu", "kernel.efi"))
c.Assert(err, IsNil)
c.Assert(asset, DeepEquals, filepath.Join("ubuntu-kernel_42.snap", "kernel.efi"))
// ensure that removal of assets also works
err = g.RemoveKernelAssets(info)
c.Assert(err, IsNil)
exists, _, err := osutil.DirExists(filepath.Dir(kernefi))
c.Assert(err, IsNil)
c.Check(exists, Equals, false)
}
func (s *grubTestSuite) TestListTrustedAssetsNotForArch(c *C) {
oldArch := arch.DpkgArchitecture()
defer arch.SetArchitecture(arch.ArchitectureType(oldArch))
arch.SetArchitecture("non-existing-architecture")
s.makeFakeGrubEFINativeEnv(c, []byte(`this is
some random boot config`))
opts := &bootloader.Options{NoSlashBoot: true}
g := bootloader.NewGrub(s.rootdir, opts)
c.Assert(g, NotNil)
tg, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
ta, err := tg.TrustedAssets()
c.Check(err, ErrorMatches, `cannot find grub assets for "non-existing-architecture"`)
c.Check(ta, HasLen, 0)
}
func (s *grubTestSuite) TestListManagedAssets(c *C) {
s.makeFakeGrubEFINativeEnv(c, []byte(`this is
some random boot config`))
opts := &bootloader.Options{NoSlashBoot: true}
g := bootloader.NewGrub(s.rootdir, opts)
c.Assert(g, NotNil)
tg, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
c.Check(tg.ManagedAssets(), DeepEquals, []string{
"EFI/ubuntu/grub.cfg",
})
opts = &bootloader.Options{Role: bootloader.RoleRecovery}
tg = bootloader.NewGrub(s.rootdir, opts).(bootloader.TrustedAssetsBootloader)
c.Check(tg.ManagedAssets(), DeepEquals, []string{
"EFI/ubuntu/grub.cfg",
})
// as it called for the root fs rather than a mount point of a partition
// with boot assets
tg = bootloader.NewGrub(s.rootdir, nil).(bootloader.TrustedAssetsBootloader)
c.Check(tg.ManagedAssets(), DeepEquals, []string{
"boot/grub/grub.cfg",
})
}
func (s *grubTestSuite) TestRecoveryUpdateBootConfigNoEdition(c *C) {
// native EFI/ubuntu setup
s.makeFakeGrubEFINativeEnv(c, []byte("recovery boot script"))
opts := &bootloader.Options{Role: bootloader.RoleRecovery}
g := bootloader.NewGrub(s.rootdir, opts)
c.Assert(g, NotNil)
restore := assets.MockInternal("grub-recovery.cfg", []byte(`# Snapd-Boot-Config-Edition: 5
this is mocked grub-recovery.conf
`))
defer restore()
tg, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
// install the recovery boot script
updated, err := tg.UpdateBootConfig()
c.Assert(err, IsNil)
c.Assert(updated, Equals, false)
c.Assert(filepath.Join(s.grubEFINativeDir(), "grub.cfg"), testutil.FileEquals, `recovery boot script`)
}
func (s *grubTestSuite) TestRecoveryUpdateBootConfigUpdates(c *C) {
// native EFI/ubuntu setup
s.makeFakeGrubEFINativeEnv(c, []byte(`# Snapd-Boot-Config-Edition: 1
recovery boot script`))
opts := &bootloader.Options{Role: bootloader.RoleRecovery}
g := bootloader.NewGrub(s.rootdir, opts)
c.Assert(g, NotNil)
restore := assets.MockInternal("grub-recovery.cfg", []byte(`# Snapd-Boot-Config-Edition: 3
this is mocked grub-recovery.conf
`))
defer restore()
restore = assets.MockInternal("grub.cfg", []byte(`# Snapd-Boot-Config-Edition: 4
this is mocked grub.conf
`))
defer restore()
tg, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
// install the recovery boot script
updated, err := tg.UpdateBootConfig()
c.Assert(err, IsNil)
c.Assert(updated, Equals, true)
// the recovery boot asset was picked
c.Assert(filepath.Join(s.grubEFINativeDir(), "grub.cfg"), testutil.FileEquals, `# Snapd-Boot-Config-Edition: 3
this is mocked grub-recovery.conf
`)
}
func (s *grubTestSuite) testBootUpdateBootConfigUpdates(c *C, oldConfig, newConfig string, update bool) {
// native EFI/ubuntu setup
s.makeFakeGrubEFINativeEnv(c, []byte(oldConfig))
opts := &bootloader.Options{NoSlashBoot: true}
g := bootloader.NewGrub(s.rootdir, opts)
c.Assert(g, NotNil)
restore := assets.MockInternal("grub.cfg", []byte(newConfig))
defer restore()
tg, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
updated, err := tg.UpdateBootConfig()
c.Assert(err, IsNil)
c.Assert(updated, Equals, update)
if update {
c.Assert(filepath.Join(s.grubEFINativeDir(), "grub.cfg"), testutil.FileEquals, newConfig)
} else {
c.Assert(filepath.Join(s.grubEFINativeDir(), "grub.cfg"), testutil.FileEquals, oldConfig)
}
}
func (s *grubTestSuite) TestNoSlashBootUpdateBootConfigNoUpdateWhenNotManaged(c *C) {
oldConfig := `not managed`
newConfig := `# Snapd-Boot-Config-Edition: 3
this update is not applied
`
// the current boot config is not managed, no update applied
const updateApplied = false
s.testBootUpdateBootConfigUpdates(c, oldConfig, newConfig, updateApplied)
}
func (s *grubTestSuite) TestNoSlashBootUpdateBootConfigUpdates(c *C) {
oldConfig := `# Snapd-Boot-Config-Edition: 2
boot script
`
// edition is higher, update is applied
newConfig := `# Snapd-Boot-Config-Edition: 3
this is updated grub.cfg
`
const updateApplied = true
s.testBootUpdateBootConfigUpdates(c, oldConfig, newConfig, updateApplied)
}
func (s *grubTestSuite) TestNoSlashBootUpdateBootConfigNoUpdate(c *C) {
oldConfig := `# Snapd-Boot-Config-Edition: 2
boot script
`
// edition is lower, no update is applied
newConfig := `# Snapd-Boot-Config-Edition: 1
this is updated grub.cfg
`
const updateApplied = false
s.testBootUpdateBootConfigUpdates(c, oldConfig, newConfig, updateApplied)
}
func (s *grubTestSuite) TestNoSlashBootUpdateBootConfigSameEdition(c *C) {
oldConfig := `# Snapd-Boot-Config-Edition: 1
boot script
`
// edition is equal, no update is applied
newConfig := `# Snapd-Boot-Config-Edition: 1
this is updated grub.cfg
`
const updateApplied = false
s.testBootUpdateBootConfigUpdates(c, oldConfig, newConfig, updateApplied)
}
func (s *grubTestSuite) TestBootUpdateBootConfigTrivialErr(c *C) {
if os.Geteuid() == 0 {
c.Skip("the test cannot be run by the root user")
}
oldConfig := `# Snapd-Boot-Config-Edition: 2
boot script
`
// edition is higher, update is applied
newConfig := `# Snapd-Boot-Config-Edition: 3
this is updated grub.cfg
`
// native EFI/ubuntu setup
s.makeFakeGrubEFINativeEnv(c, []byte(oldConfig))
restore := assets.MockInternal("grub.cfg", []byte(newConfig))
defer restore()
opts := &bootloader.Options{NoSlashBoot: true}
g := bootloader.NewGrub(s.rootdir, opts)
c.Assert(g, NotNil)
tg, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
err := os.Chmod(s.grubEFINativeDir(), 0000)
c.Assert(err, IsNil)
defer os.Chmod(s.grubEFINativeDir(), 0755)
updated, err := tg.UpdateBootConfig()
c.Assert(err, ErrorMatches, "cannot load existing config asset: .*/EFI/ubuntu/grub.cfg: permission denied")
c.Assert(updated, Equals, false)
err = os.Chmod(s.grubEFINativeDir(), 0555)
c.Assert(err, IsNil)
c.Assert(filepath.Join(s.grubEFINativeDir(), "grub.cfg"), testutil.FileEquals, oldConfig)
// writing out new config fails
err = os.Chmod(s.grubEFINativeDir(), 0111)
c.Assert(err, IsNil)
updated, err = tg.UpdateBootConfig()
c.Assert(err, ErrorMatches, `open .*/EFI/ubuntu/grub.cfg\..+: permission denied`)
c.Assert(updated, Equals, false)
c.Assert(filepath.Join(s.grubEFINativeDir(), "grub.cfg"), testutil.FileEquals, oldConfig)
}
func (s *grubTestSuite) TestStaticCmdlineForGrubAsset(c *C) {
restore := assets.MockSnippetsForEdition("grub-asset:static-cmdline", []assets.ForEditions{
{FirstEdition: 2, Snippet: []byte(`static cmdline "with spaces"`)},
})
defer restore()
cmdline := bootloader.StaticCommandLineForGrubAssetEdition("grub-asset", 1)
c.Check(cmdline, Equals, ``)
cmdline = bootloader.StaticCommandLineForGrubAssetEdition("grub-asset", 2)
c.Check(cmdline, Equals, `static cmdline "with spaces"`)
cmdline = bootloader.StaticCommandLineForGrubAssetEdition("grub-asset", 4)
c.Check(cmdline, Equals, `static cmdline "with spaces"`)
}
func (s *grubTestSuite) TestCommandLineNotManaged(c *C) {
grubCfg := "boot script\n"
// native EFI/ubuntu setup
s.makeFakeGrubEFINativeEnv(c, []byte(grubCfg))
restore := assets.MockSnippetsForEdition("grub.cfg:static-cmdline", []assets.ForEditions{
{FirstEdition: 1, Snippet: []byte(`static=1`)},
{FirstEdition: 2, Snippet: []byte(`static=2`)},
})
defer restore()
restore = assets.MockSnippetsForEdition("grub-recovery.cfg:static-cmdline", []assets.ForEditions{
{FirstEdition: 1, Snippet: []byte(`static=1 recovery`)},
{FirstEdition: 2, Snippet: []byte(`static=2 recovery`)},
})
defer restore()
opts := &bootloader.Options{NoSlashBoot: true}
mg := bootloader.NewGrub(s.rootdir, opts).(bootloader.TrustedAssetsBootloader)
args, err := mg.CommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=run",
ExtraArgs: "extra",
})
c.Assert(err, IsNil)
c.Check(args, Equals, "snapd_recovery_mode=run static=1 extra")
optsRecovery := &bootloader.Options{NoSlashBoot: true, Role: bootloader.RoleRecovery}
mgr := bootloader.NewGrub(s.rootdir, optsRecovery).(bootloader.TrustedAssetsBootloader)
args, err = mgr.CommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=recover",
SystemArg: "snapd_recovery_system=1234",
ExtraArgs: "extra",
})
c.Assert(err, IsNil)
c.Check(args, Equals, "snapd_recovery_mode=recover snapd_recovery_system=1234 static=1 recovery extra")
}
func (s *grubTestSuite) TestCommandLineMocked(c *C) {
grubCfg := `# Snapd-Boot-Config-Edition: 2
boot script
`
staticCmdline := `arg1 foo=123 panic=-1 arg2="with spaces "`
staticCmdlineEdition3 := `edition=3 static args`
restore := assets.MockSnippetsForEdition("grub.cfg:static-cmdline", []assets.ForEditions{
{FirstEdition: 1, Snippet: []byte(staticCmdline)},
{FirstEdition: 3, Snippet: []byte(staticCmdlineEdition3)},
})
defer restore()
staticCmdlineRecovery := `recovery config panic=-1`
restore = assets.MockSnippetsForEdition("grub-recovery.cfg:static-cmdline", []assets.ForEditions{
{FirstEdition: 1, Snippet: []byte(staticCmdlineRecovery)},
})
defer restore()
// native EFI/ubuntu setup
s.makeFakeGrubEFINativeEnv(c, []byte(grubCfg))
optsNoSlashBoot := &bootloader.Options{NoSlashBoot: true}
g := bootloader.NewGrub(s.rootdir, optsNoSlashBoot)
c.Assert(g, NotNil)
tg, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
extraArgs := `extra_arg=1 extra_foo=-1 panic=3 baz="more spaces"`
args, err := tg.CommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=run",
ExtraArgs: extraArgs,
})
c.Assert(err, IsNil)
c.Check(args, Equals, `snapd_recovery_mode=run arg1 foo=123 panic=-1 arg2="with spaces " extra_arg=1 extra_foo=-1 panic=3 baz="more spaces"`)
// empty mode/system do not produce confusing results
args, err = tg.CommandLine(bootloader.CommandLineComponents{
ExtraArgs: extraArgs,
})
c.Assert(err, IsNil)
c.Check(args, Equals, `arg1 foo=123 panic=-1 arg2="with spaces " extra_arg=1 extra_foo=-1 panic=3 baz="more spaces"`)
// now check the recovery bootloader
optsRecovery := &bootloader.Options{NoSlashBoot: true, Role: bootloader.RoleRecovery}
mrg := bootloader.NewGrub(s.rootdir, optsRecovery).(bootloader.TrustedAssetsBootloader)
args, err = mrg.CommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=recover",
SystemArg: "snapd_recovery_system=20200202",
ExtraArgs: extraArgs,
})
c.Assert(err, IsNil)
// static command line from recovery asset
c.Check(args, Equals, `snapd_recovery_mode=recover snapd_recovery_system=20200202 recovery config panic=-1 extra_arg=1 extra_foo=-1 panic=3 baz="more spaces"`)
// try with a different edition
grubCfg3 := `# Snapd-Boot-Config-Edition: 3
boot script
`
s.makeFakeGrubEFINativeEnv(c, []byte(grubCfg3))
tg = bootloader.NewGrub(s.rootdir, optsNoSlashBoot).(bootloader.TrustedAssetsBootloader)
c.Assert(g, NotNil)
extraArgs = `extra_arg=1`
args, err = tg.CommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=run",
ExtraArgs: extraArgs,
})
c.Assert(err, IsNil)
c.Check(args, Equals, `snapd_recovery_mode=run edition=3 static args extra_arg=1`)
// full args set overrides static arguments
args, err = tg.CommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=run",
FullArgs: "full for run mode",
})
c.Assert(err, IsNil)
c.Check(args, Equals, `snapd_recovery_mode=run full for run mode`)
args, err = mrg.CommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=recover",
SystemArg: "snapd_recovery_system=20200202",
FullArgs: "full for recover mode",
})
c.Assert(err, IsNil)
c.Check(args, Equals, `snapd_recovery_mode=recover snapd_recovery_system=20200202 full for recover mode`)
}
func (s *grubTestSuite) TestCandidateCommandLineMocked(c *C) {
grubCfg := `# Snapd-Boot-Config-Edition: 1
boot script
`
// edition on disk
s.makeFakeGrubEFINativeEnv(c, []byte(grubCfg))
edition2 := []byte(`# Snapd-Boot-Config-Edition: 2`)
edition3 := []byte(`# Snapd-Boot-Config-Edition: 3`)
edition4 := []byte(`# Snapd-Boot-Config-Edition: 4`)
restore := assets.MockInternal("grub.cfg", edition2)
defer restore()
restore = assets.MockInternal("grub-recovery.cfg", edition2)
defer restore()
restore = assets.MockSnippetsForEdition("grub.cfg:static-cmdline", []assets.ForEditions{
{FirstEdition: 1, Snippet: []byte(`edition=1`)},
{FirstEdition: 3, Snippet: []byte(`edition=3`)},
})
defer restore()
restore = assets.MockSnippetsForEdition("grub-recovery.cfg:static-cmdline", []assets.ForEditions{
{FirstEdition: 1, Snippet: []byte(`recovery edition=1`)},
{FirstEdition: 3, Snippet: []byte(`recovery edition=3`)},
{FirstEdition: 4, Snippet: []byte(`recovery edition=4up`)},
})
defer restore()
optsNoSlashBoot := &bootloader.Options{NoSlashBoot: true}
mg := bootloader.NewGrub(s.rootdir, optsNoSlashBoot).(bootloader.TrustedAssetsBootloader)
optsRecovery := &bootloader.Options{NoSlashBoot: true, Role: bootloader.RoleRecovery}
recoverymg := bootloader.NewGrub(s.rootdir, optsRecovery).(bootloader.TrustedAssetsBootloader)
args, err := mg.CandidateCommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=run",
ExtraArgs: "extra=1",
})
c.Assert(err, IsNil)
c.Check(args, Equals, `snapd_recovery_mode=run edition=1 extra=1`)
args, err = recoverymg.CandidateCommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=recover",
SystemArg: "snapd_recovery_system=20200202",
ExtraArgs: "extra=1",
})
c.Assert(err, IsNil)
c.Check(args, Equals, `snapd_recovery_mode=recover snapd_recovery_system=20200202 recovery edition=1 extra=1`)
restore = assets.MockInternal("grub.cfg", edition3)
defer restore()
restore = assets.MockInternal("grub-recovery.cfg", edition3)
defer restore()
args, err = mg.CandidateCommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=run",
ExtraArgs: "extra=1",
})
c.Assert(err, IsNil)
c.Check(args, Equals, `snapd_recovery_mode=run edition=3 extra=1`)
args, err = recoverymg.CandidateCommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=recover",
SystemArg: "snapd_recovery_system=20200202",
ExtraArgs: "extra=1",
})
c.Assert(err, IsNil)
c.Check(args, Equals, `snapd_recovery_mode=recover snapd_recovery_system=20200202 recovery edition=3 extra=1`)
// bump edition only for recovery
restore = assets.MockInternal("grub-recovery.cfg", edition4)
defer restore()
// boot bootloader unchanged
args, err = mg.CandidateCommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=run",
ExtraArgs: "extra=1",
})
c.Assert(err, IsNil)
c.Check(args, Equals, `snapd_recovery_mode=run edition=3 extra=1`)
// recovery uses a new edition
args, err = recoverymg.CandidateCommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=recover",
SystemArg: "snapd_recovery_system=20200202",
ExtraArgs: "extra=1",
})
c.Assert(err, IsNil)
c.Check(args, Equals, `snapd_recovery_mode=recover snapd_recovery_system=20200202 recovery edition=4up extra=1`)
// the static snippet is ignored when using full arg set
args, err = recoverymg.CandidateCommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=recover",
SystemArg: "snapd_recovery_system=20200202",
FullArgs: "full args set",
})
c.Assert(err, IsNil)
c.Check(args, Equals, `snapd_recovery_mode=recover snapd_recovery_system=20200202 full args set`)
}
func (s *grubTestSuite) TestCommandLineReal(c *C) {
grubCfg := `# Snapd-Boot-Config-Edition: 1
boot script
`
// native EFI/ubuntu setup
s.makeFakeGrubEFINativeEnv(c, []byte(grubCfg))
opts := &bootloader.Options{NoSlashBoot: true}
g := bootloader.NewGrub(s.rootdir, opts)
c.Assert(g, NotNil)
tg, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
extraArgs := "foo bar baz=1"
args, err := tg.CommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=run",
ExtraArgs: extraArgs,
})
c.Assert(err, IsNil)
c.Check(args, Equals, `snapd_recovery_mode=run console=ttyS0 console=tty1 panic=-1 foo bar baz=1`)
// with full args the static part is not used
args, err = tg.CommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=run",
FullArgs: "full for run mode",
})
c.Assert(err, IsNil)
c.Check(args, Equals, `snapd_recovery_mode=run full for run mode`)
// now check the recovery bootloader
opts = &bootloader.Options{NoSlashBoot: true, Role: bootloader.RoleRecovery}
mrg := bootloader.NewGrub(s.rootdir, opts).(bootloader.TrustedAssetsBootloader)
args, err = mrg.CommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=recover",
SystemArg: "snapd_recovery_system=20200202",
ExtraArgs: extraArgs,
})
c.Assert(err, IsNil)
// static command line from recovery asset
c.Check(args, Equals, `snapd_recovery_mode=recover snapd_recovery_system=20200202 console=ttyS0 console=tty1 panic=-1 foo bar baz=1`)
// similarly, when passed full args, the static part is not used
args, err = mrg.CommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=recover",
SystemArg: "snapd_recovery_system=20200202",
FullArgs: "full for recover mode",
})
c.Assert(err, IsNil)
c.Check(args, Equals, `snapd_recovery_mode=recover snapd_recovery_system=20200202 full for recover mode`)
}
func (s *grubTestSuite) TestCommandLineComponentsValidate(c *C) {
grubCfg := `# Snapd-Boot-Config-Edition: 1
boot script
`
// native EFI/ubuntu setup
s.makeFakeGrubEFINativeEnv(c, []byte(grubCfg))
opts := &bootloader.Options{NoSlashBoot: true}
g := bootloader.NewGrub(s.rootdir, opts)
c.Assert(g, NotNil)
tg, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
args, err := tg.CommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=run",
ExtraArgs: "extra is set",
FullArgs: "full is set",
})
c.Assert(err, ErrorMatches, "cannot use both full and extra components of command line")
c.Check(args, Equals, "")
// invalid for the candidate command line too
args, err = tg.CandidateCommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=run",
ExtraArgs: "extra is set",
FullArgs: "full is set",
})
c.Assert(err, ErrorMatches, "cannot use both full and extra components of command line")
c.Check(args, Equals, "")
// now check the recovery bootloader
opts = &bootloader.Options{NoSlashBoot: true, Role: bootloader.RoleRecovery}
mrg := bootloader.NewGrub(s.rootdir, opts).(bootloader.TrustedAssetsBootloader)
args, err = mrg.CommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=recover",
SystemArg: "snapd_recovery_system=20200202",
ExtraArgs: "extra is set",
FullArgs: "full is set",
})
c.Assert(err, ErrorMatches, "cannot use both full and extra components of command line")
c.Check(args, Equals, "")
// candidate recovery command line is checks validity of the components too
args, err = mrg.CandidateCommandLine(bootloader.CommandLineComponents{
ModeArg: "snapd_recovery_mode=recover",
SystemArg: "snapd_recovery_system=20200202",
ExtraArgs: "extra is set",
FullArgs: "full is set",
})
c.Assert(err, ErrorMatches, "cannot use both full and extra components of command line")
c.Check(args, Equals, "")
}
func (s *grubTestSuite) TestTrustedAssetsNativePartitionLayout(c *C) {
// native EFI/ubuntu setup
s.makeFakeGrubEFINativeEnv(c, []byte("grub.cfg"))
opts := &bootloader.Options{NoSlashBoot: true}
g := bootloader.NewGrub(s.rootdir, opts)
c.Assert(g, NotNil)
tab, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
ta, err := tab.TrustedAssets()
c.Assert(err, IsNil)
c.Check(ta, DeepEquals, map[string]string{
"EFI/boot/grubx64.efi": "grubx64.efi",
})
// recovery bootloader
recoveryOpts := &bootloader.Options{NoSlashBoot: true, Role: bootloader.RoleRecovery}
tarb := bootloader.NewGrub(s.rootdir, recoveryOpts).(bootloader.TrustedAssetsBootloader)
c.Assert(tarb, NotNil)
ta, err = tarb.TrustedAssets()
c.Assert(err, IsNil)
c.Check(ta, DeepEquals, map[string]string{
"EFI/boot/bootx64.efi": "bootx64.efi",
"EFI/boot/grubx64.efi": "grubx64.efi",
"EFI/ubuntu/shimx64.efi": "ubuntu:shimx64.efi",
"EFI/ubuntu/grubx64.efi": "ubuntu:grubx64.efi",
})
// recovery bootloader, with fallback implemented
s.makeFakeShimFallback(c)
tarb = bootloader.NewGrub(s.rootdir, recoveryOpts).(bootloader.TrustedAssetsBootloader)
c.Assert(tarb, NotNil)
ta, err = tarb.TrustedAssets()
c.Assert(err, IsNil)
c.Check(ta, DeepEquals, map[string]string{
"EFI/ubuntu/shimx64.efi": "ubuntu:shimx64.efi",
"EFI/ubuntu/grubx64.efi": "ubuntu:grubx64.efi",
"EFI/boot/bootx64.efi": "bootx64.efi",
"EFI/boot/grubx64.efi": "grubx64.efi",
})
}
func (s *grubTestSuite) TestTrustedAssetsRoot(c *C) {
s.makeFakeGrubEnv(c)
g := bootloader.NewGrub(s.rootdir, nil)
tab, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
ta, err := tab.TrustedAssets()
c.Assert(err, ErrorMatches, "internal error: trusted assets called without native host-partition layout")
c.Check(ta, IsNil)
}
func (s *grubTestSuite) TestTrustedAssetsFailAtPrepareImageTime(c *C) {
// native EFI/ubuntu setup
s.makeFakeGrubEFINativeEnv(c, []byte("grub.cfg"))
opts := []bootloader.Options{
{NoSlashBoot: true, PrepareImageTime: true},
{NoSlashBoot: true, PrepareImageTime: true, Role: bootloader.RoleRecovery}}
for _, opt := range opts {
g := bootloader.NewGrub(s.rootdir, &opt)
c.Assert(g, NotNil)
tab, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
ta, err := tab.TrustedAssets()
c.Assert(err, ErrorMatches, "internal error: retrieving boot assets at prepare image time")
c.Check(ta, IsNil)
}
}
func (s *grubTestSuite) TestRecoveryBootChains(c *C) {
s.makeFakeGrubEFINativeEnv(c, nil)
g := bootloader.NewGrub(s.rootdir, &bootloader.Options{Role: bootloader.RoleRecovery})
tab, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
chains, err := tab.RecoveryBootChains("kernel.snap")
c.Assert(err, IsNil)
c.Assert(chains, HasLen, 2)
c.Assert(chains[0], DeepEquals, []bootloader.BootFile{
{Path: "EFI/ubuntu/shimx64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/ubuntu/grubx64.efi", Role: bootloader.RoleRecovery},
{Snap: "kernel.snap", Path: "kernel.efi", Role: bootloader.RoleRecovery},
})
c.Assert(chains[1], DeepEquals, []bootloader.BootFile{
{Path: "EFI/boot/bootx64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/boot/grubx64.efi", Role: bootloader.RoleRecovery},
{Snap: "kernel.snap", Path: "kernel.efi", Role: bootloader.RoleRecovery},
})
}
func (s *grubTestSuite) TestRecoveryBootChainsWithFallback(c *C) {
s.makeFakeShimFallback(c)
s.makeFakeGrubEFINativeEnv(c, nil)
g := bootloader.NewGrub(s.rootdir, &bootloader.Options{Role: bootloader.RoleRecovery})
tab, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
chains, err := tab.RecoveryBootChains("kernel.snap")
c.Assert(err, IsNil)
c.Assert(chains, HasLen, 2)
c.Assert(chains[0], DeepEquals, []bootloader.BootFile{
{Path: "EFI/ubuntu/shimx64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/ubuntu/grubx64.efi", Role: bootloader.RoleRecovery},
{Snap: "kernel.snap", Path: "kernel.efi", Role: bootloader.RoleRecovery},
})
c.Assert(chains[1], DeepEquals, []bootloader.BootFile{
{Path: "EFI/boot/bootx64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/boot/grubx64.efi", Role: bootloader.RoleRecovery},
{Snap: "kernel.snap", Path: "kernel.efi", Role: bootloader.RoleRecovery},
})
}
func (s *grubTestSuite) TestRecoveryBootChainsNotRecoveryBootloader(c *C) {
s.makeFakeGrubEnv(c)
g := bootloader.NewGrub(s.rootdir, nil)
tab, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
_, err := tab.RecoveryBootChains("kernel.snap")
c.Assert(err, ErrorMatches, "not a recovery bootloader")
}
func (s *grubTestSuite) TestBootChains(c *C) {
s.makeFakeGrubEFINativeEnv(c, nil)
g := bootloader.NewGrub(s.rootdir, &bootloader.Options{Role: bootloader.RoleRecovery})
tab, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
g2 := bootloader.NewGrub(s.rootdir, &bootloader.Options{Role: bootloader.RoleRunMode})
chains, err := tab.BootChains(g2, "kernel.snap")
c.Assert(err, IsNil)
c.Assert(chains, HasLen, 2)
c.Assert(chains[0], DeepEquals, []bootloader.BootFile{
{Path: "EFI/ubuntu/shimx64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/ubuntu/grubx64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/boot/grubx64.efi", Role: bootloader.RoleRunMode},
{Snap: "kernel.snap", Path: "kernel.efi", Role: bootloader.RoleRunMode},
})
c.Assert(chains[1], DeepEquals, []bootloader.BootFile{
{Path: "EFI/boot/bootx64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/boot/grubx64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/boot/grubx64.efi", Role: bootloader.RoleRunMode},
{Snap: "kernel.snap", Path: "kernel.efi", Role: bootloader.RoleRunMode},
})
}
func (s *grubTestSuite) TestBootChainsWithFallback(c *C) {
s.makeFakeShimFallback(c)
s.makeFakeGrubEFINativeEnv(c, nil)
g := bootloader.NewGrub(s.rootdir, &bootloader.Options{Role: bootloader.RoleRecovery})
tab, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
g2 := bootloader.NewGrub(s.rootdir, &bootloader.Options{Role: bootloader.RoleRunMode})
chains, err := tab.BootChains(g2, "kernel.snap")
c.Assert(err, IsNil)
c.Assert(chains, HasLen, 2)
c.Assert(chains[0], DeepEquals, []bootloader.BootFile{
{Path: "EFI/ubuntu/shimx64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/ubuntu/grubx64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/boot/grubx64.efi", Role: bootloader.RoleRunMode},
{Snap: "kernel.snap", Path: "kernel.efi", Role: bootloader.RoleRunMode},
})
c.Assert(chains[1], DeepEquals, []bootloader.BootFile{
{Path: "EFI/boot/bootx64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/boot/grubx64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/boot/grubx64.efi", Role: bootloader.RoleRunMode},
{Snap: "kernel.snap", Path: "kernel.efi", Role: bootloader.RoleRunMode},
})
}
func (s *grubTestSuite) TestBootChainsArm64(c *C) {
s.makeFakeGrubEFINativeEnv(c, nil)
r := archtest.MockArchitecture("arm64")
defer r()
g := bootloader.NewGrub(s.rootdir, &bootloader.Options{Role: bootloader.RoleRecovery})
tab, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
g2 := bootloader.NewGrub(s.rootdir, &bootloader.Options{Role: bootloader.RoleRunMode})
chains, err := tab.BootChains(g2, "kernel.snap")
c.Assert(err, IsNil)
c.Assert(chains, HasLen, 2)
c.Assert(chains[0], DeepEquals, []bootloader.BootFile{
{Path: "EFI/ubuntu/shimaa64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/ubuntu/grubaa64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/boot/grubaa64.efi", Role: bootloader.RoleRunMode},
{Snap: "kernel.snap", Path: "kernel.efi", Role: bootloader.RoleRunMode},
})
c.Assert(chains[1], DeepEquals, []bootloader.BootFile{
{Path: "EFI/boot/bootaa64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/boot/grubaa64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/boot/grubaa64.efi", Role: bootloader.RoleRunMode},
{Snap: "kernel.snap", Path: "kernel.efi", Role: bootloader.RoleRunMode},
})
}
func (s *grubTestSuite) TestBootChainsArm64WithFallback(c *C) {
s.makeFakeGrubEFINativeEnv(c, nil)
s.makeFakeShimFallback(c)
r := archtest.MockArchitecture("arm64")
defer r()
g := bootloader.NewGrub(s.rootdir, &bootloader.Options{Role: bootloader.RoleRecovery})
tab, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
g2 := bootloader.NewGrub(s.rootdir, &bootloader.Options{Role: bootloader.RoleRunMode})
chains, err := tab.BootChains(g2, "kernel.snap")
c.Assert(err, IsNil)
c.Assert(chains, HasLen, 2)
c.Assert(chains[0], DeepEquals, []bootloader.BootFile{
{Path: "EFI/ubuntu/shimaa64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/ubuntu/grubaa64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/boot/grubaa64.efi", Role: bootloader.RoleRunMode},
{Snap: "kernel.snap", Path: "kernel.efi", Role: bootloader.RoleRunMode},
})
c.Assert(chains[1], DeepEquals, []bootloader.BootFile{
{Path: "EFI/boot/bootaa64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/boot/grubaa64.efi", Role: bootloader.RoleRecovery},
{Path: "EFI/boot/grubaa64.efi", Role: bootloader.RoleRunMode},
{Snap: "kernel.snap", Path: "kernel.efi", Role: bootloader.RoleRunMode},
})
}
func (s *grubTestSuite) TestBootChainsNotRecoveryBootloader(c *C) {
s.makeFakeGrubEnv(c)
g := bootloader.NewGrub(s.rootdir, nil)
tab, ok := g.(bootloader.TrustedAssetsBootloader)
c.Assert(ok, Equals, true)
g2 := bootloader.NewGrub(s.rootdir, &bootloader.Options{NoSlashBoot: true, Role: bootloader.RoleRunMode})
_, err := tab.BootChains(g2, "kernel.snap")
c.Assert(err, ErrorMatches, "not a recovery bootloader")
}
func (s *grubTestSuite) TestConstructShimEfiLoadOption(c *C) {
s.makeFakeGrubEFINativeEnv(c, nil)
g := bootloader.NewGrub(s.rootdir, &bootloader.Options{Role: bootloader.RoleRecovery})
ubl, ok := g.(bootloader.UefiBootloader)
c.Assert(ok, Equals, true)
description, assetPath, optionalData, err := ubl.ParametersForEfiLoadOption([]string{"bootx64.efi", "boot:fbx64.efi", "ubuntu:shimx64.efi"})
c.Assert(err, IsNil)
c.Assert(description, Equals, "ubuntu")
c.Assert(assetPath, Equals, fmt.Sprintf("%s/EFI/ubuntu/shimx64.efi", s.rootdir))
c.Assert(optionalData, HasLen, 0)
}
func (s *grubTestSuite) TestConstructShimEfiLoadOptionOldBootchain(c *C) {
s.makeFakeGrubEFINativeEnv(c, nil)
g := bootloader.NewGrub(s.rootdir, &bootloader.Options{Role: bootloader.RoleRecovery})
ubl, ok := g.(bootloader.UefiBootloader)
c.Assert(ok, Equals, true)
description, assetPath, optionalData, err := ubl.ParametersForEfiLoadOption([]string{"bootx64.efi"})
c.Assert(err, IsNil)
c.Assert(description, Equals, "ubuntu")
c.Assert(assetPath, Equals, fmt.Sprintf("%s/EFI/boot/bootx64.efi", s.rootdir))
c.Assert(optionalData, HasLen, 0)
}