Skip to content

Commit

Permalink
CI: Update gerrit-verify workflow
Browse files Browse the repository at this point in the history
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 <[email protected]>
Change-Id: Iab224759e0df9730fba42fe79f8eef92e170d01e
  • Loading branch information
askb committed Oct 24, 2024
1 parent 909962b commit 7058c95
Showing 1 changed file with 23 additions and 11 deletions.
34 changes: 23 additions & 11 deletions .github/workflows/gerrit-verify.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,8 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Clear votes
uses: lfit/[email protected]
# 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 }}
Expand All @@ -60,16 +61,21 @@ 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

actionlint:
needs: prepare
runs-on: ubuntu-latest
steps:
- uses: lfit/[email protected]
- 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
Expand All @@ -84,11 +90,15 @@ jobs:
needs: prepare
runs-on: ubuntu-latest
steps:
- uses: lfit/[email protected]
- 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
Expand All @@ -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: |
Expand All @@ -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() }}
Expand All @@ -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 }}
Expand All @@ -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

0 comments on commit 7058c95

Please sign in to comment.