Skip to content

Commit

Permalink
Replace ToServices/ToPorts combination in CiliumNetworkPolicy bec…
Browse files Browse the repository at this point in the history
…ause of breakage in Cilium v1.14 (#128)

See cilium/cilium#20067 - the combination is not supported anymore and Cilium prints warnings about the non-effective policy.
  • Loading branch information
AndiDog authored Jan 22, 2024
1 parent a413718 commit 34cef0d
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 6 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [Unreleased]

### Fixed

- Replace `ToServices`/`ToPorts` combination in CiliumNetworkPolicy because of breakage in Cilium v1.14

## [0.19.1] - 2024-01-18

### Changed
Expand Down
10 changes: 4 additions & 6 deletions helm/cilium/templates/extra-policies/configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,12 +18,10 @@ data:
endpointSelector: {} # any
egress:
- toServices:
- k8sServiceSelector:
namespace: kube-system
selector:
matchLabels:
k8s-app: coredns
- toEndpoints:
- matchLabels:
k8s-app: coredns
k8s:io.kubernetes.pod.namespace: kube-system
toPorts:
- ports:
Expand Down

0 comments on commit 34cef0d

Please sign in to comment.