Skip to content

Commit

Permalink
gh/workflows: enable loadBalancer acceleration for some suites
Browse files Browse the repository at this point in the history
Set loadBalancer.acceleration=testing-only in two datapath
conformance tests. I had originally planned to modify 12-15 but
those have encryption enabled. Instead, enable it on 6-7. Compared
to Jenkins we drop  testing for (EP routes=true,
acceleration=testing-only) from the matrix.

Updates #24151

Signed-off-by: Lorenz Bauer <lmb@isovalent.com>
  • Loading branch information
lmb committed Apr 5, 2023
1 parent 31293bd commit 98ddd32
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion .github/workflows/conformance-datapath.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -169,6 +169,7 @@ jobs:
lb-mode: 'snat'
egress-gateway: 'true'
host-fw: 'true'
lb-acceleration: 'testing-only'

- name: '7'
kernel: 'bpf-next-main'
Expand All @@ -177,6 +178,7 @@ jobs:
tunnel: 'disabled'
lb-mode: 'snat'
egress-gateway: 'true'
lb-acceleration: 'testing-only'

- name: '8'
kernel: 'bpf-next-main'
Expand Down Expand Up @@ -317,6 +319,10 @@ jobs:
if [ "${{ matrix.egress-gateway }}" == "true" ]; then
EGRESS_GATEWAY="--helm-set=egressGateway.enabled=true"
fi
LB_ACCELERATION=""
if [ "${{ matrix.lb-acceleration }}" != "" ]; then
LB_ACCELERATION="--helm-set=loadBalancer.acceleration=${{ matrix.lb-acceleration }}"
fi
ENCRYPT=""
if [ "${{ matrix.encryption }}" != "" ]; then
Expand All @@ -331,7 +337,7 @@ jobs:
HOST_FW="--helm-set=hostFirewall.enabled=true"
fi
CONFIG="${CILIUM_INSTALL_DEFAULTS} ${TUNNEL} ${LB_MODE} ${ENDPOINT_ROUTES} ${IPV6} ${EGRESS_GATEWAY} ${ENCRYPT} ${HOST_FW}"
CONFIG="${CILIUM_INSTALL_DEFAULTS} ${TUNNEL} ${LB_MODE} ${ENDPOINT_ROUTES} ${IPV6} ${EGRESS_GATEWAY} ${ENCRYPT} ${HOST_FW} ${LB_ACCELERATION}"
echo "cilium_install_defaults=${CONFIG}" >> $GITHUB_OUTPUT
- name: Set commit status to pending
Expand Down

0 comments on commit 98ddd32

Please sign in to comment.