Skip to content

Add IPv6 and dual-stack support for SubnetIPReservation#1435

Draft
wenqiq wants to merge 3 commits into
vmware-tanzu:mainfrom
wenqiq:topic/wenqi/SubnetIPReservationIPv6
Draft

Add IPv6 and dual-stack support for SubnetIPReservation#1435
wenqiq wants to merge 3 commits into
vmware-tanzu:mainfrom
wenqiq:topic/wenqi/SubnetIPReservationIPv6

Conversation

@wenqiq
Copy link
Copy Markdown
Contributor

@wenqiq wenqiq commented May 15, 2026

No description provided.

wenqiq added 2 commits May 17, 2026 01:27
Support VpcConnectivityProfile.Ipv6Blocks in IPBlocksInfo sync

Include paths from VpcConnectivityProfile.Ipv6Blocks into the same
externalIPBlockPaths set as ExternalIpBlocks during IPBlocksInfo
reconciliation. This ensures that external IPv6 IP blocks assigned
to a VPC connectivity profile have their CIDRs and IP ranges
surfaced in the IPBlocksInfo CR (ExternalIPCIDRs / ExternalIPRanges)
without requiring any CRD schema change.

Signed-off-by: Wenqi Qiu <wenqi.qiu@broadcom.com>
Signed-off-by: Wenqi Qiu <wenqi.qiu@broadcom.com>
@wenqiq wenqiq force-pushed the topic/wenqi/SubnetIPReservationIPv6 branch 2 times, most recently from 13bc3ab to 4ef05d7 Compare May 19, 2026 02:32
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 19, 2026

Codecov Report

❌ Patch coverage is 83.33333% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.96%. Comparing base (ff79596) to head (45b1d35).

Files with missing lines Patch % Lines
pkg/nsx/services/subnetipreservation/builder.go 64.28% 0 Missing and 5 partials ⚠️
...netipreservation/subnetipreservation_controller.go 81.25% 0 Missing and 3 partials ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1435      +/-   ##
==========================================
- Coverage   77.02%   76.96%   -0.07%     
==========================================
  Files         156      156              
  Lines       22048    22084      +36     
==========================================
+ Hits        16983    16996      +13     
+ Misses       3859     3858       -1     
- Partials     1206     1230      +24     
Flag Coverage Δ
unit-tests 76.96% <83.33%> (-0.07%) ⬇️
Files with missing lines Coverage Δ
pkg/nsx/services/ipblocksinfo/ipblocksinfo.go 85.46% <100.00%> (+0.53%) ⬆️
pkg/nsx/services/nsxserviceaccount/cluster.go 80.61% <100.00%> (ø)
...netipreservation/subnetipreservation_controller.go 71.91% <81.25%> (-4.19%) ⬇️
pkg/nsx/services/subnetipreservation/builder.go 75.60% <64.28%> (-24.40%) ⬇️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: Wenqi Qiu <wenqi.qiu@broadcom.com>
@wenqiq wenqiq force-pushed the topic/wenqi/SubnetIPReservationIPv6 branch from 45b1d35 to 8cce586 Compare May 19, 2026 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants