Skip to content

Commit

Permalink
clustermesh: Enable legacy host routing for Ingress
Browse files Browse the repository at this point in the history
After #22006, BPF host routing is enabled by default, this commit is to
enable legacy host routing as a workaround, as the response packet might
be dropped. Further investigation is tracked under #31653.

Signed-off-by: Tam Mach <tam.mach@cilium.io>
  • Loading branch information
sayboras committed May 10, 2024
1 parent 4ce4bb1 commit de9c87b
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions .github/workflows/conformance-clustermesh.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -275,9 +275,12 @@ jobs:
CILIUM_INSTALL_INGRESS=""
if [ "${{ matrix.kube-proxy }}" == "none" ]; then
# The ingress controller requires KPR to be enabled.
CILIUM_INSTALL_INGRESS="--helm-set=ingressController.enabled=true \
--helm-set-string=kubeProxyReplacement=true"
CILIUM_INSTALL_INGRESS="--helm-set=ingressController.enabled=true"
# Once https://github.com/cilium/cilium/issues/31653 is fixed, we can remove tunnel check
# Use the legacy host routing in case of tunnel disabled
if [ "${{ matrix.tunnel }}" == "disabled" ]; then
CILIUM_INSTALL_INGRESS+=" --helm-set=bpf.hostLegacyRouting=true"
fi
fi
CONNECTIVITY_TEST_DEFAULTS="--hubble=false \
Expand Down

0 comments on commit de9c87b

Please sign in to comment.