Skip to content

Commit

Permalink
More comments
Browse files Browse the repository at this point in the history
Signed-off-by: Itxaka <[email protected]>
  • Loading branch information
Itxaka committed Jan 31, 2025
1 parent 7db9c88 commit 6ac4c8a
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
5 changes: 3 additions & 2 deletions pkg/stages/stages.go
Original file line number Diff line number Diff line change
Expand Up @@ -157,6 +157,7 @@ func GetInstallStage(sis values.System, logger types.KairosLogger) ([]schema.Sta
}

// For trusted boot we need to select the correct kernel packages manually
// TODO: Have a flag in the config to add the full linux-firmware package?
if config.DefaultConfig.TrustedBoot {
// TODO: Check for other distros/families
if sis.Distro == values.Ubuntu {
Expand Down Expand Up @@ -380,8 +381,8 @@ func GetCleanupStage(sis values.System, l types.KairosLogger) []schema.Stage {
Remove: filteredPkgs,
},
},
{
Name: "Autoremove packages",
{ // TODO: Send this upstream to the yip Packages plugin?
Name: "Autoremove packages in Debian family",
OnlyIfOs: "Ubuntu.*|Debian.*",
Commands: []string{
"apt-get autoremove -y",
Expand Down
4 changes: 3 additions & 1 deletion pkg/values/packagemaps.go
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,9 @@ var KernelPackages = PackageMap{
}

// KernelPackagesTrustedBoot Separated kernel package for trusted boot as we dont want to install the same packages on both variants
// we need to keep teh trusted boot variant as small as possible so we cwant more control over it
// we need to keep teh trusted boot variant as small as possible so we want more control over it
// In this case, only Ubuntu has an specific smallest kernel package as its the only distro that supports trusted boot
// Fedora also works but we havent make it slim yet
var KernelPackagesTrustedBoot = PackageMap{
Debian: {
ArchAMD64: {
Expand Down

0 comments on commit 6ac4c8a

Please sign in to comment.