kubeovn / kube-ovn / 28633980013

27%

push

github

feat: support per-pod IP family selection (#6950)

* feat: support per-pod IP family selection

Allow pods and attachment networks to request IPv4-only or IPv6-only allocation on dual-stack subnets via ip_family annotations. Keep existing dual-stack behavior when the annotation is omitted and add unit coverage for controller, IPAM, validation, and CNI mask handling.

Signed-off-by: jimyag <git@jimyag.com>

* fix: address IP family review feedback

Preserve subnet DHCP options when pod IP family cannot be determined and add coverage for the empty pod IP case. Also log gateway family mismatches and clarify CNI single-family mask behavior in tests.

Signed-off-by: jimyag <git@jimyag.com>

* refactor: clarify IP family helper intent

Rename CNI gateway and DHCP helper functions to describe their IP-family behavior. Add comments explaining why CNI and DHCP paths follow the actual allocated pod address families.

Signed-off-by: jimyag <git@jimyag.com>

* test: cover IP family review gaps

Validate per-interface static IP annotations against provider-scoped ip_family and add controller coverage for named IPPool allocation with requested IPv6 family.

Signed-off-by: jimyag <git@jimyag.com>

* refactor: derive IP family annotation values from protocol constants

Avoid hard-coded lowercase ip family strings by deriving accepted annotation values from kube-ovn protocol constants while preserving the lowercase-only validation contract.

Signed-off-by: jimyag <git@jimyag.com>

* refactor: avoid implying fixed dual-stack lengths

Use zero-value slices where single-family and dual-stack paths can append one or two entries, making the allocation and CNI mask code reflect the actual variable cardinality.

Signed-off-by: jimyag <git@jimyag.com>

* fix: handle IP family review edge cases

Parse static IPs before family validation so malformed addresses return parse errors. Also make subnet family errors provider-neutral and reject malformed CNI CIDRs without panics.

Si... (continued)

161 of 207 new or added lines in 8 files covered. (77.78%)

16558 of 60451 relevant lines covered (27.39%)

0.32 hits per line