-
Notifications
You must be signed in to change notification settings - Fork 3.7k
exception CIDR can be bigger than the main CIDR and still pass validation #44614
Copy link
Copy link
Closed
Closed
Copy link
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.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.needs/triageThis issue requires triaging to establish severity and next steps.This issue requires triaging to establish severity and next steps.sig/policyImpacts whether traffic is allowed or denied based on user-defined policies.Impacts whether traffic is allowed or denied based on user-defined policies.
Description
Is there an existing issue for this?
- I have searched the existing issues
Version
equal or higher than v1.19.1 and lower than v1.20.0
What happened?
I assume validation should reject it.
How can we reproduce the issue?
apiVersion: "cilium.io/v2"
kind: CiliumNetworkPolicy
metadata:
name: broken-except
spec:
endpointSelector:
matchLabels:
app: test
egress:
- toCIDRSet:
- cidr: "10.0.0.0/24"
except:
- "10.0.0.0/16"
I was checking different policies and somehow this policy passes the validation.
Cilium Version
I was testing it on 1.19
Kernel Version
6.16.12-1rodete2-amd64
Kubernetes Version
v1.32.2
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
Reactions are currently unavailable
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.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.needs/triageThis issue requires triaging to establish severity and next steps.This issue requires triaging to establish severity and next steps.sig/policyImpacts whether traffic is allowed or denied based on user-defined policies.Impacts whether traffic is allowed or denied based on user-defined policies.