Skip to content

[CP 332] fix: use CNI_PATH env for host-device plugin lookup on OpenShift#93

Merged
sundar-pds merged 1 commit into
ROCm:mainfrom
ci-penbot-01:CP.O2O.pensando.network-operator.332.rocm.network-operator.main
Jun 11, 2026
Merged

[CP 332] fix: use CNI_PATH env for host-device plugin lookup on OpenShift#93
sundar-pds merged 1 commit into
ROCm:mainfrom
ci-penbot-01:CP.O2O.pensando.network-operator.332.rocm.network-operator.main

Conversation

@ci-penbot-01

Copy link
Copy Markdown
Contributor

cp of pensando/network-operator#332

The amd-host-device CNI plugin hardcoded /opt/cni/bin as the path to
find the delegate host-device binary. On OpenShift CoreOS, /opt/cni is
a broken symlink (points to /usr/lib/opt/cni which doesn't exist), so
the plugin fails with "failed to find plugin host-device in path
[/opt/cni/bin]".

Read CNI_PATH from the environment (set by Multus/kubelet per the CNI
spec) with a fallback that includes /var/lib/cni/bin (where OpenShift
Multus actually installs CNI binaries) and /usr/libexec/cni (OS-bundled
binaries). Update release notes for v1.2.1.

Co-authored-by: Yuva Shankar <11082310+yuva29@users.noreply.github.com>
Co-authored-by: Claude Opus 4 (1M context) <noreply@anthropic.com>
(cherry picked from commit ff31ef5bfd82d3825686d92a47f68146a39b95cf)

@sundar-pds sundar-pds left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@sundar-pds sundar-pds merged commit 6e550db into ROCm:main Jun 11, 2026
2 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.

3 participants