Conformance Ingress (ci-ingress) #4589
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| name: Conformance Ingress (ci-ingress) | |
| # Any change in triggers needs to be reflected in the concurrency group. | |
| on: | |
| workflow_dispatch: | |
| inputs: | |
| PR-number: | |
| description: "Pull request number." | |
| required: true | |
| context-ref: | |
| description: "Context in which the workflow runs. If PR is from a fork, will be the PR target branch (general case). If PR is NOT from a fork, will be the PR branch itself (this allows committers to test changes to workflows directly from PRs)." | |
| required: true | |
| SHA: | |
| description: "SHA under test (head of the PR branch)." | |
| required: true | |
| extra-args: | |
| description: "[JSON object] Arbitrary arguments passed from the trigger comment via regex capture group. Parse with 'fromJson(inputs.extra-args).argName' in workflow." | |
| required: false | |
| default: '{}' | |
| push: | |
| branches: | |
| - v1.15 | |
| - ft/v1.15/** | |
| paths-ignore: | |
| - 'Documentation/**' | |
| - 'test/**' | |
| # By specifying the access of one of the scopes, all of those that are not | |
| # specified are set to 'none'. | |
| permissions: | |
| # To be able to access the repository with actions/checkout | |
| contents: read | |
| # To allow retrieving information from the PR API | |
| pull-requests: read | |
| # To be able to set commit status | |
| statuses: write | |
| concurrency: | |
| # Structure: | |
| # - Workflow name | |
| # - Event type | |
| # - A unique identifier depending on event type: | |
| # - schedule: SHA | |
| # - workflow_dispatch: PR number | |
| # | |
| # This structure ensures a unique concurrency group name is generated for each | |
| # type of testing, such that re-runs will cancel the previous run. | |
| group: | | |
| ${{ github.workflow }} | |
| ${{ github.event_name }} | |
| ${{ | |
| (github.event_name == 'push' && github.sha) || | |
| (github.event_name == 'workflow_dispatch' && github.event.inputs.PR-number) | |
| }} | |
| cancel-in-progress: true | |
| env: | |
| cilium_cli_ci_version: | |
| CILIUM_CLI_MODE: helm | |
| kind_config: .github/kind-config.yaml | |
| metallb_version: 0.12.1 | |
| timeout: 5m | |
| jobs: | |
| commit-status-start: | |
| if: ${{ github.event_name != 'push' }} | |
| name: Commit Status Start | |
| runs-on: ubuntu-latest | |
| steps: | |
| - name: Set initial commit status | |
| uses: myrotvorets/set-commit-status-action@3730c0a348a2ace3c110851bed53331bc6406e9f # v2.0.1 | |
| with: | |
| sha: ${{ inputs.SHA || github.sha }} | |
| ingress-conformance-test: | |
| name: Ingress Conformance Test | |
| runs-on: ubuntu-latest | |
| timeout-minutes: 120 | |
| strategy: | |
| fail-fast: false | |
| matrix: | |
| include: | |
| - name: Without XDP | |
| kube-proxy-replacement: true | |
| enable-node-port: false | |
| bpf-lb-acceleration: disabled | |
| loadbalancer-mode: dedicated | |
| default-ingress-controller: false | |
| - name: With XDP | |
| kube-proxy-replacement: true | |
| enable-node-port: false | |
| bpf-lb-acceleration: native | |
| loadbalancer-mode: dedicated | |
| default-ingress-controller: false | |
| - name: With Shared LB | |
| kube-proxy-replacement: true | |
| enable-node-port: false | |
| bpf-lb-acceleration: disabled | |
| loadbalancer-mode: shared | |
| default-ingress-controller: false | |
| - name: With Default Ingress Controller | |
| kube-proxy-replacement: true | |
| enable-node-port: false | |
| bpf-lb-acceleration: disabled | |
| loadbalancer-mode: dedicated | |
| default-ingress-controller: true | |
| - name: Without KPR | |
| kube-proxy-replacement: false | |
| enable-node-port: true | |
| bpf-lb-acceleration: disabled | |
| loadbalancer-mode: dedicated | |
| default-ingress-controller: false | |
| steps: | |
| - name: Checkout context ref (trusted) | |
| uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1 | |
| with: | |
| ref: ${{ inputs.context-ref || github.sha }} | |
| persist-credentials: false | |
| - name: Set Environment Variables | |
| uses: ./.github/actions/set-env-variables | |
| - name: Install Cilium CLI | |
| uses: cilium/cilium-cli@a9579d1afc528b085930c5237a69ee77b2c2be76 # v0.16.4 | |
| with: | |
| repository: ${{ env.CILIUM_CLI_RELEASE_REPO }} | |
| release-version: ${{ env.CILIUM_CLI_VERSION }} | |
| ci-version: ${{ env.cilium_cli_ci_version }} | |
| - name: Get Cilium's default values | |
| id: default_vars | |
| uses: ./.github/actions/helm-default | |
| with: | |
| image-tag: ${{ inputs.SHA }} | |
| chart-dir: ./untrusted/install/kubernetes/cilium | |
| - name: Set image tag | |
| id: vars | |
| run: | | |
| echo sha=${{ steps.default_vars.outputs.sha }} >> $GITHUB_OUTPUT | |
| CILIUM_INSTALL_DEFAULTS="${{ steps.default_vars.outputs.cilium_install_defaults }} \ | |
| --helm-set=debug.verbose=envoy \ | |
| --helm-set kubeProxyReplacement=${{ matrix.kube-proxy-replacement }} \ | |
| --helm-set nodePort.enabled=${{ matrix.enable-node-port }} \ | |
| --helm-set=ingressController.enabled=true \ | |
| --helm-set=ingressController.loadbalancerMode=${{ matrix.loadbalancer-mode }} \ | |
| --helm-set=ingressController.default=${{ matrix.default-ingress-controller }} \ | |
| --helm-set=extraConfig.bpf-lb-acceleration=${{ matrix.bpf-lb-acceleration }} \ | |
| --helm-set=l2announcements.enabled=true \ | |
| --helm-set=devices='{eth0}'" | |
| echo cilium_install_defaults=${CILIUM_INSTALL_DEFAULTS} >> $GITHUB_OUTPUT | |
| # Warning: since this is a privileged workflow, subsequent workflow job | |
| # steps must take care not to execute untrusted code. | |
| - name: Checkout pull request branch (NOT TRUSTED) | |
| uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1 | |
| with: | |
| ref: ${{ steps.vars.outputs.sha }} | |
| persist-credentials: false | |
| path: untrusted | |
| sparse-checkout: | | |
| install/kubernetes/cilium | |
| examples | |
| - name: Create kind cluster | |
| uses: helm/kind-action@99576bfa6ddf9a8e612d83b513da5a75875caced # v1.9.0 | |
| with: | |
| version: ${{ env.KIND_VERSION }} | |
| node_image: ${{ env.KIND_K8S_IMAGE }} | |
| kubectl_version: ${{ env.KIND_K8S_VERSION }} | |
| config: ${{ env.kind_config }} | |
| wait: 0 # The control-plane never becomes ready, since no CNI is present | |
| - name: Checkout ingress-controller-conformance | |
| uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1 | |
| with: | |
| # Use the forked repo with retry mechanism | |
| # Please refer to https://github.com/kubernetes-sigs/ingress-controller-conformance/pull/101 for more details. | |
| repository: cilium/ingress-controller-conformance | |
| path: ingress-controller-conformance | |
| ref: 010bbae21b71d9785660b87908dfe2ba8cd2f25d | |
| persist-credentials: false | |
| - name: Install Ingress conformance test tool | |
| timeout-minutes: 10 | |
| run: | | |
| cd ingress-controller-conformance | |
| make build | |
| - name: Wait for images to be available | |
| timeout-minutes: 30 | |
| shell: bash | |
| run: | | |
| for image in cilium-ci operator-generic-ci ; do | |
| until docker manifest inspect quay.io/${{ env.QUAY_ORGANIZATION_DEV }}/$image:${{ steps.vars.outputs.sha }} &> /dev/null; do sleep 45s; done | |
| done | |
| - name: Install Cilium | |
| id: install-cilium | |
| run: | | |
| cilium install ${{ steps.vars.outputs.cilium_install_defaults }} | |
| - name: Wait for Cilium to be ready | |
| run: | | |
| cilium status --wait | |
| kubectl get pods -n kube-system | |
| - name: Install Cilium LB IPPool and L2 Announcement Policy | |
| timeout-minutes: 10 | |
| run: | | |
| KIND_NET_CIDR=$(docker network inspect kind -f '{{(index .IPAM.Config 0).Subnet}}') | |
| LB_CIDR=$(echo ${KIND_NET_CIDR} | sed "s@0.0/16@255.200/28@") | |
| echo "Deploying LB-IPAM Pool..." | |
| cat << EOF > pool.yaml | |
| apiVersion: "cilium.io/v2alpha1" | |
| kind: CiliumLoadBalancerIPPool | |
| metadata: | |
| name: "pool" | |
| spec: | |
| cidrs: | |
| - cidr: "$LB_CIDR" | |
| EOF | |
| kubectl apply -f pool.yaml | |
| echo "Deploying L2-Announcement Policy..." | |
| cat << 'EOF' > l2policy.yaml | |
| apiVersion: "cilium.io/v2alpha1" | |
| kind: CiliumL2AnnouncementPolicy | |
| metadata: | |
| name: l2policy | |
| spec: | |
| loadBalancerIPs: true | |
| interfaces: | |
| - eth0 | |
| nodeSelector: | |
| matchExpressions: | |
| - key: node-role.kubernetes.io/control-plane | |
| operator: DoesNotExist | |
| EOF | |
| kubectl apply -f l2policy.yaml | |
| - name: Create sample workload | |
| timeout-minutes: 5 | |
| run: | | |
| kubectl apply -n default -f https://raw.githubusercontent.com/istio/istio/release-1.11/samples/bookinfo/platform/kube/bookinfo.yaml | |
| if [ ${{ matrix.default-ingress-controller }} = "true" ]; then | |
| # remove ingressClassName line from basic-ingress.yaml | |
| sed -i '/ingressClassName/d' untrusted/examples/kubernetes/servicemesh/basic-ingress.yaml | |
| kubectl apply -n default -f untrusted/examples/kubernetes/servicemesh/basic-ingress.yaml | |
| kubectl wait -n default --for=condition=Ready --all pod --timeout=${{ env.timeout }} | |
| fi | |
| kubectl apply -n default -f untrusted/examples/kubernetes/servicemesh/basic-ingress.yaml | |
| kubectl wait -n default --for=condition=Ready --all pod --timeout=${{ env.timeout }} | |
| - name: Run Sanity check (external) | |
| timeout-minutes: 5 | |
| run: | | |
| lb=$(kubectl get ingress basic-ingress -o jsonpath='{.status.loadBalancer.ingress[0].ip}') | |
| curl -s -v --connect-timeout 5 --max-time 20 --retry 3 --retry-all-errors --retry-delay 5 --fail -- http://"$lb" | |
| # By now the service should be up, no need to do the manual retries for the second request | |
| curl -s -v --connect-timeout 5 --max-time 20 --retry 3 --fail -- http://"$lb"/details/1 | |
| - name: Run Sanity check (internal to NodePort) | |
| if: ${{ matrix.kube-proxy-replacement == 'true' }} | |
| timeout-minutes: 5 | |
| run: | | |
| if [ ${{ matrix.loadbalancer-mode }} = "dedicated" ]; then | |
| node_port=$(kubectl get svc cilium-ingress-basic-ingress -o jsonpath='{.spec.ports[?(@.port==80)].nodePort}') | |
| else | |
| node_port=$(kubectl get -n kube-system svc cilium-ingress -o jsonpath='{.spec.ports[?(@.port==80)].nodePort}') | |
| fi | |
| docker exec -i chart-testing-control-plane curl -s -v --connect-timeout 5 --max-time 20 --retry 3 --fail http://localhost:$node_port/details/1 | |
| - name: Cleanup Sanity check | |
| timeout-minutes: 5 | |
| run: | | |
| # Clean up after sanity check to avoid any conflicts with the conformance test | |
| kubectl delete -n default -f untrusted/examples/kubernetes/servicemesh/basic-ingress.yaml | |
| kubectl delete -n default -f https://raw.githubusercontent.com/istio/istio/release-1.11/samples/bookinfo/platform/kube/bookinfo.yaml | |
| kubectl wait ingress basic-ingress --for=delete | |
| - name: Run Ingress conformance test | |
| timeout-minutes: 30 | |
| run: | | |
| cd ingress-controller-conformance | |
| ./ingress-controller-conformance -ingress-class cilium -wait-time-for-ingress-status 60s -wait-time-for-ready 60s | |
| - name: Post-test information gathering | |
| if: ${{ !success() && steps.install-cilium.outcome != 'skipped' }} | |
| run: | | |
| kubectl get pods --all-namespaces -o wide | |
| cilium status | |
| cilium sysdump --output-filename cilium-sysdump-out | |
| shell: bash {0} # Disable default fail-fast behaviour so that all commands run independently | |
| - name: Upload artifacts | |
| if: ${{ !success() }} | |
| uses: actions/upload-artifact@1746f4ab65b179e0ea60a494b83293b640dd5bba # v4.3.2 | |
| with: | |
| name: cilium-sysdump-out-${{ matrix.name }} | |
| path: cilium-sysdump-out-*.zip | |
| retention-days: 5 | |
| commit-status-final: | |
| if: ${{ always() && github.event_name != 'push' }} | |
| name: Commit Status Final | |
| needs: ingress-conformance-test | |
| runs-on: ubuntu-latest | |
| steps: | |
| - name: Set final commit status | |
| uses: myrotvorets/set-commit-status-action@3730c0a348a2ace3c110851bed53331bc6406e9f # v2.0.1 | |
| with: | |
| sha: ${{ inputs.SHA || github.sha }} | |
| status: ${{ needs.ingress-conformance-test.result }} |