From 7df8bbdda628f79ee1ccb5e10ecae74962a1d670 Mon Sep 17 00:00:00 2001 From: Anil Belur Date: Sun, 25 Aug 2024 11:53:53 +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. Signed-off-by: Anil Belur Change-Id: Ifbe9c566efd4da73d9e274da0c5ecba13fd380f3 --- .github/workflows/gerrit-verify.yaml | 32 +++++++++++++++++++--------- 1 file changed, 22 insertions(+), 10 deletions(-) diff --git a/.github/workflows/gerrit-verify.yaml b/.github/workflows/gerrit-verify.yaml index 48bc58f..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.9 + - 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,11 +90,15 @@ jobs: needs: prepare runs-on: ubuntu-latest steps: - - uses: lfit/checkout-gerrit-change-action@v0.9 + - 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.10" - name: Run static analysis and format checkers @@ -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@v0.9 + 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.8 + 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