Skip to content

Commit

Permalink
ci-gke: Add test for hair-pinning Ingress case
Browse files Browse the repository at this point in the history
This is to make sure that we have a coverage for hair pinning traffic
i.e. backend is on the same node with client. Additionally, this test
will provide some coverage with cloud provider (e.g. GKE), as all other
Ingress/Gateway API/L7 LB tests are running with kind clusters.

Signed-off-by: Tam Mach <tam.mach@cilium.io>
  • Loading branch information
sayboras committed May 10, 2024
1 parent 32b2784 commit 4ce4bb1
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 1 deletion.
4 changes: 4 additions & 0 deletions .github/actions/gke/test-config-helm.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,3 +12,7 @@ config:
- type: "tunnel-ipsec"
index: 4
cilium-install-opts: "--helm-set=encryption.enabled=true --helm-set=encryption.type=ipsec --datapath-mode=tunnel"
- type: "tunnel-ingress-controller"
index: 5
cilium-install-opts: "--helm-set=kubeProxyReplacement=true --helm-set=ingressController.enabled=true"
nodes: 1
1 change: 1 addition & 0 deletions .github/actions/gke/test-config-schema.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,3 +4,4 @@ testConfigItem:
type: str()
index: int()
cilium-install-opts: str()
nodes: int(required=False)
2 changes: 1 addition & 1 deletion .github/workflows/conformance-gke.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -224,7 +224,7 @@ jobs:
--cluster-ipv4-cidr="/21" \
--services-ipv4-cidr="/24" \
--image-type COS_CONTAINERD \
--num-nodes 2 \
--num-nodes ${{ matrix.config.nodes || 2 }} \
--machine-type e2-custom-2-4096 \
--disk-type pd-standard \
--disk-size 20GB \
Expand Down

0 comments on commit 4ce4bb1

Please sign in to comment.