Skip to content

Commit ca9a420

Browse files
committed
feat: Add configurable device plugin resource name for node management port
Add nodeMgmtPortDpResourceName to the Helm chart to allow specifying a device plugin resource for the ovnkube-node management port container. When set, the resource is included in container requests/limits and exposed via the OVNKUBE_NODE_MGMT_PORT_DP_RESOURCE_NAME environment variable. Signed-off-by: Yury Kulazhenkov <ykulazhenkov@nvidia.com>
1 parent 8e6c9b9 commit ca9a420

File tree

3 files changed

+18
-0
lines changed

3 files changed

+18
-0
lines changed

helm/ovn-kubernetes-dpf/templates/host-with-dpu-manifests.yaml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,7 @@
11
{{- if .Values.nodeWithDPUManifests.enabled }}
2+
{{- if and .Values.nodeWithDPUManifests.nodeMgmtPortNetdev .Values.nodeWithDPUManifests.nodeMgmtPortDpResourceName }}
3+
{{- fail "nodeWithDPUManifests.nodeMgmtPortNetdev and nodeWithDPUManifests.nodeMgmtPortDpResourceName are mutually exclusive, only one can be set" }}
4+
{{- end }}
25
---
36
apiVersion: v1
47
kind: ServiceAccount
@@ -159,6 +162,13 @@ spec:
159162
requests:
160163
cpu: 100m
161164
memory: 300Mi
165+
{{ if .Values.nodeWithDPUManifests.nodeMgmtPortDpResourceName }}
166+
{{ .Values.nodeWithDPUManifests.nodeMgmtPortDpResourceName }}: "1"
167+
{{ end }}
168+
{{ if .Values.nodeWithDPUManifests.nodeMgmtPortDpResourceName }}
169+
limits:
170+
{{ .Values.nodeWithDPUManifests.nodeMgmtPortDpResourceName }}: "1"
171+
{{ end }}
162172
env:
163173
- name: OVN_DAEMONSET_VERSION
164174
value: "1.2.0"
@@ -262,6 +272,8 @@ spec:
262272
value: "dpu-host"
263273
- name: OVNKUBE_NODE_MGMT_PORT_NETDEV
264274
value: {{ default "" .Values.nodeWithDPUManifests.nodeMgmtPortNetdev | quote }}
275+
- name: OVNKUBE_NODE_MGMT_PORT_DP_RESOURCE_NAME
276+
value: {{ default "" .Values.nodeWithDPUManifests.nodeMgmtPortDpResourceName | quote }}
265277
- name: OVN_HOST_NETWORK_NAMESPACE
266278
valueFrom:
267279
configMapKeyRef:

helm/ovn-kubernetes-dpf/values.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,10 @@ nodeWithDPUManifests:
2828
repository: ${OVNKUBERNETES_IMAGE}
2929
tag: ${TAG}
3030
pullPolicy: IfNotPresent
31+
# -- Node management port netdev name. Mutually exclusive with nodeMgmtPortDpResourceName.
3132
nodeMgmtPortNetdev: ""
33+
# -- Node management port device plugin resource name. Mutually exclusive with nodeMgmtPortNetdev.
34+
nodeMgmtPortDpResourceName: ""
3235
dpuServiceAccountName: ovn-dpu
3336
dpuServiceAccountNamespace: ovn-kubernetes
3437
cniBinDir: "/opt/cni/bin"

helm/ovn-kubernetes-dpf/values.yaml.tmpl

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,10 @@ nodeWithDPUManifests:
1919
repository: ${OVNKUBERNETES_IMAGE}
2020
tag: ${TAG}
2121
pullPolicy: IfNotPresent
22+
# -- Node management port netdev name. Mutually exclusive with nodeMgmtPortDpResourceName.
2223
nodeMgmtPortNetdev: ""
24+
# -- Node management port device plugin resource name. Mutually exclusive with nodeMgmtPortNetdev.
25+
nodeMgmtPortDpResourceName: ""
2326
dpuServiceAccountName: ovn-dpu
2427
dpuServiceAccountNamespace: ovn-kubernetes
2528
cniBinDir: "/opt/cni/bin"

0 commit comments

Comments
 (0)