Skip to content

Commit

Permalink
ci/multi-pool: Use nonMasqueradeCIDRs for all pod CIDRs
Browse files Browse the repository at this point in the history
With the previous commit, we removed the restriction that the user must
set a native routing CIDR. This commit adjusts the multi-pool workflow
to make use of that change (and incidentally also tests that the
previous commit via CI).

Signed-off-by: Sebastian Wicki <sebastian@isovalent.com>
  • Loading branch information
gandro authored and christarazi committed Aug 29, 2023
1 parent 98964b1 commit a9a2c3e
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/conformance-multi-pool.yaml
Expand Up @@ -91,9 +91,8 @@ jobs:
--helm-set=kubeProxyReplacement=strict \
--helm-set=bpf.masquerade=true \
--helm-set=bpf.hostLegacyRouting=true\
--helm-set=ipv4NativeRoutingCIDR=10.0.0.0/8 \
--helm-set=ipMasqAgent.enabled=true \
--helm-set=ipMasqAgent.config.nonMasqueradeCIDRs='{192.168.0.0/16}' \
--helm-set=ipMasqAgent.config.nonMasqueradeCIDRs='{10.0.0.0/8,192.168.0.0/16}' \
--helm-set=ipam.mode=multi-pool \
--helm-set=ipam.operator.autoCreateCiliumPodIPPools.default.ipv4.cidrs='{10.10.0.0/16}' \
--helm-set=ipam.operator.autoCreateCiliumPodIPPools.default.ipv4.maskSize=24 \
Expand Down

0 comments on commit a9a2c3e

Please sign in to comment.