From d4766bee6f56bbcde9af538230562b8cf5a6751b Mon Sep 17 00:00:00 2001 From: Stavros Kontopoulos Date: Fri, 17 Jan 2025 18:58:04 +0200 Subject: [PATCH] Disable probe tests for kourier tls runtime (#15702) * disable probe tests with kourier tls runtime * remove exclude * use the testflag --- .github/workflows/kind-e2e.yaml | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/.github/workflows/kind-e2e.yaml b/.github/workflows/kind-e2e.yaml index dac3365b2898..ed213a72ab82 100644 --- a/.github/workflows/kind-e2e.yaml +++ b/.github/workflows/kind-e2e.yaml @@ -143,14 +143,6 @@ jobs: - test-suite: e2e test-path: ./test/e2e - exclude: - # Disabled due to flakiness: https://github.com/knative/serving/issues/15697 - - k8s-version: v1.30.x - ingress: kourier-tls - test-suite: runtime - - k8s-version: v1.31.x - ingress: kourier-tls - test-suite: runtime env: KIND: 1 INGRESS_CLASS: ${{ matrix.ingress-class || matrix.ingress }}.ingress.networking.knative.dev @@ -218,13 +210,18 @@ jobs: - name: Test ${{ matrix.test-suite }} run: | + FEATURE_FLAGS="-enable-alpha -enable-beta" + if [[ "${{ matrix.ingress}}" == "kourier-tls" ]] && [[ "${{ matrix.test-suite }}" == "runtime" ]]; then + # Disabled due to flakiness: https://github.com/knative/serving/issues/15697 + FEATURE_FLAGS="$FEATURE_FLAGS -disable-optional-api" + fi gotestsum --format testname -- \ -race -count=1 -parallel=1 -tags=e2e \ -timeout=30m \ ${{ matrix.test-path }} \ -skip-cleanup-on-fail \ -disable-logstream \ - -enable-alpha -enable-beta \ + $FEATURE_FLAGS \ --ingress-class=${{ matrix.ingress-class || matrix.ingress }}.ingress.networking.knative.dev \ ${{ matrix.test-flags }}