From a4cb9ae624a59f0c44031ffb32343843ee3366b4 Mon Sep 17 00:00:00 2001 From: Francois Ferrand Date: Wed, 23 Oct 2024 19:41:44 +0200 Subject: [PATCH] Merge alerts workflow No reason to keep separate, and it helps avoid a github bug/limitation when creating a check: and the Issue: ZENKO-4876 --- .github/workflows/alerts.yaml | 9 +++++---- .github/workflows/end2end.yaml | 15 ++++++++++----- 2 files changed, 15 insertions(+), 9 deletions(-) diff --git a/.github/workflows/alerts.yaml b/.github/workflows/alerts.yaml index 540caa6557..73f818c6f5 100644 --- a/.github/workflows/alerts.yaml +++ b/.github/workflows/alerts.yaml @@ -1,10 +1,11 @@ name: Test alerts on: - push: - branches-ignore: - - 'development/**' - - 'q/*' + workflow_call: + secrets: + GIT_ACCESS_TOKEN: + description: 'GitHub token' + required: true jobs: run-alert-tests: diff --git a/.github/workflows/end2end.yaml b/.github/workflows/end2end.yaml index 5dcb1ddcb6..dd838f5a4d 100644 --- a/.github/workflows/end2end.yaml +++ b/.github/workflows/end2end.yaml @@ -121,6 +121,10 @@ jobs: - name: Verify monitoring dashboard versions run: bash ./.github/scripts/check_versions.sh + check-alerts: + uses: ./.github/workflows/alerts.yaml + secrets: inherit + check-workflows: runs-on: ubuntu-22.04 steps: @@ -379,7 +383,7 @@ jobs: cache-to: type=gha,mode=max,scope=end2end-ctst end2end-http: - needs: [build-kafka, build-test-image, check-dashboard-versions] + needs: [build-kafka, build-test-image] runs-on: - ubuntu - focal @@ -428,7 +432,7 @@ jobs: run: kind delete cluster end2end-pra: - needs: [build-kafka, check-dashboard-versions, lint-and-build-ctst] + needs: [build-kafka, lint-and-build-ctst] runs-on: ubuntu-22.04-16core env: GIT_ACCESS_TOKEN: ${{ secrets.GIT_ACCESS_TOKEN }} @@ -488,7 +492,7 @@ jobs: run: kind delete cluster end2end-https: - needs: [build-kafka, build-test-image, check-dashboard-versions] + needs: [build-kafka, build-test-image] runs-on: - ubuntu - focal @@ -540,7 +544,7 @@ jobs: run: kind delete cluster end2end-sharded: - needs: [build-kafka, build-test-image, check-dashboard-versions] + needs: [build-kafka, build-test-image] runs-on: - ubuntu-22.04-8core # Enable this for Ring-based tests @@ -580,7 +584,7 @@ jobs: run: kind delete cluster ctst-end2end-sharded: - needs: [build-kafka, lint-and-build-ctst, check-dashboard-versions] + needs: [build-kafka, lint-and-build-ctst] runs-on: - ubuntu-22.04-8core steps: @@ -629,6 +633,7 @@ jobs: write-final-status: runs-on: ubuntu-latest needs: + - check-alerts - check-dashboard-versions - check-workflows - build-doc