search

openshift / origin

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

USHIFT-3143: Skip metrics and adjust termination message policy monitor tests #28787

Closed pacevedom closed 1 month ago

pacevedom commented 1 month ago

MicroShift does not support metrics server yet. It is unclear how it will do when it does, so for now skip this test.

openshift-ci-robot commented 1 month ago

@pacevedom: This pull request references USHIFT-3143 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 task to target the "4.16.0" version, but no target version was set.

In response to [this](https://github.com/openshift/origin/pull/28787): >MicroShift does not support metrics server yet. It is unclear how it will do when it does, so for now skip this 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.
pacevedom commented 1 month ago

/cc @dgoodwin

pacevedom commented 1 month ago

/retest-required

pacevedom commented 1 month ago

/hold

pacevedom commented 1 month ago

/hold cancel

dgoodwin commented 1 month ago

/lgtm

We need to get a presubmit for microshift in here soon.

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD ac69ee533bf789dac59e9e62c29110e28284a6f8 and 2 for PR HEAD 855fe84fa60c22baee88401c3d4bdcbf2ed2d6de in total

pacevedom commented 1 month ago

/hold Need to investigate a thing about these tests.

pacevedom commented 1 month ago

/cc @dgoodwin ptal again, I changed one of the tests to allow it to work on MicroShift.

dgoodwin commented 1 month ago

/lgtm

openshift-ci[bot] commented 1 month ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dgoodwin, pacevedom

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)~~ [dgoodwin] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
pacevedom commented 1 month ago

/hold cancel

pacevedom commented 1 month ago

/retest-required

pacevedom commented 1 month ago

/retest-required

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD fa6c7e625ed785cf169ab9185a5826ff753cd50c and 2 for PR HEAD cb99e593d9d7e6adbdf78a1475cb031393ee873d in total

pacevedom commented 1 month ago

/retest-required

pacevedom commented 1 month ago

/retest-required

pacevedom commented 1 month ago

/retest-required

dgoodwin commented 1 month ago

I'm not totally sure what's happening here but I fear your presubmits are stuck running code that was reverted last week. Those panics should have been resolved, and I don't see them firing anywhere else. We've seen this before where PRs get "stuck" on old code and have to be reopened as a new PR. Honestly I think that's what I'd recommend at this point.

openshift-trt-bot commented 1 month ago

Job Failure Risk Analysis for sha: cb99e593d9d7e6adbdf78a1475cb031393ee873d

Job Name Failure Risk
pull-ci-openshift-origin-master-e2e-aws-ovn-single-node-upgrade High
[sig-arch] events should not repeat pathologically for ns/openshift-kube-apiserver-operator
This test has passed 99.67% of 3938 runs on release 4.17 [Overall] in the last week.
pull-ci-openshift-origin-master-e2e-aws-ovn-cgroupsv2 Medium
Undiagnosed panic detected in pod
This test has passed 91.67% of 12 runs on jobs ['periodic-ci-openshift-release-master-nightly-4.17-e2e-aws-ovn-cgroupsv2'] in the last 14 days.
pull-ci-openshift-origin-master-e2e-aws-csi Medium
Undiagnosed panic detected in pod
This test has passed 97.71% of 2625 runs on release 4.17 [Overall] in the last week.
pull-ci-openshift-origin-master-e2e-agnostic-ovn-cmd Medium
Undiagnosed panic detected in pod
This test has passed 97.22% of 72 runs on release 4.17 [amd64 azure ha ovn] in the last week.
pull-ci-openshift-origin-master-e2e-aws-ovn-single-node Low
Undiagnosed panic detected in pod
This test has passed 76.47% of 17 runs on release 4.17 [amd64 aws ovn single-node] in the last week.
pacevedom commented 1 month ago

/retest-required

openshift-ci[bot] commented 1 month ago

New changes are detected. LGTM label has been removed.

openshift-ci[bot] commented 1 month ago

@pacevedom: 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 c37bbe146ffe40c6cee08cfc447b479e89f5daa2 link false /test e2e-aws-ovn-single-node-serial
ci/prow/e2e-agnostic-ovn-cmd c37bbe146ffe40c6cee08cfc447b479e89f5daa2 link false /test e2e-agnostic-ovn-cmd
ci/prow/e2e-aws-ovn-fips c37bbe146ffe40c6cee08cfc447b479e89f5daa2 link true /test e2e-aws-ovn-fips

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).