Skip to content

Commit a1f5962

Browse files
committed
ci: fix cilium chaining e2e (#2391)
1 parent 8a584a7 commit a1f5962

File tree

5 files changed

+29
-14
lines changed

5 files changed

+29
-14
lines changed

.github/workflows/build-x86-image.yaml

+5-3
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ concurrency:
2222

2323
env:
2424
GOSEC_VERSION: '2.15.0'
25-
HELM_VERSION: v3.10.3
25+
HELM_VERSION: v3.11.1
2626

2727
jobs:
2828
build-kube-ovn:
@@ -894,11 +894,13 @@ jobs:
894894
sudo cp -r /root/.kube/ ~/.kube/
895895
sudo chown -R $(id -un). ~/.kube/
896896
897-
- name: Install Kube-OVN
898-
run: make kind-install
897+
- name: Install Kube-OVN with Cilium chaining
898+
run: make kind-install-cilium-chaining
899899

900900
- name: Run E2E
901901
working-directory: ${{ env.E2E_DIR }}
902+
env:
903+
E2E_CILIUM_CHAINING: "true"
902904
run: make k8s-conformance-e2e
903905

904906
- name: Cleanup

Makefile

+7-11
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ CONTROL_PLANE_TAINTS = node-role.kubernetes.io/master node-role.kubernetes.io/co
1313
MULTUS_IMAGE = ghcr.io/k8snetworkplumbingwg/multus-cni:stable
1414
MULTUS_YAML = https://raw.githubusercontent.com/k8snetworkplumbingwg/multus-cni/master/deployments/multus-daemonset.yml
1515

16-
CILIUM_VERSION = 1.11.6
16+
CILIUM_VERSION = 1.12.7
1717
CILIUM_IMAGE_REPO = quay.io/cilium/cilium
1818

1919
VPC_NAT_GW_IMG = $(REGISTRY)/vpc-nat-gateway:$(VERSION)
@@ -259,10 +259,6 @@ kind-init-ipv6:
259259
kind-init-dual:
260260
@ip_family=dual $(MAKE) kind-init
261261

262-
.PHONY: kind-init-cilium
263-
kind-init-cilium:
264-
@kube_proxy_mode=iptables $(MAKE) kind-init
265-
266262
.PHONY: kind-load-image
267263
kind-load-image:
268264
$(call kind_load_image,kube-ovn,$(REGISTRY)/kube-ovn:$(VERSION))
@@ -439,28 +435,28 @@ kind-install-multus:
439435
kubectl apply -f "$(MULTUS_YAML)"
440436
kubectl -n kube-system rollout status ds kube-multus-ds
441437

442-
.PHONY: kind-install-cilium
443-
kind-install-cilium: kind-load-image kind-untaint-control-plane
438+
.PHONY: kind-install-cilium-chaining
439+
kind-install-cilium-chaining: kind-load-image kind-untaint-control-plane
444440
$(eval KUBERNETES_SERVICE_HOST = $(shell kubectl get nodes kube-ovn-control-plane -o jsonpath='{.status.addresses[0].address}'))
445441
$(call docker_ensure_image_exists,$(CILIUM_IMAGE_REPO):v$(CILIUM_VERSION))
446442
$(call kind_load_image,kube-ovn,$(CILIUM_IMAGE_REPO):v$(CILIUM_VERSION))
447-
kubectl apply -f yamls/chaining.yaml
443+
kubectl apply -f yamls/cilium-chaining.yaml
448444
helm repo add cilium https://helm.cilium.io/
449445
helm install cilium cilium/cilium \
450446
--version $(CILIUM_VERSION) \
451447
--namespace=kube-system \
452448
--set k8sServiceHost=$(KUBERNETES_SERVICE_HOST) \
453449
--set k8sServicePort=6443 \
454450
--set tunnel=disabled \
451+
--set sessionAffinity=true \
455452
--set enableIPv4Masquerade=false \
456-
--set enableIdentityMark=false \
457453
--set cni.chainingMode=generic-veth \
458454
--set cni.customConf=true \
459455
--set cni.configMap=cni-configuration
460456
kubectl -n kube-system rollout status ds cilium --timeout 300s
461-
bash dist/images/cilium.sh
457+
bash dist/images/install-cilium-cli.sh
462458
sed 's/VERSION=.*/VERSION=$(VERSION)/' dist/images/install.sh | \
463-
ENABLE_LB=false ENABLE_NP=false WITHOUT_KUBE_PROXY=true CNI_CONFIG_PRIORITY=10 bash
459+
ENABLE_LB=false ENABLE_NP=false CNI_CONFIG_PRIORITY=10 bash
464460
kubectl describe no
465461

466462
.PHONY: kind-reload

dist/images/.dockerignore

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
build.sh
22
install.sh
3+
install-cilium-cli.sh
34
ovn-ic-db-docker.sh
45
generate-ssl-docker.sh
56
cleanup.sh

dist/images/install-cilium-cli.sh

+16
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
#!/usr/bin/env bash
2+
3+
set -e
4+
5+
CILIUM_CLI_VERSION=$(curl -s https://raw.githubusercontent.com/cilium/cilium-cli/master/stable.txt)
6+
CILIUM_CLI_ARCH=amd64
7+
if [ "$(uname -m)" = "aarch64" ]; then
8+
CILIUM_CLI_ARCH=arm64
9+
fi
10+
11+
curl -L --fail --remote-name-all https://github.com/cilium/cilium-cli/releases/download/${CILIUM_CLI_VERSION}/cilium-linux-${CILIUM_CLI_ARCH}.tar.gz{,.sha256sum}
12+
if command -v sha256sum >/dev/null; then
13+
sha256sum --check cilium-linux-${CILIUM_CLI_ARCH}.tar.gz.sha256sum
14+
fi
15+
sudo tar xzvfC cilium-linux-${CILIUM_CLI_ARCH}.tar.gz /usr/local/bin
16+
rm cilium-linux-${CILIUM_CLI_ARCH}.tar.gz{,.sha256sum}
File renamed without changes.

0 commit comments

Comments
 (0)