openshift / origin

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

virt: Skip operator unhealthy alarm #29305

Open qinqon opened 3 days ago

qinqon commented 3 days ago

During openshift virt the operator emits an alarm until all the sub components are not healthy, this change skip that alarm so monitor test do not fail.

qinqon commented 3 days ago

/test e2e-aws-ovn-virt-techpreview

qinqon commented 2 days ago

/retest-required

openshift-ci[bot] commented 2 days ago

@qinqon: 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-upgrade 133dc70c62e24f752d34b83fa4bedeb571367629 link false /test e2e-aws-ovn-single-node-upgrade
ci/prow/e2e-aws-ovn-single-node 133dc70c62e24f752d34b83fa4bedeb571367629 link false /test e2e-aws-ovn-single-node
ci/prow/e2e-agnostic-ovn-cmd 133dc70c62e24f752d34b83fa4bedeb571367629 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-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).
qinqon commented 1 day ago

/cc @stbenjam Can you take a look ?

qinqon commented 1 day ago

/cherry-pick release-4.18

openshift-cherrypick-robot commented 1 day ago

@qinqon: once the present PR merges, I will cherry-pick it on top of release-4.18 in a new PR and assign it to you.

In response to [this](https://github.com/openshift/origin/pull/29305#issuecomment-2490352830): >/cherry-pick release-4.18 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.
qinqon commented 1 day ago

/hold

Looks like there is a fix upstream -> https://github.com/kubevirt/hyperconverged-cluster-operator/pull/3170

stbenjam commented 21 hours ago

/lgtm

openshift-ci[bot] commented 21 hours ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: maiqueb, qinqon, stbenjam

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)~~ [stbenjam] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment