SchSeba / sriov-cni-1 / 26096743799

63%

master: 46%

push

github

fix: gate effective MAC restore on cached state instead of user config in ReleaseVF

The ReleaseVF function previously used `conf.MAC != ""` to guard the
effective MAC address restore during pod teardown. This condition only
triggers when the user explicitly sets a MAC via CNI config/annotation,
but the effective MAC may have been changed by the driver or other
processes regardless of user configuration.

Change the guard to `conf.OrigVfState.EffectiveMAC != ""` so the
effective MAC is always restored when it was captured during SetupVF,
matching the pattern already used by other fields like MTU
(`conf.OrigVfState.MTU != 0`) and the recent fix to ResetVFConfig
for admin MAC (`conf.OrigVfState.AdminMAC != ""`).

Signed-off-by: Sebastian Sch <sebassch@gmail.com>

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

970 of 1544 relevant lines covered (62.82%)

8.73 hits per line