-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Closed
Labels
feature/lb-ipamgood-first-issueGood starting point for new developers, which requires minimal understanding of Cilium.Good starting point for new developers, which requires minimal understanding of Cilium.kind/enhancementThis would improve or streamline existing functionality.This would improve or streamline existing functionality.
Description
LB-IPAM has a feature which allows services to share an IP under a number of conditions. One of these is that ports are not allowed to overlap. So far, it didn't matter for which protocol these ports were since the Cilium datapath was unable to differentiate between TCP and UDP. Now that #33434 has merged, we should adjust this check so a service with port UDP/80 and and a service with port TCP/80 can share an IP.
The logic in question lives here:
cilium/operator/pkg/lbipam/service_store.go
Line 107 in 6b211cc
for _, port1 := range sv.Ports { |
Metadata
Metadata
Assignees
Labels
feature/lb-ipamgood-first-issueGood starting point for new developers, which requires minimal understanding of Cilium.Good starting point for new developers, which requires minimal understanding of Cilium.kind/enhancementThis would improve or streamline existing functionality.This would improve or streamline existing functionality.
Type
Projects
Status
Done