UPSTREAM: <carry>: re-enable disabled HPA tests

This commit re-enables HPA tests by removing it from the disabled_tests.go specs.

This also prevents HPA tests from running when using from the kubernetes/openshift OTE suites.
They will now also be skipped by default when running the openshift-hack/test-kubernetes-e2e.sh script.
This is because these tests use a ton of resources and flake in a high concurrency environment.

Instead, a new HPA-only prow job will be configured to run these tests, with a lower concurrency.

Signed-off-by: Max Cao <[email protected]>

Author: maxcao13

openshift-hack/cmd/k8s-tests-ext/disabled_tests.go

@@ -161,9 +161,6 @@ func filterOutDisabledSpecs(specs et.ExtensionTestSpecs) et.ExtensionTestSpecs {
 
 			// https://issues.redhat.com/browse/OCPBUGS-45275
 			"[sig-network] Connectivity Pod Lifecycle should be able to connect to other Pod from a terminating Pod",
-
-			// https://issues.redhat.com/browse/OCPBUGS-61376
-			"[sig-autoscaling] [Feature:HPA] Horizontal pod autoscaling",
 		},
 		// tests that need to be temporarily disabled while the rebase is in progress.
 		"RebaseInProgress": {
@@ -212,8 +209,6 @@ func filterOutDisabledSpecs(specs et.ExtensionTestSpecs) et.ExtensionTestSpecs {
 		// tests that are known flaky
 		"Flaky": {
 			"Job should run a job to completion when tasks sometimes fail and are not locally restarted", // seems flaky, also may require too many resources
-			// TODO(node): test works when run alone, but not in the suite in CI
-			"[Feature:HPA] Horizontal pod autoscaling (scale resource: CPU) [sig-autoscaling] ReplicationController light Should scale from 1 pod to 2 pods",
 		},
 	}
 

openshift-hack/cmd/k8s-tests-ext/k8s-tests.go

@@ -160,6 +160,7 @@ func withExcludedTestsFilter(baseExpr string) string {
 		"[Slow]",
 		"[Flaky]",
 		"[Local]",
+		"[Feature:HPA]", // We enable these tests through its own HPA prow job
 	}
 
 	filter := ""

openshift-hack/cmd/k8s-tests-ext/labels.go

@@ -17,8 +17,6 @@ func addLabelsToSpecs(specs et.ExtensionTestSpecs) {
 
 			"Should be able to support the 1.7 Sample API Server using the current Aggregator", // down apiservices break other clients today https://bugzilla.redhat.com/show_bug.cgi?id=1623195
 
-			"[Feature:HPA] Horizontal pod autoscaling (scale resource: CPU) [sig-autoscaling] ReplicationController light Should scale from 1 pod to 2 pods",
-
 			"should prevent Ingress creation if more than 1 IngressClass marked as default", // https://bugzilla.redhat.com/show_bug.cgi?id=1822286
 
 			"[sig-network] IngressClass [Feature:Ingress] should set default value on new IngressClass", //https://bugzilla.redhat.com/show_bug.cgi?id=1833583

openshift-hack/test-kubernetes-e2e.sh

@@ -32,6 +32,8 @@ TEST_SUITE="${TEST_SUITE:-parallel}"
 COMMON_SKIPS="\[Slow\]|\[Disruptive\]|\[Flaky\]|\[Disabled:.+\]|\[Skipped:${PLATFORM}\]|${NETWORK_SKIPS}"
 # Skip tests for features that require a TechPreview cluster. TODO: Remove when the feature is enabled by default.
 COMMON_SKIPS="\[OCPFeatureGate:VolumeGroupSnapshot\]|\[Feature:OffByDefault\]|\[DRA\]|${COMMON_SKIPS}"
+# Skip HPA tests because they use a heavy resources in parallel and flake. There is a separate HPA prow job that runs these tests.
+COMMON_SKIPS="${COMMON_SKIPS}|\[Feature:HPA\]"
 
 case "${TEST_SUITE}" in
 serial)