Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

Commit

Permalink
ref(e2e): move k8s version test config to CI
Browse files Browse the repository at this point in the history
This change removes the tests defined in e2e_k8s_version_test.go and
instead configures CI to run an existing test with different kind
clusters running specific versions of Kubernetes.

The versions tested have been changed from v1.21.11 to v1.21.10 and from
v1.22.8 to v1.22.7 because kind has never published node images for the
versions previously defined.

Fixes #4659

Signed-off-by: Jon Huhn <johuhn@microsoft.com>
  • Loading branch information
nojnhuh committed May 27, 2022
1 parent f73b9af commit 5ec3e75
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 124 deletions.
21 changes: 19 additions & 2 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -202,7 +202,16 @@ jobs:
needs: build
strategy:
matrix:
k8s_version: [""]
focus: [""]
bucket: [1, 2, 3, 4, 5, 6, 7, 8, 9, 10]
include:
- k8s_version: v1.21.10
focus: "Test traffic flowing from client to server with a Kubernetes Service for the Source: HTTP"
bucket: ".*"
- k8s_version: v1.22.7
focus: "Test traffic flowing from client to server with a Kubernetes Service for the Source: HTTP"
bucket: ".*"
env:
CTR_TAG: ${{ github.sha }}
CTR_REGISTRY: "localhost:5000" # unused for kind, but currently required in framework
Expand Down Expand Up @@ -233,13 +242,21 @@ jobs:
id: test
env:
K8S_NAMESPACE: "osm-system"
run: go test ./tests/e2e -test.v -ginkgo.v -ginkgo.progress -installType=KindCluster -test.timeout 0 -test.failfast -ginkgo.failFast -ginkgo.focus='\[Bucket ${{ matrix.bucket }}\]'
run: go test ./tests/e2e -test.v -ginkgo.v -ginkgo.progress -installType=KindCluster -kindClusterVersion='${{ matrix.k8s_version }}' -test.timeout 0 -test.failfast -ginkgo.failFast -ginkgo.focus='\[Bucket ${{ matrix.bucket }}\].*${{ matrix.focus }}'
continue-on-error: true
- name: Set Logs name
if: ${{ steps.test.conclusion != 'skipped' }}
run: |
if [[ -n "${{ matrix.k8s_version }}" ]]; then
echo "ARTIFACT_NAME=test_logs_k8s_version_${{ matrix.k8s_version }}" >> $GITHUB_ENV
else
echo "ARTIFACT_NAME=test_logs_bucket_${{ matrix.bucket }}" >> $GITHUB_ENV
fi
- name: Upload test logs
if: ${{ steps.test.conclusion != 'skipped' }}
uses: actions/upload-artifact@v2
with:
name: test_logs_bucket_${{ matrix.bucket }}
name: ${{ env.ARTIFACT_NAME }}
path: /tmp/test**/*
- name: Check continue tests
if: ${{ steps.test.conclusion != 'skipped' && steps.test.outcome == 'failure'}}
Expand Down
122 changes: 0 additions & 122 deletions tests/e2e/e2e_k8s_version_test.go

This file was deleted.

0 comments on commit 5ec3e75

Please sign in to comment.