Checkup, vmi spec: Don't use host huge pages

There is no need to use the host's huge pages as the memory backend of the vm under test. Signed-off-by: Orel Misan <omisan@redhat.com>
kiagnose · Nov 23, 2023 · 6315667 · 6315667
1 parent d0ae21d
commit 6315667
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/pkg/internal/checkup/checkup.go b/pkg/internal/checkup/checkup.go
@@ -171,7 +171,6 @@ func newRealtimeVMI(checkupConfig config.Config) *kvcorev1.VirtualMachineInstanc
 		CPUSocketsCount   = 1
 		CPUCoresCount     = 3
 		CPUTreadsCount    = 1
-		hugePageSize      = "1Gi"
 		guestMemory       = "4Gi"
 		rootDiskName      = "rootdisk"
 		cloudInitDiskName = "cloudinitdisk"
@@ -187,7 +186,7 @@ user: user`
 		vmi.WithoutCRIOCPUQuota(),
 		vmi.WithoutCRIOIRQLoadBalancing(),
 		vmi.WithRealtimeCPU(CPUSocketsCount, CPUCoresCount, CPUTreadsCount),
-		vmi.WithMemory(hugePageSize, guestMemory),
+		vmi.WithMemory(guestMemory),
 		vmi.WithoutAutoAttachGraphicsDevice(),
 		vmi.WithoutAutoAttachMemBalloon(),
 		vmi.WithAutoAttachSerialConsole(),

diff --git a/pkg/internal/checkup/vmi/vmi.go b/pkg/internal/checkup/vmi/vmi.go
@@ -159,12 +159,11 @@ func WithVirtIODisk(name string) Option {
 	}
 }
 
-func WithMemory(hugePageSize, guestMemory string) Option {
+func WithMemory(guestMemory string) Option {
 	return func(vmi *kvcorev1.VirtualMachineInstance) {
 		guestMemoryQuantity := resource.MustParse(guestMemory)
 		vmi.Spec.Domain.Memory = &kvcorev1.Memory{
-			Hugepages: &kvcorev1.Hugepages{PageSize: hugePageSize},
-			Guest:     &guestMemoryQuantity,
+			Guest: &guestMemoryQuantity,
 		}
 	}
 }