From dfb366667ed7e29153fbe63d0467af51ba8f7c48 Mon Sep 17 00:00:00 2001 From: Anil Belur Date: Thu, 24 Oct 2024 12:32:07 +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: I1ab4fff237875f9d9722237a552b6b3f9554f73c 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 72aa927..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.6 + - 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.6 + - 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.6 + 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: | @@ -124,7 +133,7 @@ jobs: env: PY_COLORS: "1" ANSIBLE_FORCE_COLOR: "1" - MOLECULE_DISTRO: ${{ matrix.distro }} + MOLECULE_PLATFORM_NAME: ${{ matrix.distro }} vote: if: ${{ always() }} @@ -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