From 76df8acb8007f1f6dc587bcf39afafb0ec30d980 Mon Sep 17 00:00:00 2001 From: Peter Hunt Date: Thu, 18 Nov 2021 09:24:05 -0500 Subject: [PATCH] test: update major page fault values for summary test as well as use a variable instead of a constant Signed-off-by: Peter Hunt --- test/e2e_node/summary_test.go | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/test/e2e_node/summary_test.go b/test/e2e_node/summary_test.go index 8a652e0fe23..8a451b7d82d 100644 --- a/test/e2e_node/summary_test.go +++ b/test/e2e_node/summary_test.go @@ -116,7 +116,7 @@ var _ = SIGDescribe("Summary API [NodeConformance]", func() { }) } expectedPageFaultsUpperBound := 1000000 - expectedMajorPageFaultsUpperBound := 10 + expectedMajorPageFaultsUpperBound := 15 if IsCgroup2UnifiedMode() { // On cgroupv2 these stats are recursive, so make sure they are at least like the value set // above for the container. @@ -200,8 +200,8 @@ var _ = SIGDescribe("Summary API [NodeConformance]", func() { "UsageBytes": bounded(10*e2evolume.Kb, 80*e2evolume.Mb), "WorkingSetBytes": bounded(10*e2evolume.Kb, 80*e2evolume.Mb), "RSSBytes": bounded(1*e2evolume.Kb, 80*e2evolume.Mb), - "PageFaults": bounded(100, 1000000), - "MajorPageFaults": bounded(0, 10), + "PageFaults": bounded(100, expectedPageFaultsUpperBound), + "MajorPageFaults": bounded(0, expectedMajorPageFaultsUpperBound), }), "Accelerators": gomega.BeEmpty(), "Rootfs": ptrMatchAllFields(gstruct.Fields{ @@ -247,8 +247,8 @@ var _ = SIGDescribe("Summary API [NodeConformance]", func() { "UsageBytes": bounded(10*e2evolume.Kb, 80*e2evolume.Mb), "WorkingSetBytes": bounded(10*e2evolume.Kb, 80*e2evolume.Mb), "RSSBytes": bounded(1*e2evolume.Kb, 80*e2evolume.Mb), - "PageFaults": bounded(0, 1000000), - "MajorPageFaults": bounded(0, 10), + "PageFaults": bounded(0, expectedPageFaultsUpperBound), + "MajorPageFaults": bounded(0, expectedMajorPageFaultsUpperBound), }), "VolumeStats": gstruct.MatchAllElements(summaryObjectID, gstruct.Elements{ "test-empty-dir": gstruct.MatchAllFields(gstruct.Fields{