kubeovn / kube-ovn / 27017056817

26%

Build Type

push

github

Committed by web-flow

Commit Message

perf(daemon): list subnets and pods once per gateway setter (#6827)

* perf(daemon): list subnets and pods once per gateway setter

runGateway runs every 3s, and each of setIPSet/setPolicyRouting/setIptables
re-listed the whole subnet cache up to 15 times per dual-stack tick (and pods
2-4 times). List subnets/pods once per setter and thread the slice through the
helpers, dropping the now-redundant error returns whose only source was List.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>

* perf(daemon): log getCidrByProtocol errors instead of dropping them

Address review feedback: getSubnetsNeedNAT, getSubnetsDistributedGateway and
getDefaultVpcSubnetsCIDR silently skipped subnets when getCidrByProtocol
failed, and StartTProxyTCPPortProbe swallowed getTProxyConditionPod errors.
Log these errors to match the existing handling in reconcileNatOutGoingPolicyIPset
and generateNatOutgoingPolicyChainRules; the error path only triggers on
malformed CIDRs, not on normal single-stack subnets.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>

---------

Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Coverage Stats

0 of 80 new or added lines in 3 files covered. (0.0%)

433 existing lines in 5 files now uncovered.

14867 of 57891 relevant lines covered (25.68%)

0.3 hits per line