From ffb71c9e5a20743cfc0182e7633f90939db2ef6e Mon Sep 17 00:00:00 2001 From: Scott Dodson Date: Thu, 12 Mar 2026 16:50:46 -0400 Subject: [PATCH 1/2] OCPBUGS-78485: 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 --- pkg/test/ginkgo/testNames.txt | 2 +- test/extended/operators/resources.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/test/ginkgo/testNames.txt b/pkg/test/ginkgo/testNames.txt index 22e9e13e01b6..dd8d63508746 100644 --- a/pkg/test/ginkgo/testNames.txt +++ b/pkg/test/ginkgo/testNames.txt @@ -248,7 +248,7 @@ "[sig-arch] Managed cluster should ensure pods use downstream images from our release image with proper ImagePullPolicy [Suite:openshift/conformance/parallel]" "[sig-arch] Managed cluster should have operators on the cluster version [apigroup:config.openshift.io] [Suite:openshift/conformance/parallel]" "[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]" -"[sig-arch] Managed cluster should set requests but not limits [Suite:openshift/conformance/parallel]" +"[sig-arch] Managed cluster should set requests but not limits [Early] [Suite:openshift/conformance/parallel]" "[sig-arch] [Conformance] FIPS TestFIPS [apigroup:config.openshift.io] [Suite:openshift/conformance/parallel/minimal]" "[sig-arch] [Conformance] sysctl pod should not start for sysctl not on whitelist kernel.msgmax [Suite:openshift/conformance/parallel/minimal]" "[sig-arch] [Conformance] sysctl pod should not start for sysctl not on whitelist net.ipv4.ip_dynaddr [Suite:openshift/conformance/parallel/minimal]" diff --git a/test/extended/operators/resources.go b/test/extended/operators/resources.go index 5a1cd87c0329..aa7db1c53b49 100644 --- a/test/extended/operators/resources.go +++ b/test/extended/operators/resources.go @@ -29,7 +29,7 @@ var _ = g.Describe("[sig-arch] Managed cluster", func() { // // Release architects can justify an exception with text but must ensure CONVENTIONS.md is updated to document // why the exception is granted. - g.It("should set requests but not limits", func() { + g.It("should set requests but not limits [Early]", func() { pods, err := oc.KubeFramework().ClientSet.CoreV1().Pods("").List(context.Background(), metav1.ListOptions{}) if err != nil { e2e.Failf("unable to list pods: %v", err) From 5518828c199a34cf2a721b0f2d027330ecd53966 Mon Sep 17 00:00:00 2001 From: Scott Dodson Date: Thu, 12 Mar 2026 16:50:46 -0400 Subject: [PATCH 2/2] OCPBUGS-78485: 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 --- .../util/annotate/generated/zz_generated.annotations.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/extended/util/annotate/generated/zz_generated.annotations.go b/test/extended/util/annotate/generated/zz_generated.annotations.go index cbe9b4ad1b08..75486c2552e9 100644 --- a/test/extended/util/annotate/generated/zz_generated.annotations.go +++ b/test/extended/util/annotate/generated/zz_generated.annotations.go @@ -271,7 +271,7 @@ var Annotations = map[string]string{ "[sig-arch] Managed cluster should recover when operator-owned objects are deleted [Disruptive][apigroup:config.openshift.io]": " [Serial]", - "[sig-arch] Managed cluster should set requests but not limits": " [Suite:openshift/conformance/parallel]", + "[sig-arch] Managed cluster should set requests but not limits [Early]": " [Suite:openshift/conformance/parallel]", "[sig-arch] [Conformance] FIPS TestFIPS": " [Suite:openshift/conformance/parallel/minimal]",