SchSeba / sriov-cni-1 / 24315117358

63%

master: 46%

push

github

Always restore VF admin MAC address on pod teardown

Previously, the VF administrative MAC address was only restored during
CNI DEL when conf.MAC was set, meaning only when the MAC was explicitly
configured via Pod annotation. This left the VF MAC unreset when users
configure it through other means (e.g., out-of-band tooling) rather than
through the Pod annotation — a common pattern with StatefulSets or
Deployments where annotation-based MAC configuration would cause
duplicate MAC addresses across replicas.

Change the condition from checking conf.MAC to checking
conf.OrigVfState.AdminMAC, so the original MAC is always restored if it
was captured in the cached VF state, regardless of how it was originally
configured.

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.72 hits per line