From 7058c955452d92238ceb71ddbe801ac88bc6b0c6 Mon Sep 17 00:00:00 2001 From: Anil Belur Date: Thu, 24 Oct 2024 12:45:54 +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: Iab224759e0df9730fba42fe79f8eef92e170d01e --- .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