openshift-kni / performance-addon-operators

Operators related to optimizing OpenShift clusters for applications sensitive to cpu and network latency
Apache License 2.0
46 stars 60 forks source link

[release-4.10] e2e: stabilize performance update tests #948

Closed openshift-cherrypick-robot closed 2 years ago

openshift-cherrypick-robot commented 2 years ago

This is an automated cherry-pick of #861

/assign shajmakh

coveralls commented 2 years ago

Pull Request Test Coverage Report for Build 2625


Totals Coverage Status
Change from base Build 2621: 0.0%
Covered Lines: 1592
Relevant Lines: 2630

💛 - Coveralls
shajmakh commented 2 years ago

This is needed because kubelet-update tests are failing to update the MCP.

openshift-ci[bot] commented 2 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fromanirh, openshift-cherrypick-robot

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-kni/performance-addon-operators/blob/release-4.10/OWNERS)~~ [fromanirh] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
shajmakh commented 2 years ago

fixes the CI failures: https://github.com/openshift-kni/performance-addon-operators/pull/949 I'd prefer to override those two failing jobs so we don't need to rebase the cherry-pick branch of the bot, which will need to be done manually. Rather merge this one #948 and then merge #949 so if any rebase is needed it will be simpler.

ffromani commented 2 years ago

/override e2e-gcp /override e2e-gcp-operator-upgrade

openshift-ci[bot] commented 2 years ago

@fromanirh: /override requires failed status contexts, check run or a prowjob name to operate on. The following unknown contexts/checkruns were given:

Only the following failed contexts/checkruns were expected:

If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.

In response to [this](https://github.com/openshift-kni/performance-addon-operators/pull/948#issuecomment-1249346123): >/override e2e-gcp >/override e2e-gcp-operator-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.
ffromani commented 2 years ago

/override ci/prow/e2e-gcp

openshift-ci[bot] commented 2 years ago

@fromanirh: Overrode contexts on behalf of fromanirh: ci/prow/e2e-gcp

In response to [this](https://github.com/openshift-kni/performance-addon-operators/pull/948#issuecomment-1249347804): >/override ci/prow/e2e-gcp 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.
ffromani commented 2 years ago

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

openshift-ci[bot] commented 2 years ago

@fromanirh: Overrode contexts on behalf of fromanirh: ci/prow/e2e-gcp-operator-upgrade

In response to [this](https://github.com/openshift-kni/performance-addon-operators/pull/948#issuecomment-1249347987): >/override ci/prow/e2e-gcp-operator-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.