kubeovn / kube-ovn / 31349036419

36%

Build Type

push

github

Committed by web-flow

Commit Message

ipam: restore counter views when preserving the other family's static address (#7187)

In a dual-stack subnet, `GetStaticAddress` replacing one family's static address
preserves the other. The preserve path calls `releaseAddr` and restores only the
lookup maps, not the counter sets, so the preserved address stays in
`Available`/`Released` and out of `Using`. Once the pool exhausts and recycles
from `Released`, a later random allocation can hand the still-configured address
to another pod, causing a duplicate address and drifting `usingIPs`/`availableIPs`.

Restore the counter views with the maps: re-add the address to `Using` and
remove it from `Available`/`Released` at subnet and pool level, mirroring
`releaseV4Addr`/`releaseV6Addr`. The set operations are idempotent, so the
out-of-CIDR/reserved and multi-owner paths are unaffected.

Signed-off-by: kiarash <kiarash.azarnia@gmail.com>

Coverage Stats

16 of 18 new or added lines in 1 file covered. (88.89%)

2 existing lines in 1 file now uncovered.

18907 of 61944 relevant lines covered (30.52%)

0.35 hits per line