Skip to content

Commit 220665b

Browse files
authored
Merge pull request #5654 from sdodson/ocpbugs-76444
OCPBUGS-76444: pkg/daemon/update: Remove kernel-rt-kvm
2 parents 285e0ce + 8576459 commit 220665b

1 file changed

Lines changed: 3 additions & 25 deletions

File tree

pkg/daemon/update.go

Lines changed: 3 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -1741,10 +1741,7 @@ func (dn *CoreOSDaemon) switchKernel(oldConfig, newConfig *mcfgv1.MachineConfig)
17411741
logSystem("Re-applying kernel type %s", newKtype)
17421742
}
17431743

1744-
kernelPackages, err := dn.getKernelPackagesForTargetRelease()
1745-
if err != nil {
1746-
return fmt.Errorf("failed to get kernel packages for target release: %w", err)
1747-
}
1744+
kernelPackages := dn.getKernelPackagesForTargetRelease()
17481745
if newKtype == ctrlcommon.KernelTypeRealtime {
17491746
// Switch to RT kernel
17501747
args := []string{"override", "remove"}
@@ -1768,36 +1765,17 @@ func (dn *CoreOSDaemon) switchKernel(oldConfig, newConfig *mcfgv1.MachineConfig)
17681765
}
17691766

17701767
// getKernelPackagesForTargetRelease returns the list of kernel packaged for the running OS release.
1771-
func (dn *CoreOSDaemon) getKernelPackagesForTargetRelease() (releaseKernelPackages, error) {
1768+
func (dn *CoreOSDaemon) getKernelPackagesForTargetRelease() releaseKernelPackages {
17721769
// TODO: Drop this code and use https://github.com/coreos/rpm-ostree/issues/2542 instead
17731770

1774-
// Fetch the OS deployments to infer the target OS version from them
1775-
booted, staged, err := dn.NodeUpdaterClient.GetBootedAndStagedDeployment()
1776-
if err != nil {
1777-
return releaseKernelPackages{}, fmt.Errorf("error fetching OS deployments : %v", err)
1778-
}
1779-
1780-
// If there's a staged deployment the packages will be installed in it instead of in the current booted deployment
1781-
targetDeployment := booted
1782-
if staged != nil {
1783-
targetDeployment = staged
1784-
}
1785-
17861771
kernelPackages := releaseKernelPackages{
17871772
defaultKernel: []string{"kernel", "kernel-core", "kernel-modules", "kernel-modules-core", "kernel-modules-extra"},
17881773
hugePagesKernel: []string{"kernel-64k-core", "kernel-64k-modules", "kernel-64k-modules-core", "kernel-64k-modules-extra"},
17891774
// Note this list explicitly does *not* include kernel-rt as that is a meta-package that tries to pull in a lot
17901775
// of other dependencies we don't want for historical reasons.
17911776
realtimeKernel: []string{"kernel-rt-core", "kernel-rt-modules", "kernel-rt-modules-extra"},
17921777
}
1793-
1794-
// RHEL10 early bugfix of OCPBUGS-62925
1795-
// RHEL10 doesn't ship with kernel-rt-kvm
1796-
targetVersion := NewTargetOSVersionFromDeployment(targetDeployment)
1797-
if !targetVersion.IsEL10() {
1798-
kernelPackages.realtimeKernel = append(kernelPackages.realtimeKernel, "kernel-rt-kvm")
1799-
}
1800-
return kernelPackages, nil
1778+
return kernelPackages
18011779
}
18021780

18031781
// updateFiles writes files specified by the nodeconfig to disk. it also writes

0 commit comments

Comments
 (0)