Skip to content

Commit

Permalink
gha: Enable Ingress Controller tests in conformance-e2e
Browse files Browse the repository at this point in the history
The Ingress related connectivity tests are already available in cilium
cli, this commit is to enable the same in conformance e2e tests for any
case having kpr=true.

Relates: cilium/cilium-cli#1533
Relates: cilium/cilium-cli#2106
Signed-off-by: Tam Mach <tam.mach@cilium.io>
  • Loading branch information
sayboras committed Nov 14, 2023
1 parent 71908e7 commit b04e17e
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 1 deletion.
11 changes: 10 additions & 1 deletion .github/actions/cilium-config/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,9 @@ inputs:
mutual-auth:
description: 'Enable mTLS-based Mutual Authentication'
default: true
ingress-controller:
description: 'Enable ingress controller, required kubeProxyReplacement'
default: false
devices:
description: 'List of native devices to attach datapath programs'
default: ''
Expand Down Expand Up @@ -142,5 +145,11 @@ runs:
HOST_FW="--helm-set=hostFirewall.enabled=true"
fi
CONFIG="${DEFAULTS} ${IMAGE} ${TUNNEL} ${DEVICES} ${LB_MODE} ${ENDPOINT_ROUTES} ${IPV6} ${MASQ} ${EGRESS_GATEWAY} ${ENCRYPT} ${HOST_FW} ${LB_ACCELERATION}"
if [ "${{ inputs.kpr }}" == "true" ]; then
if [ "${{ inputs.ingress-controller }}" == "true" ]; then
INGRESS_CONTROLLER="--helm-set=ingressController.enabled=true"
fi
fi
CONFIG="${DEFAULTS} ${IMAGE} ${TUNNEL} ${DEVICES} ${LB_MODE} ${ENDPOINT_ROUTES} ${IPV6} ${MASQ} ${EGRESS_GATEWAY} ${ENCRYPT} ${HOST_FW} ${LB_ACCELERATION} ${INGRESS_CONTROLLER}"
echo "config=${CONFIG}" >> $GITHUB_OUTPUT
9 changes: 9 additions & 0 deletions .github/workflows/conformance-e2e.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -111,6 +111,7 @@ jobs:
lb-mode: 'snat'
endpoint-routes: 'true'
egress-gateway: 'true'
ingress-controller: 'true'

- name: '5'
# renovate: datasource=docker depName=quay.io/lvh-images/kind
Expand All @@ -124,6 +125,7 @@ jobs:
endpoint-routes: 'true'
egress-gateway: 'true'
host-fw: 'true'
ingress-controller: 'true'

- name: '6'
# renovate: datasource=docker depName=quay.io/lvh-images/kind
Expand All @@ -135,6 +137,7 @@ jobs:
egress-gateway: 'true'
host-fw: 'true'
lb-acceleration: 'testing-only'
ingress-controller: 'true'

- name: '7'
# renovate: datasource=docker depName=quay.io/lvh-images/kind
Expand All @@ -147,6 +150,7 @@ jobs:
lb-mode: 'snat'
egress-gateway: 'true'
lb-acceleration: 'testing-only'
ingress-controller: 'true'

- name: '8'
# renovate: datasource=docker depName=quay.io/lvh-images/kind
Expand All @@ -169,6 +173,7 @@ jobs:
lb-mode: 'snat'
endpoint-routes: 'true'
egress-gateway: 'true'
ingress-controller: 'true'

- name: '10'
# renovate: datasource=docker depName=quay.io/lvh-images/kind
Expand All @@ -183,6 +188,7 @@ jobs:
lb-mode: 'dsr'
endpoint-routes: 'true'
egress-gateway: 'true'
ingress-controller: 'true'

- name: '11'
# renovate: datasource=docker depName=quay.io/lvh-images/kind
Expand All @@ -196,6 +202,7 @@ jobs:
encryption-node: 'true'
lb-mode: 'snat'
egress-gateway: 'true'
ingress-controller: 'true'

- name: '12'
# renovate: datasource=docker depName=quay.io/lvh-images/kind
Expand All @@ -209,6 +216,7 @@ jobs:
encryption-node: 'true'
lb-mode: 'snat'
egress-gateway: 'true'
ingress-controller: 'true'

- name: '13'
# renovate: datasource=docker depName=quay.io/lvh-images/kind
Expand Down Expand Up @@ -257,6 +265,7 @@ jobs:
encryption-node: ${{ matrix.encryption-node }}
egress-gateway: ${{ matrix.egress-gateway }}
host-fw: ${{ matrix.host-fw }}
ingress-controller: ${{ matrix.ingress-controller }}
misc: ${{ matrix.misc }}

# Warning: since this is a privileged workflow, subsequent workflow job
Expand Down

0 comments on commit b04e17e

Please sign in to comment.