-
Notifications
You must be signed in to change notification settings - Fork 119
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade SR-IOV provider nodes image to k8s-1.22 #694
Merged
Merged
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,101 @@ | ||
# K8S 1.22.2 with SR-IOV in a Kind cluster | ||
|
||
Provides a pre-deployed containerized k8s cluster with version 1.22.2 that runs | ||
using [KinD](https://github.com/kubernetes-sigs/kind) | ||
The cluster is completely ephemeral and is recreated on every cluster restart. The KubeVirt containers are built on the | ||
local machine and are then pushed to a registry which is exposed at | ||
`localhost:5000`. | ||
|
||
This version also expects to have SR-IOV enabled nics (SR-IOV Physical Function) on the current host, and will move | ||
physical interfaces into the `KinD`'s cluster worker node(s) so that they can be used through multus and SR-IOV | ||
components. | ||
|
||
This providers also deploys [multus](https://github.com/k8snetworkplumbingwg/multus-cni) | ||
, [sriov-cni](https://github.com/k8snetworkplumbingwg/sriov-cni) | ||
and [sriov-device-plugin](https://github.com/k8snetworkplumbingwg/sriov-network-device-plugin). | ||
|
||
## Bringing the cluster up | ||
|
||
```bash | ||
export KUBEVIRT_PROVIDER=kind-1.22-sriov | ||
export KUBEVIRT_NUM_NODES=3 | ||
make cluster-up | ||
|
||
$ cluster-up/kubectl.sh get nodes | ||
NAME STATUS ROLES AGE VERSION | ||
sriov-control-plane Ready control-plane,master 20h v1.22.2 | ||
sriov-worker Ready worker 20h v1.22.2 | ||
sriov-worker2 Ready worker 20h v1.22.2 | ||
|
||
$ cluster-up/kubectl.sh get pods -n kube-system -l app=multus | ||
NAME READY STATUS RESTARTS AGE | ||
kube-multus-ds-amd64-d45n4 1/1 Running 0 20h | ||
kube-multus-ds-amd64-g26xh 1/1 Running 0 20h | ||
kube-multus-ds-amd64-mfh7c 1/1 Running 0 20h | ||
|
||
$ cluster-up/kubectl.sh get pods -n sriov -l app=sriov-cni | ||
NAME READY STATUS RESTARTS AGE | ||
kube-sriov-cni-ds-amd64-fv5cr 1/1 Running 0 20h | ||
kube-sriov-cni-ds-amd64-q95q9 1/1 Running 0 20h | ||
|
||
$ cluster-up/kubectl.sh get pods -n sriov -l app=sriovdp | ||
NAME READY STATUS RESTARTS AGE | ||
kube-sriov-device-plugin-amd64-h7h84 1/1 Running 0 20h | ||
kube-sriov-device-plugin-amd64-xrr5z 1/1 Running 0 20h | ||
``` | ||
|
||
## Bringing the cluster down | ||
|
||
```bash | ||
export KUBEVIRT_PROVIDER=kind-1.22-sriov | ||
make cluster-down | ||
``` | ||
|
||
This destroys the whole cluster, and moves the SR-IOV nics to the root network namespace. | ||
|
||
## Setting a custom kind version | ||
|
||
In order to use a custom kind image / kind version, export `KIND_NODE_IMAGE`, `KIND_VERSION`, `KUBECTL_PATH` before | ||
running cluster-up. For example in order to use kind 0.9.0 (which is based on k8s-1.19.1) use: | ||
|
||
```bash | ||
export KIND_NODE_IMAGE="kindest/node:v1.19.1@sha256:98cf5288864662e37115e362b23e4369c8c4a408f99cbc06e58ac30ddc721600" | ||
export KIND_VERSION="0.9.0" | ||
export KUBECTL_PATH="/usr/bin/kubectl" | ||
``` | ||
|
||
This allows users to test or use custom images / different kind versions before making them official. | ||
See https://github.com/kubernetes-sigs/kind/releases for details about node images according to the kind version. | ||
|
||
## Running multi SR-IOV clusters locally | ||
|
||
Kubevirtci SR-IOV provider supports running two clusters side by side with few known limitations. | ||
|
||
General considerations: | ||
|
||
- A SR-IOV PF must be available for each cluster. In order to achieve that, there are two options: | ||
|
||
1. Assign just one PF for each worker node of each cluster by using `export PF_COUNT_PER_NODE=1` (this is the default | ||
value). | ||
2. Optional method: `export PF_BLACKLIST=<PF names>` the non used PFs, in order to prevent them from being allocated to | ||
the current cluster. The user can list the PFs that should not be allocated to the current cluster, keeping in mind | ||
that at least one (or 2 in case of migration), should not be listed, so they would be allocated for the current | ||
cluster. Note: another reason to blacklist a PF, is in case its has a defect or should be kept for other operations ( | ||
for example sniffing). | ||
|
||
- Clusters should be created one by another and not in parallel (to avoid races over SR-IOV PF's). | ||
- The cluster names must be different. This can be achieved by setting `export CLUSTER_NAME=sriov2` on the 2nd cluster. | ||
The default `CLUSTER_NAME` is `sriov`. The 2nd cluster registry would be exposed at `localhost:5001` automatically, | ||
once the `CLUSTER_NAME` | ||
is set to a non default value. | ||
- Each cluster should be created on its own git clone folder, i.e: | ||
`/root/project/kubevirtci1` | ||
`/root/project/kubevirtci2` | ||
In order to switch between them, change dir to that folder and set the env variables `KUBECONFIG` | ||
and `KUBEVIRT_PROVIDER`. | ||
- In case only one PF exists, for example if running on prow which will assign only one PF per job in its own DinD, | ||
Kubevirtci is agnostic and nothing needs to be done, since all conditions above are met. | ||
- Upper limit of the number of clusters that can be run on the same time equals number of PFs / number of PFs per | ||
cluster, therefore, in case there is only one PF, only one cluster can be created. Locally the actual limit currently | ||
supported is two clusters. | ||
- In order to use `make cluster-down` please make sure the right `CLUSTER_NAME` is exported. |
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are some 1.19 eferences.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't remove that one because its describe how to use different versions of Kind and node images
Since we use the latest I think it make sense to leave it like this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fair enough