Skip to content

Commit 246d363

Browse files
authored
Merge pull request kubernetes#121700 from kannon92/fix-summary-more
missed a few summary upper limits for major page faults
2 parents cf4d031 + baebc37 commit 246d363

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

test/e2e_node/summary_test.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -120,12 +120,12 @@ var _ = SIGDescribe("Summary API", framework.WithNodeConformance(), func() {
120120
})
121121
}
122122
expectedPageFaultsUpperBound := 1000000
123-
expectedMajorPageFaultsUpperBound := 15
123+
expectedMajorPageFaultsUpperBound := 1e9
124124
if IsCgroup2UnifiedMode() {
125125
// On cgroupv2 these stats are recursive, so make sure they are at least like the value set
126126
// above for the container.
127127
expectedPageFaultsUpperBound = 1e9
128-
expectedMajorPageFaultsUpperBound = 100000
128+
expectedMajorPageFaultsUpperBound = 1e9
129129
}
130130

131131
podsContExpectations := sysContExpectations().(*gstruct.FieldsMatcher)
@@ -158,7 +158,7 @@ var _ = SIGDescribe("Summary API", framework.WithNodeConformance(), func() {
158158
"WorkingSetBytes": bounded(100*e2evolume.Kb, memoryLimit),
159159
"RSSBytes": bounded(100*e2evolume.Kb, memoryLimit),
160160
"PageFaults": bounded(1000, 1e9),
161-
"MajorPageFaults": bounded(0, 100000),
161+
"MajorPageFaults": bounded(0, 1e9),
162162
})
163163
systemContainers["misc"] = miscContExpectations
164164
}
@@ -281,7 +281,7 @@ var _ = SIGDescribe("Summary API", framework.WithNodeConformance(), func() {
281281
// this now returns /sys/fs/cgroup/memory.stat total_rss
282282
"RSSBytes": bounded(1*e2evolume.Kb, memoryLimit),
283283
"PageFaults": bounded(1000, 1e9),
284-
"MajorPageFaults": bounded(0, 100000),
284+
"MajorPageFaults": bounded(0, 1e9),
285285
}),
286286
"Swap": swapExpectation(memoryLimit),
287287
// TODO(#28407): Handle non-eth0 network interface names.

0 commit comments

Comments
 (0)