kubeovn / kube-ovn / 23684855463

26%

push

github

fix: wrap EIP error shadowing and use uint64 for iptables counters (#6538)

* fix(controller): wrap original error instead of shadowing in EIP nat lookup

Three error handlers used `err := errors.New(...)` which shadowed the
outer `err` from getOvnEipNat/getIptablesEipNat, discarding the actual
root cause (e.g. OVSDB query failure, network timeout). Replace with
`err = fmt.Errorf("...: %w", err)` to preserve the error chain.

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

* fix(daemon): use uint64 for iptables counters to prevent overflow

iptables packet/byte counters are kernel uint64 values but were stored
as int and parsed with strconv.Atoi. When counters exceed MaxInt64 the
parse fails silently. Change GwIPTablesCounters fields to uint64 and
use strconv.ParseUint for correct handling of large counter values.

Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>
Co-Authored-By: Claude Opus 4.6 (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.6 (1M context) <noreply@anthropic.com>

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

13489 of 55696 relevant lines covered (24.22%)

0.28 hits per line