Skip to content

Conversation

jianlinliu
Copy link
Contributor

@jianlinliu jianlinliu commented Sep 18, 2025

  1. Per slack discussion, we are going to add one more test case - "ensure 1 worker node at least gets ready", once the case fails, it will skip the following operators health check.
  2. Always create junit directory if it does not exist

@openshift-ci openshift-ci bot requested review from deads2k and sjenning September 18, 2025 15:03
Copy link
Contributor

openshift-ci bot commented Sep 18, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: jianlinliu
Once this PR has been reviewed and has the lgtm label, please assign dennisperiquet 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

@jianlinliu jianlinliu changed the title add one test case in e2e-analysis TRT-2275: add one more test in e2e-analysis Sep 18, 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 Sep 18, 2025
@openshift-ci-robot
Copy link

openshift-ci-robot commented Sep 18, 2025

@jianlinliu: This pull request references TRT-2275 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.21.0" version, but no target version was set.

In response to this:

Per slack discussion, we are going to add one more test case - "ensure 1 worker node at least gets ready", once the case fails, it will skip the following operators health check.

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.

@jianlinliu jianlinliu changed the title TRT-2275: add one more test in e2e-analysis some enhancement for in e2e-analysis Sep 24, 2025
@openshift-ci-robot
Copy link

@jianlinliu: No Jira issue is referenced in the title of this pull request.
To reference a jira issue, add 'XYZ-NNN:' to the title of this pull request and request another refresh with /jira refresh.

In response to this:

Per slack discussion, we are going to add one more test case - "ensure 1 worker node at least gets ready", once the case fails, it will skip the following operators health check.

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 removed the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Sep 24, 2025
@jianlinliu jianlinliu changed the title some enhancement for in e2e-analysis some enhancement for e2e-analysis Sep 24, 2025
if workerReadyCount > 0 {
tm.AddTestCase(tc, "", "")
} else {
message := "No any Ready and Scheduable worker node"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No -> Not any Ready...

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apologies, I wish I had paid more attention to this when we were reviewing. A better message would be:

No Schedulable worker nodes available

@neisw
Copy link
Contributor

neisw commented Sep 25, 2025

/retest-required

Copy link
Contributor

openshift-ci bot commented Sep 25, 2025

@jianlinliu: 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-gcp-ovn-techpreview-serial-2of2 60f2a81 link false /test e2e-gcp-ovn-techpreview-serial-2of2
ci/prow/e2e-metal-ipi-ovn-dualstack 60f2a81 link false /test e2e-metal-ipi-ovn-dualstack
ci/prow/e2e-aws-disruptive 60f2a81 link false /test e2e-aws-disruptive
ci/prow/e2e-azure 60f2a81 link false /test e2e-azure
ci/prow/e2e-gcp-ovn-techpreview 60f2a81 link false /test e2e-gcp-ovn-techpreview
ci/prow/e2e-aws-ovn-edge-zones 26d342d link false /test e2e-aws-ovn-edge-zones
ci/prow/e2e-aws-ovn-microshift 26d342d link true /test e2e-aws-ovn-microshift
ci/prow/okd-scos-e2e-aws-ovn 26d342d link false /test okd-scos-e2e-aws-ovn
ci/prow/e2e-gcp-ovn-rt-upgrade 26d342d link false /test e2e-gcp-ovn-rt-upgrade
ci/prow/e2e-metal-ipi-ovn 26d342d link false /test e2e-metal-ipi-ovn
ci/prow/e2e-metal-ipi-ovn-kube-apiserver-rollout 26d342d link false /test e2e-metal-ipi-ovn-kube-apiserver-rollout
ci/prow/e2e-openstack-ovn 26d342d link false /test e2e-openstack-ovn
ci/prow/e2e-aws-ovn-microshift-serial 26d342d link true /test e2e-aws-ovn-microshift-serial
ci/prow/e2e-aws-ovn-single-node 26d342d link false /test e2e-aws-ovn-single-node

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.

Copy link

openshift-trt bot commented Sep 25, 2025

Job Failure Risk Analysis for sha: 26d342d

Job Name Failure Risk
pull-ci-openshift-origin-main-e2e-aws-ovn-microshift IncompleteTests
Tests for this run (22) are below the historical average (1283): IncompleteTests (not enough tests ran to make a reasonable risk analysis; this could be due to infra, installation, or upgrade problems)
pull-ci-openshift-origin-main-e2e-aws-ovn-microshift-serial IncompleteTests
Tests for this run (22) are below the historical average (650): IncompleteTests (not enough tests ran to make a reasonable risk analysis; this could be due to infra, installation, or upgrade problems)
pull-ci-openshift-origin-main-e2e-metal-ipi-ovn IncompleteTests
Tests for this run (103) are below the historical average (2743): IncompleteTests (not enough tests ran to make a reasonable risk analysis; this could be due to infra, installation, or upgrade problems)
pull-ci-openshift-origin-main-e2e-metal-ipi-ovn-kube-apiserver-rollout IncompleteTests
Tests for this run (104) are below the historical average (1555): IncompleteTests (not enough tests ran to make a reasonable risk analysis; this could be due to infra, installation, or upgrade problems)
pull-ci-openshift-origin-main-e2e-openstack-ovn IncompleteTests
Tests for this run (104) are below the historical average (1885): IncompleteTests (not enough tests ran to make a reasonable risk analysis; this could be due to infra, installation, or upgrade problems)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants