-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Bump ENDPOINTS_MAP_SIZE dynamically based on endpoint count #3075
Copy link
Copy link
Closed
Labels
area/daemonImpacts operation of the Cilium daemon.Impacts operation of the Cilium daemon.area/datapathImpacts bpf/ or low-level forwarding details, including map management and monitor messages.Impacts bpf/ or low-level forwarding details, including map management and monitor messages.help-wantedYou can help! Post a detailed plan on the issue or create a PR to solve this issue.You can help! Post a detailed plan on the issue or create a PR to solve this issue.kind/featureThis introduces new functionality.This introduces new functionality.staleThe stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale.The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale.
Description
Metadata
Metadata
Assignees
Labels
area/daemonImpacts operation of the Cilium daemon.Impacts operation of the Cilium daemon.area/datapathImpacts bpf/ or low-level forwarding details, including map management and monitor messages.Impacts bpf/ or low-level forwarding details, including map management and monitor messages.help-wantedYou can help! Post a detailed plan on the issue or create a PR to solve this issue.You can help! Post a detailed plan on the issue or create a PR to solve this issue.kind/featureThis introduces new functionality.This introduces new functionality.staleThe stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale.The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale.
This map size is currently hardcoded, but it should be straightforward to expand this on demand.
cilium/pkg/maps/lxcmap/lxcmap.go
Lines 34 to 35 in f8e147b
cilium/daemon/daemon.go
Line 822 in 5792bad
cilium/bpf/lib/maps.h
Line 33 in 6094981
cilium/bpf/lib/maps.h
Line 320 in 6094981