From a815851ae7542e9c2659559e99a122ea456fd91f Mon Sep 17 00:00:00 2001 From: Anil Belur Date: Thu, 24 Oct 2024 12:49:14 +1000 Subject: [PATCH] CI: Update gerrit-verify workflow Centos7, Centos8, and Ubuntu1804 are all EOL and should be removed from the testing matrix. This will allow us to focus on the supported distros. Pull in the latest updates. Change-Id: Ia92426676adb2d72eede27f8cb1a342c5986a73e Signed-off-by: Anil Belur --- .github/workflows/gerrit-verify.yaml | 34 +++++++++++++++++++++++----------- 1 file changed, 23 insertions(+), 11 deletions(-) diff --git a/.github/workflows/gerrit-verify.yaml b/.github/workflows/gerrit-verify.yaml index 3eb5e59..040838b 100644 --- a/.github/workflows/gerrit-verify.yaml +++ b/.github/workflows/gerrit-verify.yaml @@ -51,7 +51,8 @@ jobs: runs-on: ubuntu-latest steps: - name: Clear votes - uses: lfit/gerrit-review-action@v0.3 + # yamllint disable-line rule:line-length + uses: lfit/gerrit-review-action@9627b9a144f2a2cad70707ddfae87c87dce60729 # v0.8 with: host: ${{ vars.LFIT_GERRIT_SERVER }} username: ${{ vars.LFIT_GERRIT_SSH_USER }} @@ -60,6 +61,7 @@ jobs: gerrit-change-number: ${{ inputs.GERRIT_CHANGE_NUMBER }} gerrit-patchset-number: ${{ inputs.GERRIT_PATCHSET_NUMBER }} vote-type: clear + comment-only: false - name: Allow replication run: sleep 10s @@ -67,9 +69,13 @@ jobs: needs: prepare runs-on: ubuntu-latest steps: - - uses: lfit/checkout-gerrit-change-action@v0.4 + - name: Gerrit Checkout + # yamllint disable-line rule:line-length + uses: lfit/checkout-gerrit-change-action@54d751e8bd167bc91f7d665dabe33fae87aaaa63 # v0.9 with: gerrit-refspec: ${{ inputs.GERRIT_REFSPEC }} + gerrit-project: ${{ inputs.GERRIT_PROJECT }} + gerrit-url: ${{ vars.GERRIT_URL }} delay: "0s" - name: Download actionlint id: get_actionlint @@ -84,13 +90,17 @@ jobs: needs: prepare runs-on: ubuntu-latest steps: - - uses: lfit/checkout-gerrit-change-action@v0.4 + - name: Gerrit Checkout + # yamllint disable-line rule:line-length + uses: lfit/checkout-gerrit-change-action@54d751e8bd167bc91f7d665dabe33fae87aaaa63 # v0.9 with: gerrit-refspec: ${{ inputs.GERRIT_REFSPEC }} + gerrit-project: ${{ inputs.GERRIT_PROJECT }} + gerrit-url: ${{ vars.GERRIT_URL }} delay: "0s" - - uses: actions/setup-python@v4 + - uses: actions/setup-python@v5 with: - python-version: "3.8" + python-version: "3.10" - name: Run static analysis and format checkers run: pipx run pre-commit run --all-files --show-diff-on-failure @@ -101,18 +111,17 @@ jobs: strategy: matrix: distro: - - centos7 - - centos8 - - ubuntu1804 - ubuntu2004 - ubuntu2204 fail-fast: false steps: - name: Gerrit Checkout # yamllint disable-line rule:line-length - uses: lfit/checkout-gerrit-change-action@70360ca2f8bee3e6a15224d8a03f8e017b1ac91f # v0.4 + uses: lfit/checkout-gerrit-change-action@54d751e8bd167bc91f7d665dabe33fae87aaaa63 # v0.9 with: gerrit-refspec: ${{ inputs.GERRIT_REFSPEC }} + gerrit-project: ${{ inputs.GERRIT_PROJECT }} + gerrit-url: ${{ vars.GERRIT_URL }} delay: "0s" - name: Install test dependencies. run: | @@ -137,9 +146,11 @@ jobs: ] runs-on: ubuntu-latest steps: - - uses: technote-space/workflow-conclusion-action@v3 + - name: Get conclusion + # yamllint disable-line rule:line-length + uses: im-open/workflow-conclusion@e4f7c4980600fbe0818173e30931d3550801b992 # v2.2.3 - name: Set vote - uses: lfit/gerrit-review-action@v0.3 + uses: lfit/gerrit-review-action@9627b9a144f2a2cad70707ddfae87c87dce60729 # v0.8 with: host: ${{ vars.LFIT_GERRIT_SERVER }} username: ${{ vars.LFIT_GERRIT_SSH_USER }} @@ -148,3 +159,4 @@ jobs: gerrit-change-number: ${{ inputs.GERRIT_CHANGE_NUMBER }} gerrit-patchset-number: ${{ inputs.GERRIT_PATCHSET_NUMBER }} vote-type: ${{ env.WORKFLOW_CONCLUSION }} + comment-only: false -- 2.16.6