Skip to content

Commit

Permalink
gha: Enable Ingress Controller tests in conformance-e2e
Browse files Browse the repository at this point in the history
[ upstream commit c8ee339 ]
[ upstream commit 9311af0 ]

The Ingress related connectivity tests are already available in cilium
cli, this commit is to enable the same in conformance e2e tests for some
tests having kpr as true.

Relates: cilium/cilium-cli#1533
Relates: cilium/cilium-cli#2106
Signed-off-by: Tam Mach <tam.mach@cilium.io>
Signed-off-by: Sebastian Wicki <sebastian@isovalent.com>
  • Loading branch information
sayboras authored and gandro committed Apr 8, 2024
1 parent 1c7b252 commit f4b4dbe
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 1 deletion.
12 changes: 11 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 @@ -141,5 +144,12 @@ 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"
INGRESS_CONTROLLER+=" --helm-set=ingressController.service.type=NodePort"
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
5 changes: 5 additions & 0 deletions .github/workflows/conformance-e2e.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -144,6 +144,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 Down Expand Up @@ -193,6 +194,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 @@ -206,6 +208,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 @@ -234,6 +237,7 @@ jobs:
lb-mode: 'snat'
egress-gateway: 'true'
misc: 'encryption.wireguard.encapsulate=true'
ingress-controller: 'true'

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

- name: Install Cilium CLI
uses: cilium/cilium-cli@a9579d1afc528b085930c5237a69ee77b2c2be76 # v0.16.4
Expand Down

0 comments on commit f4b4dbe

Please sign in to comment.