Skip to content

Commit 4b57a68

Browse files
sdodsonclaude
andcommitted
OCPBUGS-8933: run resource request/limit check early
Add [Early] to the "should set requests but not limits" test so it runs in the early test bucket before the main parallel test suites. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 7fd854e commit 4b57a68

3 files changed

Lines changed: 3 additions & 3 deletions

File tree

pkg/test/ginkgo/testNames.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -248,7 +248,7 @@
248248
"[sig-arch] Managed cluster should ensure pods use downstream images from our release image with proper ImagePullPolicy [Suite:openshift/conformance/parallel]"
249249
"[sig-arch] Managed cluster should have operators on the cluster version [apigroup:config.openshift.io] [Suite:openshift/conformance/parallel]"
250250
"[sig-arch] Managed cluster should only include cluster daemonsets that have maxUnavailable or maxSurge update of 10 percent or maxUnavailable of 33 percent [apigroup:config.openshift.io] [Suite:openshift/conformance/parallel]"
251-
"[sig-arch] Managed cluster should set requests but not limits [Suite:openshift/conformance/parallel]"
251+
"[sig-arch] Managed cluster should set requests but not limits [Early] [Suite:openshift/conformance/parallel]"
252252
"[sig-arch] [Conformance] FIPS TestFIPS [apigroup:config.openshift.io] [Suite:openshift/conformance/parallel/minimal]"
253253
"[sig-arch] [Conformance] sysctl pod should not start for sysctl not on whitelist kernel.msgmax [Suite:openshift/conformance/parallel/minimal]"
254254
"[sig-arch] [Conformance] sysctl pod should not start for sysctl not on whitelist net.ipv4.ip_dynaddr [Suite:openshift/conformance/parallel/minimal]"

pkg/test/ginkgo/test_summaries.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@
127127
},
128128
{
129129
"Release": "4.22",
130-
"TestName": "[sig-arch] Managed cluster should set requests but not limits [Suite:openshift/conformance/parallel]",
130+
"TestName": "[sig-arch] Managed cluster should set requests but not limits [Early] [Suite:openshift/conformance/parallel]",
131131
"TotalTestCount": 11392,
132132
"TotalFailureCount": 138,
133133
"TotalFlakeCount": 695,

test/extended/operators/resources.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ var _ = g.Describe("[sig-arch] Managed cluster", func() {
2929
//
3030
// Release architects can justify an exception with text but must ensure CONVENTIONS.md is updated to document
3131
// why the exception is granted.
32-
g.It("should set requests but not limits", func() {
32+
g.It("should set requests but not limits [Early]", func() {
3333
pods, err := oc.KubeFramework().ClientSet.CoreV1().Pods("").List(context.Background(), metav1.ListOptions{})
3434
if err != nil {
3535
e2e.Failf("unable to list pods: %v", err)

0 commit comments

Comments
 (0)