Fix ServiceLB dual-stack ingress IP listing

[brandond]

Proposed Changes

Fix ServiceLB dual-stack ingress IP listing

Types of Changes

bugfix

Verification

See linked issue

Testing

added a unit test for this function

Linked Issues

• ServiceLB advertises node IPs that do not match the service IPFamily #6978

User-Facing Change

Resolved an issue with ServiceLB that would cause it to advertise node IPv6 addresses, even if the cluster or service was not enabled for dual-stack operation.

Further Comments

[codecov-commenter]

Codecov Report

Base: 9.51% // Head: 9.65% // Increases project coverage by +0.13% 🎉

Coverage data is based on head (ba0702d) compared to base (2156015).
Patch coverage: 100.00% of modified lines in pull request are covered.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files

@@            Coverage Diff            @@
##           master   #6979      +/-   ##
=========================================
+ Coverage    9.51%   9.65%   +0.13%     
=========================================
  Files         146     146              
  Lines       10757   10738      -19     
=========================================
+ Hits         1024    1037      +13     
+ Misses       9515    9483      -32     
  Partials      218     218              

Flag	Coverage Δ
unittests	9.65% <100.00%> (+0.13%)	⬆️

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

Impacted Files	Coverage Δ
pkg/cloudprovider/servicelb.go	3.56% <100.00%> (+3.56%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.