SchSeba / sriov-network-device-plugin / 22921557073

75%

master: 75%

push

github

Fix VFIO device path missing from env variables

The vfioInfoProvider was not including the VFIO device path
(e.g., /dev/vfio/162) in the environment variables because
GetEnvVal() was called before GetDeviceSpecs() during allocation.

The VFIO device file path was computed in GetDeviceSpecs() and stored
in vfioDevContainer, but since GetEnvVal() runs first in the Allocate()
flow, the field was always empty when building env vars.

Fix: Compute the VFIO device file on the fly in both GetDeviceSpecs()
and GetEnvVal() so the values are always current regardless of call
order. This also handles the case where device files may change during
the lifetime of the device plugin (e.g., IOMMU group reassignment).

Before: {"vfio":{"mount":"/dev/vfio/vfio"}}
After:  {"vfio":{"mount":"/dev/vfio/vfio","dev-mount":"/dev/vfio/162"}}

Signed-off-by: Author <author@example.com>

6 of 6 new or added lines in 1 file covered. (100.0%)

2131 of 2857 relevant lines covered (74.59%)

4.81 hits per line