kubeovn / kube-ovn / 28151356021

27%

Build Type

push

github

Committed by web-flow

Commit Message

feat: Add kubeovn support for dynamic ip allocation/byo dhcp (#6885)

* add kubeovn support for dynamic ip allocation/byo dhcp

Signed-off-by: Tom Zhao <zhaoxu@microsoft.com>

* address PR review: fix webhook denial logic, reject excludeIps for MAC-only subnet, flush stale IPs in MAC-only mode

Signed-off-by: Tom Zhao <zhaoxu@microsoft.com>

* feat(mac-only): IPAM-backed MAC-only subnet allocation + protocol validation

Implement the IPAM-backed design for MAC-only underlay subnets (BYO-DHCP /
external DHCP) per maintainer review feedback. A MAC-only subnet is an underlay
subnet (Spec.Vlan != "") created without a CIDR (Spec.CIDRBlock == ""); kube-ovn
allocates only a MAC per pod NIC and the guest obtains its IP from an external
DHCP server.

- apis: add ProtocolMac = "Mac" protocol constant.
- ipam: add NewMacOnlySubnet + getMacOnlyAddress so GetRandomAddress allocates a
  MAC only (random or static, idempotent via NicToMac/MacToPod); register
  CIDR-less subnets in AddOrUpdateSubnet (guarding against stripping an existing
  CIDR with ErrInvalidCIDR); fix releaseAddr to clean up MacToPod/NicToMac for
  mac-only NICs so MACs are not leaked.
- controller: classify mac-only subnets as ProtocolMac in formatAddress and
  register them in IPAM (no calcSubnetStatusIP since there are no IP ranges);
  acquireAddress allocates a per-interface/provider MAC for mac-only pods.
- util: accept ProtocolMac in ValidateSubnet so mac-only subnets pass logical
  switch validation and reach Ready (fixes "Mac is not a valid protocol type"
  ValidateLogicalSwitchFailed surfaced by e2e on a live KIND cluster).
- ipam tests: TestMacOnlySubnet, TestMacOnlySubnetCannotStripCIDR.

Signed-off-by: Tom Zhao <zhaoxu@microsoft.com>

* test(e2e): add mac-only subnet allocation e2e (controller-level + multus secondary interface)

Adds two conformance specs under the underlay suite:
- Controller-level: an underlay subnet without a CIDR is classified as
  Protocol=Mac, reaches Ready, an... (continued)

Coverage Stats

200 of 355 new or added lines in 13 files covered. (56.34%)

4 existing lines in 2 files now uncovered.

16265 of 60202 relevant lines covered (27.02%)

0.31 hits per line