Skip to content

Commit

Permalink
conformance: Added bpf-hostlegacyrouting option
Browse files Browse the repository at this point in the history
Added a configurable bpf-hostlegacyrouting option to the conformance
tests as Cilium 1.16.5+ introduced a breaking change for Talos
'forwardKubeDNSToHost'. See cilium/cilium#35098
for more details.

Signed-off-by: Philip Schmid <[email protected]>
  • Loading branch information
PhilipSchmid committed Jan 7, 2025
1 parent 419eff3 commit c26c947
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 0 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/conformance-pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ jobs:
kube-proxy-replacement: "true"
socketlb: false
bpf-masquerade: true
bpf-hostlegacyrouting: true
ipam-mode: 'kubernetes'
ipv4: true
ipv6: false
Expand Down Expand Up @@ -88,6 +89,7 @@ jobs:
--set ipv4.enabled=${{ matrix.config.ipv4 }} \
--set ipv6.enabled=${{ matrix.config.ipv6 }} \
--set bpf.masquerade=${{ matrix.config.bpf-masquerade }} \
--set bpf.hostLegacyRouting=${{ matrix.config.bpf-hostlegacyrouting }} \
--set kubeProxyReplacement=${{ matrix.config.kube-proxy-replacement }} \
--set socketLB.enabled=${{ matrix.config.socketlb }} \
--set ipam.mode=${{ matrix.config.ipam-mode }} \
Expand Down
7 changes: 7 additions & 0 deletions .github/workflows/conformance.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ jobs:
kube-proxy-replacement: "true"
socketlb: false
bpf-masquerade: true
bpf-hostlegacyrouting: true
ipam-mode: 'kubernetes'
ipv4: true
ipv6: false
Expand All @@ -49,6 +50,7 @@ jobs:
kube-proxy-replacement: "true"
socketlb: false
bpf-masquerade: true
bpf-hostlegacyrouting: true
ipam-mode: 'kubernetes'
ipv4: true
ipv6: false
Expand All @@ -63,6 +65,7 @@ jobs:
kube-proxy-replacement: "false"
socketlb: true
bpf-masquerade: false
bpf-hostlegacyrouting: true
ipam-mode: 'kubernetes'
ipv4: true
ipv6: false
Expand All @@ -77,6 +80,7 @@ jobs:
kube-proxy-replacement: "false"
socketlb: true
bpf-masquerade: true
bpf-hostlegacyrouting: true
ipam-mode: 'kubernetes'
ipv4: true
ipv6: false
Expand All @@ -90,6 +94,7 @@ jobs:
kube-proxy-replacement: "true"
socketlb: false
bpf-masquerade: true
bpf-hostlegacyrouting: true
ipam-mode: 'cluster-pool'
ipv4: true
ipv6: false
Expand All @@ -104,6 +109,7 @@ jobs:
kube-proxy-replacement: "true"
socketlb: false
bpf-masquerade: true
bpf-hostlegacyrouting: true
ipam-mode: 'kubernetes'
ipv4: true
ipv6: false
Expand Down Expand Up @@ -163,6 +169,7 @@ jobs:
--set ipv4.enabled=${{ matrix.config.ipv4 }} \
--set ipv6.enabled=${{ matrix.config.ipv6 }} \
--set bpf.masquerade=${{ matrix.config.bpf-masquerade }} \
--set bpf.hostLegacyRouting=${{ matrix.config.bpf-hostlegacyrouting }} \
--set kubeProxyReplacement=${{ matrix.config.kube-proxy-replacement }} \
--set socketLB.enabled=${{ matrix.config.socketlb }} \
--set ipam.mode=${{ matrix.config.ipam-mode }} \
Expand Down

0 comments on commit c26c947

Please sign in to comment.