Skip to content

Conversation

@maxcao13
Copy link
Member

WIP

@openshift-ci-robot openshift-ci-robot added the backports/unvalidated-commits Indicates that not all commits come to merged upstream PRs. label Oct 21, 2025
@openshift-ci-robot
Copy link

@maxcao13: the contents of this pull request could not be automatically validated.

The following commits could not be validated and must be approved by a top-level approver:

Comment /validate-backports to re-evaluate validity of the upstream PRs, for example when they are merged upstream.

@openshift-ci openshift-ci bot requested review from benluddy and bertinatto October 21, 2025 03:37
@openshift-ci
Copy link

openshift-ci bot commented Oct 21, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: maxcao13
Once this PR has been reviewed and has the lgtm label, please assign bertinatto for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@maxcao13 maxcao13 changed the title tests(hpa): re-enable HPA tests in OTE suite WIP: OCPBUGS-61376: re-enable HPA tests in OTE suite Oct 21, 2025
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 21, 2025
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Oct 21, 2025
@openshift-ci-robot
Copy link

@maxcao13: This pull request references Jira Issue OCPBUGS-61376, which is valid. The bug has been moved to the POST state.

3 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target version (4.21.0) matches configured target version for branch (4.21.0)
  • bug is in the state ASSIGNED, which is one of the valid states (NEW, ASSIGNED, POST)

Requesting review from QA contact:
/cc @prozehna

The bug has been updated to refer to the pull request using the external bug tracker.

In response to this:

WIP

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci-robot openshift-ci-robot added the jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. label Oct 21, 2025
@openshift-ci openshift-ci bot requested a review from prozehna October 21, 2025 03:39
@maxcao13
Copy link
Member Author

/retest

@maxcao13
Copy link
Member Author

maxcao13 commented Oct 21, 2025

Yup HPA tests are definitely still failing in parallel. They are still running because of the Parents field I think?:

	// Run HPA specific tests which are designed to be run in parallel
	kubeTestsExtension.AddSuite(e.Suite{
		Name: "kubernetes/autoscaling/hpa",
		Parents: []string{
			"openshift/conformance/parallel",
		},
		Qualifiers: []string{`name.contains("[Feature:HPA]")`},
	})

I need to remove that, but then this requires a separate ci-job for HPA tests to execute at all.

I'm not inclined to have a separate suite just for HPA tests because then we would need to have HPA specific jobs for every platform...? Or maybe we can just get away with an AWS only job since HPA shouldn't be any different on other platforms.

I would rather fix this, by just having some sort of isolation within the current k8s conformance suites where HPA tests run in parallel in a separate execution isolated from all other parallel tests, but from what I understand this is not possible with how OTE works right now.

EDIT: I think we can separate HPA tests in isolation with, but this means that if you run all conformance parallel tests normally without openshift-tests binary, HPA tests will still fail. I think most of our CI jobs uses openshift-tests though, so hopefully not a problem?

@openshift-ci-robot
Copy link

@maxcao13: the contents of this pull request could not be automatically validated.

The following commits could not be validated and must be approved by a top-level approver:

Comment /validate-backports to re-evaluate validity of the upstream PRs, for example when they are merged upstream.

@openshift-ci-robot
Copy link

@maxcao13: the contents of this pull request could not be automatically validated.

The following commits could not be validated and must be approved by a top-level approver:

Comment /validate-backports to re-evaluate validity of the upstream PRs, for example when they are merged upstream.

@maxcao13
Copy link
Member Author

I don't know if this will work, but let's try:

/testwith openshift/kubernetes/master/e2e-gcp openshift/origin#30411

@maxcao13
Copy link
Member Author

maxcao13 commented Oct 22, 2025

/testwith openshift/kubernetes/master/e2e-gcp openshift/origin#30411

This is failing and running into openshift/ci-tools#4769

@maxcao13
Copy link
Member Author

/retest

HPA tests have now been isolated when using openshift-tests. 🤞

@maxcao13
Copy link
Member Author

/test e2e-gcp

@openshift-ci
Copy link

openshift-ci bot commented Oct 28, 2025

@maxcao13: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-aws-ovn-cgroupsv2 862d391 link true /test e2e-aws-ovn-cgroupsv2
ci/prow/e2e-aws-ovn-techpreview 862d391 link false /test e2e-aws-ovn-techpreview
ci/prow/e2e-aws-ovn-techpreview-serial 862d391 link false /test e2e-aws-ovn-techpreview-serial
ci/prow/e2e-aws-ovn-runc 862d391 link true /test e2e-aws-ovn-runc
ci/prow/okd-scos-e2e-aws-ovn 862d391 link false /test okd-scos-e2e-aws-ovn
ci/prow/e2e-metal-ipi-ovn-ipv6 862d391 link true /test e2e-metal-ipi-ovn-ipv6
ci/prow/e2e-aws-crun-wasm 862d391 link true /test e2e-aws-crun-wasm
ci/prow/e2e-aws-ovn-serial 862d391 link true /test e2e-aws-ovn-serial
ci/prow/e2e-aws-ovn-crun 862d391 link true /test e2e-aws-ovn-crun
ci/prow/k8s-e2e-gcp-serial 862d391 link true /test k8s-e2e-gcp-serial
ci/prow/e2e-aws-ovn-fips 862d391 link true /test e2e-aws-ovn-fips
ci/prow/e2e-gcp 862d391 link true /test e2e-gcp

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backports/unvalidated-commits Indicates that not all commits come to merged upstream PRs. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants