SchSeba / sriov-network-device-plugin

75%

master: 75%

LAST BUILD ON BRANCH add_vfio_device_to_info_provider

branch: SELECT

CHANGE BRANCH

x

Sync Branches

• No branch selected
• add_helper_function
• add_vfio_device_to_info_provider
• bump_go_libs
• bump_packages
• bump_to_golang_1.18_k8s_1.24
• enable_func_test
• fix_device_spec_rdma
• fix_device_spec_rdma_2
• fix_grpc_deprecation
• fix_unittest
• master
• multiple_bumps
• token_support
• update_devices_on_shutdown

Build # 20917891540

Build Type

push

github

Committed by SchSeba

Commit Message

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: Move the VFIO device file computation to the constructor
(NewVfioInfoProvider) so the values are available immediately for
both GetEnvVal() and GetDeviceSpecs() regardless of call order.

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

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

Run Details

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

2134 of 2860 relevant lines covered (74.62%)

4.8 hits per line