-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Closed
Labels
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.area/kprAnything related to our kube-proxy replacement.Anything related to our kube-proxy replacement.kind/bugThis is a bug in the Cilium logic.This is a bug in the Cilium logic.kind/community-reportThis was reported by a user in the Cilium community, eg via Slack.This was reported by a user in the Cilium community, eg via Slack.kind/tech-debtTechnical debtTechnical debt
Description
Is there an existing issue for this?
- I have searched the existing issues
Version
equal or higher than v1.17.4 and lower than v1.18.0
What happened?
The evaluation for the configmap keys enable-external-ips and
enable-host-port
depend on a deprecated value for Kube Proxy replacement "partial". This should probably be evaluated against KPR=false since partial is a deprecated value.
In 1.16 this tested against KPR being set to either partial or false.
How can we reproduce the issue?
- Install CIlium with Helm using v1.17.4
- Use --set to set the values
kubeProxyReplacement=false
along withhostPort.enabled=true
orexternalIPs.enabled=true
- View the cilium-config configmap to see neither entry for
enable-external-ips
orenable-host-port
present
Cilium Version
1.17.4
Kernel Version
6.14.8-orbstack-00288-g80b66077b748
Kubernetes Version
Client Version: v1.33.1
Kustomize Version: v5.6.0
Server Version: v1.30.0
Regression
No response
Sysdump
No response
Relevant log output
Anything else?
No response
Cilium Users Document
- Are you a user of Cilium? Please add yourself to the Users doc
Code of Conduct
- I agree to follow this project's Code of Conduct
Metadata
Metadata
Assignees
Labels
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.area/kprAnything related to our kube-proxy replacement.Anything related to our kube-proxy replacement.kind/bugThis is a bug in the Cilium logic.This is a bug in the Cilium logic.kind/community-reportThis was reported by a user in the Cilium community, eg via Slack.This was reported by a user in the Cilium community, eg via Slack.kind/tech-debtTechnical debtTechnical debt