kubevirtci icon indicating copy to clipboard operation
kubevirtci copied to clipboard

cluster-up, cnao: Do not deploy OVS-CNI

Open ormergi opened this issue 9 months ago • 2 comments

What this PR does / why we need it: There is no actual need to deploy OVS-CNI dy default at the moment, we dont test OVS CNI e2e in kubevirt/kubevirt.

Also the ephemeral cluster nodes does not have OVS back-end installed causing OVS-CNI pods to get into crush-loop all over CI.

In case OVS-CNI is required in certain environments, it can be deployed by patching the existing cluster-network-addons instance.

Set CNAO to not deploy OVS-CNI pods and reduce some memory consumption.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged): Fixes #

Special notes for your reviewer:

Checklist

This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR. Approvers are expected to review this list.

Release note:

cluster-up: CNAO does not deploy OVS-CNI by default

ormergi avatar May 02 '24 15:05 ormergi

/cc @EdDev

ormergi avatar May 12 '24 12:05 ormergi

Rebasesd

ormergi avatar May 12 '24 12:05 ormergi

/cc @phoracek

ormergi avatar May 20 '24 09:05 ormergi

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: brianmcarey

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:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

kubevirt-bot avatar May 21 '24 13:05 kubevirt-bot

/test check-provision-k8s-1.29

brianmcarey avatar May 21 '24 13:05 brianmcarey

/retest-required This bot automatically retries required jobs that failed/flaked on approved PRs. Silence the bot with an /lgtm cancel or /hold comment for consistent failures.

kubevirt-commenter-bot avatar May 21 '24 15:05 kubevirt-commenter-bot

@ormergi: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
check-up-kind-1.27-vgpu 9316d4128eb4358f3135eb8f7e64906efe5d2fab link false /test check-up-kind-1.27-vgpu

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

kubevirt-bot avatar May 21 '24 16:05 kubevirt-bot

/retest-required This bot automatically retries required jobs that failed/flaked on approved PRs. Silence the bot with an /lgtm cancel or /hold comment for consistent failures.

kubevirt-commenter-bot avatar May 21 '24 19:05 kubevirt-commenter-bot

Added it to the bump script so this change wont be overridden upon bumping https://github.com/kubevirt/kubevirtci/pull/1215/files#diff-a62a4a90160c0961988d25e478dc85642e5404173a3151b42afd8777e90308d1R61 thx

oshoval avatar Jul 04 '24 11:07 oshoval