Skip to content

feat: Add device plugin resource name for node management port#18

Merged
hareeshpc merged 1 commit intoMellanox:mainfrom
ykulazhenkov:pr-device-plugin-resource
Feb 11, 2026
Merged

feat: Add device plugin resource name for node management port#18
hareeshpc merged 1 commit intoMellanox:mainfrom
ykulazhenkov:pr-device-plugin-resource

Conversation

@ykulazhenkov
Copy link
Contributor

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.

@ykulazhenkov ykulazhenkov force-pushed the pr-device-plugin-resource branch 2 times, most recently from 85dad54 to ca9a420 Compare February 11, 2026 07:59
Copy link
Member

@vasrem vasrem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

…t 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>
@ykulazhenkov ykulazhenkov force-pushed the pr-device-plugin-resource branch from ca9a420 to 0f21e99 Compare February 11, 2026 09:55
Copy link
Member

@tobiasgiese tobiasgiese left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hareeshpc hareeshpc self-assigned this Feb 11, 2026
@aserdean
Copy link
Collaborator

LGTM

@hareeshpc hareeshpc merged commit 5ab230f into Mellanox:main Feb 11, 2026
4 checks passed
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.

5 participants