-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Closed
Labels
area/agentCilium agent related.Cilium agent related.area/servicemeshGH issues or PRs regarding servicemeshGH issues or PRs regarding servicemeshfeature/k8s-ingresskind/bugThis is a bug in the Cilium logic.This is a bug in the Cilium logic.
Description
Is there an existing issue for this?
- I have searched the existing issues
What happened?
When deploying an ingress entry for Cilium and changing the ingressClass to something else first Cilium will keep all created resources around.
To reproduce:
- Deploy ingress + an example app: https://docs.cilium.io/en/v1.12/gettingstarted/servicemesh/ingress/#gs-ingress
- check if the LB service got created
kubectl get service
kubectl edit ingress
(or apply a change) and change ingressClass to something else (like nginx)- check
kubectl get service
and still see the LB service
Cilium Version
latest master and v1.12.5
Kernel Version
n/a
Kubernetes Version
v1.26.1
Sysdump
No response
Relevant log output
No response
Anything else?
The update ingress checks only the new ingress field ingressClass := getIngressClassName(event.newIngress)
It should also be handled correct on operator startup to check any changes while it was down.
Code of Conduct
- I agree to follow this project's Code of Conduct
Metadata
Metadata
Assignees
Labels
area/agentCilium agent related.Cilium agent related.area/servicemeshGH issues or PRs regarding servicemeshGH issues or PRs regarding servicemeshfeature/k8s-ingresskind/bugThis is a bug in the Cilium logic.This is a bug in the Cilium logic.