Fix init node transport interface bug

[Jexf]

1.Fix adding node routes crash bug, which caused by uninitialized DualStackIPs param.

2.Fix ipv4 address and ipv6 address confused bug and return the first ipv4 and ipv6 ips for transport interface.

Fixes #2665

Signed-off-by: Wu zhengdong [email protected]

[codecov-commenter]

Codecov Report

Merging #2670 (320f85c) into main (04619b6) will increase coverage by 4.99%.
The diff coverage is 25.00%.

@@            Coverage Diff             @@
##             main    #2670      +/-   ##
==========================================
+ Coverage   60.66%   65.66%   +4.99%     
==========================================
  Files         285      285              
  Lines       22997    26351    +3354     
==========================================
+ Hits        13951    17303    +3352     
+ Misses       7545     7433     -112     
- Partials     1501     1615     +114     

Flag	Coverage Δ
e2e-tests	57.09% <25.00%> (?)
kind-e2e-tests	48.41% <25.00%> (+0.05%)	⬆️
unit-tests	41.05% <25.00%> (-0.04%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
pkg/agent/util/net.go	47.05% <0.00%> (+10.69%)	⬆️
...gent/controller/noderoute/node_route_controller.go	62.21% <100.00%> (+6.87%)	⬆️
pkg/controller/egress/ipallocator/allocator.go	65.00% <0.00%> (-15.42%)	⬇️
pkg/controller/networkpolicy/endpoint_querier.go	77.64% <0.00%> (-13.79%)	⬇️
pkg/legacyapis/core/v1alpha2/register.go	69.23% <0.00%> (-10.77%)	⬇️
pkg/apis/stats/register.go	71.42% <0.00%> (-10.39%)	⬇️
pkg/legacyapis/stats/register.go	71.42% <0.00%> (-10.39%)	⬇️
pkg/ovs/openflow/ofctrl_meter.go	33.84% <0.00%> (-10.16%)	⬇️
pkg/legacyapis/security/v1alpha1/register.go	73.33% <0.00%> (-10.00%)	⬇️
.../registry/networkpolicy/clustergroupmember/rest.go	78.26% <0.00%> (-9.98%)	⬇️
... and 271 more

[tnqn]

@Jexf thanks for catching the issues before it's released!

Apparently the corresponding change #2436 is lack of unit tests to cover the two functions.

I will first merge this to fix the bug. @Jexf It would be great if you could have another PR to help fix the unit tests for the two functions so the error won't happen again. Please let me know if you feel ok to do it, or I would request @lzhecheng to follow up.

[tnqn]

/test-all

[Jexf]

@Jexf thanks for catching the issues before it's released!

Apparently the corresponding change #2436 is lack of unit tests to cover the two functions.

I will first merge this to fix the bug. @Jexf It would be great if you could have another PR to help fix the unit tests for the two functions so the error won't happen again. Please let me know if you feel ok to do it, or I would request @lzhecheng to follow up.

Thank you for reminding me, I will add unit tests for the two functions in another PR.

[lzhecheng]

/test-windows-e2e /test-windows-networkpolicy