SchSeba / sriov-cni / 25500636424

61%

master: 61%

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%)

981 of 1602 relevant lines covered (61.24%)

8.5 hits per line