search

openshift / origin

Conformance test suite for OpenShift
http://www.openshift.org
Apache License 2.0
8.49k stars 4.7k forks source link

NO-JIRA: Introduce version annotation test for Windows nodes #29190

Closed jrvaldes closed 2 weeks ago

jrvaldes commented 1 month ago

This PR proposes a new group of tests targeting Windows nodes in the test cluster and introduces a simple Windows-specific check as the first step in providing test coverage for Windows nodes.

openshift-ci-robot commented 1 month ago

@jrvaldes: This pull request explicitly references no jira issue.

In response to [this](https://github.com/openshift/origin/pull/29190): >This PR proposes a new group of tests targeting Windows nodes in the test cluster and introduces a simple Windows-specific check as the first step in providing test coverage for Windows nodes. Instructions for interacting with me using PR comments are available [here](https://prow.ci.openshift.org/command-help?repo=openshift%2Forigin). If you have questions or suggestions related to my behavior, please file an issue against the [openshift-eng/jira-lifecycle-plugin](https://github.com/openshift-eng/jira-lifecycle-plugin/issues/new) repository.
openshift-ci[bot] commented 1 month ago

Skipping CI for Draft Pull Request. If you want CI signal for your change, please convert it to an actual PR. You can still manually trigger a test run with /test all

jrvaldes commented 1 month ago

/CC @deads2k

jrvaldes commented 1 month ago

/test ?

openshift-ci[bot] commented 1 month ago

@jrvaldes: The following commands are available to trigger required jobs:

The following commands are available to trigger optional jobs:

Use /test all to run the following jobs that were automatically triggered:

In response to [this](https://github.com/openshift/origin/pull/29190#issuecomment-2412762877): >/test ? Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository.
jrvaldes commented 1 month ago

/test lint

jrvaldes commented 1 month ago

/test unit

jrvaldes commented 1 month ago

/test verify

deads2k commented 1 month ago

/approve

jrvaldes commented 1 month ago

/retest-required

jrvaldes commented 1 month ago

/tide refresh

jrvaldes commented 1 month ago

/CC @openshift/openshift-team-windows-containers

openshift-ci[bot] commented 1 month ago

@jrvaldes: GitHub didn't allow me to request PR reviews from the following users: openshift/openshift-team-windows-containers.

Note that only openshift members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to [this](https://github.com/openshift/origin/pull/29190#issuecomment-2413963597): >/CC @openshift/openshift-team-windows-containers Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository.
saifshaikh48 commented 1 month ago

/lgtm

Thanks for doing this @jrvaldes!

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD 65c5d9d86c3e15806ac7a6cce50bd5de84481421 and 2 for PR HEAD aa74be85c112a147118590c25c9766ee0f0064b0 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD 96fd23e3633a833d5780a2a1135bb7759d926e1b and 1 for PR HEAD aa74be85c112a147118590c25c9766ee0f0064b0 in total

openshift-trt-bot commented 1 month ago

Job Failure Risk Analysis for sha: aa74be85c112a147118590c25c9766ee0f0064b0

Job Name Failure Risk
pull-ci-openshift-origin-master-e2e-metal-ipi-ovn-ipv6 IncompleteTests
Tests for this run (21) are below the historical average (2233): IncompleteTests (not enough tests ran to make a reasonable risk analysis; this could be due to infra, installation, or upgrade problems)
openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD 9303d0ee0a1543c088f5a9285a5405b5d10a3ad7 and 0 for PR HEAD aa74be85c112a147118590c25c9766ee0f0064b0 in total

openshift-ci[bot] commented 1 month ago

@jrvaldes: 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-metal-ipi-ovn aa74be85c112a147118590c25c9766ee0f0064b0 link false /test e2e-metal-ipi-ovn
ci/prow/e2e-aws-ovn-single-node aa74be85c112a147118590c25c9766ee0f0064b0 link false /test e2e-aws-ovn-single-node
ci/prow/e2e-aws-ovn-ipsec-serial aa74be85c112a147118590c25c9766ee0f0064b0 link false /test e2e-aws-ovn-ipsec-serial
ci/prow/e2e-metal-ipi-ovn-kube-apiserver-rollout aa74be85c112a147118590c25c9766ee0f0064b0 link false /test e2e-metal-ipi-ovn-kube-apiserver-rollout

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
sebsoto commented 1 month ago

/lgtm

openshift-ci[bot] commented 1 month ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: deads2k, jrvaldes, saifshaikh48, sebsoto

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

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/openshift/origin/blob/master/OWNERS)~~ [deads2k] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
jrvaldes commented 1 month ago

@deads2k please consider overriding the failed tests as they are unrelated to the proposed change.

jrvaldes commented 1 month ago

/retest-required

jrvaldes commented 1 month ago

/retest-required

jrvaldes commented 1 month ago

/tide refresh

jrvaldes commented 1 month ago

/test e2e-aws-ovn-ipsec-serial

openshift-ci[bot] commented 1 month ago

@jrvaldes: The specified target(s) for /test were not found. The following commands are available to trigger required jobs:

The following commands are available to trigger optional jobs:

Use /test all to run the following jobs that were automatically triggered:

In response to [this](https://github.com/openshift/origin/pull/29190#issuecomment-2419659745): >/test e2e-aws-ovn-ipsec-serial Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository.
jrvaldes commented 1 month ago

@deads2k please consider overriding the failed tests as they are unrelated to the proposed change.

@celebdor could you help with the failed tests?

jrvaldes commented 1 month ago

/retest-required

deads2k commented 2 weeks ago

/override ci/prow/e2e-aws-ovn-ipsec-serial

openshift-ci[bot] commented 2 weeks ago

@deads2k: Overrode contexts on behalf of deads2k: ci/prow/e2e-aws-ovn-ipsec-serial

In response to [this](https://github.com/openshift/origin/pull/29190#issuecomment-2458183729): >/override ci/prow/e2e-aws-ovn-ipsec-serial Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository.
jrvaldes commented 2 weeks ago

/tide refresh

openshift-bot commented 2 weeks ago

[ART PR BUILD NOTIFIER]

Distgit: openshift-enterprise-tests This PR has been included in build openshift-enterprise-tests-container-v4.18.0-202411060509.p0.g4a1cead.assembly.stream.el9. All builds following this will include this PR.