search

openshift / origin

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

NO-ISSUE: automated update synthetic test data #28693

Closed DennisPeriquet closed 3 weeks ago

DennisPeriquet commented 3 weeks ago

This is https://github.com/openshift/origin/pull/28642 with a change to the unit test to match azure instead of aws where we can meet the minJobs requirement.

dgoodwin commented 3 weeks ago

/retitle NO-ISSUE: automated update synthetic test data /lgtm

openshift-ci-robot commented 3 weeks ago

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

In response to [this](https://github.com/openshift/origin/pull/28693): >This is https://github.com/openshift/origin/pull/28642 with a change to the unit test. 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-robot commented 3 weeks ago

/retest-required

Remaining retests: 0 against base HEAD 2d18729439228c125441ad977c5a8a195ba87a16 and 2 for PR HEAD 5988528d2e60071d9546c0b1dcae983245f78166 in total

openshift-ci-robot commented 3 weeks ago

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

In response to [this](https://github.com/openshift/origin/pull/28693): >This is https://github.com/openshift/origin/pull/28642 with a change to the unit test to match `azure` instead of `aws` where we can meet the minJobs requirement. 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.
dgoodwin commented 3 weeks ago

/lgtm

DennisPeriquet commented 3 weeks ago

this PR is about the query_results.json files and a unit test; the unit test passes so the failure I'm overriding contributes nothing to testing.

/override ci/prow/e2e-gcp-ovn-upgrade

openshift-ci[bot] commented 3 weeks ago

@DennisPeriquet: Overrode contexts on behalf of DennisPeriquet: ci/prow/e2e-gcp-ovn-upgrade

In response to [this](https://github.com/openshift/origin/pull/28693#issuecomment-2040430273): >this PR is about the query_results.json files and a unit test; the unit test passes so the failure I'm overriding contributes nothing to testing. > >/override ci/prow/e2e-gcp-ovn-upgrade 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/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
DennisPeriquet commented 3 weeks ago

/label lgtm

openshift-ci[bot] commented 3 weeks ago

@DennisPeriquet: The label(s) /label lgtm cannot be applied. These labels are supported: acknowledge-critical-fixes-only, platform/aws, platform/azure, platform/baremetal, platform/google, platform/libvirt, platform/openstack, ga, tide/merge-method-merge, tide/merge-method-rebase, tide/merge-method-squash, px-approved, docs-approved, qe-approved, no-qe, downstream-change-needed, rebase/manual, cluster-config-api-changed, approved, backport-risk-assessed, bugzilla/valid-bug, cherry-pick-approved, jira/valid-bug, staff-eng-approved. Is this label configured under labels -> additional_labels or labels -> restricted_labels in plugin.yaml?

In response to [this](https://github.com/openshift/origin/pull/28693#issuecomment-2040488856): >/label lgtm 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/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
neisw commented 3 weeks ago

/lgtm

openshift-ci[bot] commented 3 weeks ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: DennisPeriquet, dgoodwin, neisw

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)~~ [DennisPeriquet,dgoodwin,neisw] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
openshift-ci-robot commented 3 weeks ago

/retest-required

Remaining retests: 0 against base HEAD 6d7739a4dc36916b02dec3e23cfe420ce10623a4 and 2 for PR HEAD dc7f5d7a146c80d597669c46f66bc6b2c7d91396 in total

openshift-trt-bot commented 3 weeks ago

Job Failure Risk Analysis for sha: dc7f5d7a146c80d597669c46f66bc6b2c7d91396

Job Name Failure Risk
pull-ci-openshift-origin-master-e2e-metal-ipi-ovn-ipv6 IncompleteTests
Tests for this run (19) are below the historical average (1007): 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-master-e2e-aws-ovn-single-node-serial Low
[sig-arch] events should not repeat pathologically for ns/openshift-etcd-operator
This test has passed 30.77% of 65 runs on jobs ['periodic-ci-openshift-release-master-nightly-4.16-e2e-aws-ovn-single-node-serial'] in the last 14 days.
DennisPeriquet commented 3 weeks ago

the metal job does not contribute to testing of this PR; only the unit test matters.

/override ci/prow/e2e-metal-ipi-ovn-ipv6

openshift-ci[bot] commented 3 weeks ago

@DennisPeriquet: Overrode contexts on behalf of DennisPeriquet: ci/prow/e2e-metal-ipi-ovn-ipv6

In response to [this](https://github.com/openshift/origin/pull/28693#issuecomment-2041056665): >the metal job does not contribute to testing of this PR; only the unit test matters. > >/override ci/prow/e2e-metal-ipi-ovn-ipv6 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/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
openshift-ci[bot] commented 3 weeks ago

@DennisPeriquet: 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-single-node-serial dc7f5d7a146c80d597669c46f66bc6b2c7d91396 link false /test e2e-aws-ovn-single-node-serial
ci/prow/e2e-aws-ovn-single-node dc7f5d7a146c80d597669c46f66bc6b2c7d91396 link false /test e2e-aws-ovn-single-node
ci/prow/e2e-aws-ovn-single-node-upgrade dc7f5d7a146c80d597669c46f66bc6b2c7d91396 link false /test e2e-aws-ovn-single-node-upgrade
ci/prow/e2e-agnostic-ovn-cmd dc7f5d7a146c80d597669c46f66bc6b2c7d91396 link false /test e2e-agnostic-ovn-cmd

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/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
openshift-bot commented 3 weeks ago

[ART PR BUILD NOTIFIER]

This PR has been included in build openshift-enterprise-tests-container-v4.16.0-202404061145.p0.g1972e66.assembly.stream.el8 for distgit openshift-enterprise-tests. All builds following this will include this PR.