Skip to content
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

feat: support dpdk interface hotplug for kubevirt #3438

Merged
merged 1 commit into from
Nov 24, 2023

Conversation

watermelon-brother
Copy link
Contributor

What type of this PR

feat: support dpdk interface hotplug for kubevirt

Which issue(s) this PR fixes

none

WHAT

If you want kubevirt support dpdk interface hotplug, nad should define as below:

apiVersion: k8s.cni.cncf.io/v1
kind: NetworkAttachmentDefinition
metadata:
  name: ovn-dpdk
  namespace: default
spec:
  config: >-
    {
        "cniVersion": "0.3.0", 
        "type": "kube-ovn", 
        "server_socket": "/run/openvswitch/kube-ovn-daemon.sock", 
        "provider": "xx.xx.ovn",
        "vhost_user_socket_consumption": "kubevirt" # must set
    }

HOW

I will push a pr about kubevirt dpdk feature in recent times.

@watermelon-brother watermelon-brother force-pushed the hotplug-dpdk-interface branch 3 times, most recently from e8b6170 to a876bc4 Compare November 20, 2023 03:46
pkg/daemon/handler_linux.go Dismissed Show dismissed Hide dismissed
pkg/daemon/handler_linux.go Dismissed Show dismissed Hide dismissed
pkg/daemon/handler_linux.go Dismissed Show dismissed Hide dismissed
@oilbeater
Copy link
Collaborator

@watermelon-brother please rebase and fix the conflict

@watermelon-brother watermelon-brother force-pushed the hotplug-dpdk-interface branch 3 times, most recently from 8d74c18 to 8621eed Compare November 21, 2023 05:48
@zhangzujian zhangzujian force-pushed the hotplug-dpdk-interface branch from 8621eed to 8a31aed Compare November 23, 2023 00:41
@oilbeater oilbeater merged commit be68461 into kubeovn:master Nov 24, 2023
59 checks passed
bobz965 pushed a commit that referenced this pull request Nov 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants