workflows: Ensure force-skip-ci skips all actions

Before this change it was only applied to the static-checks, but if
we're already taking the extreme path of skipping the CI, we better
ensure we skip all the actions and not just a few of them.

Fixes: #3471

Signed-off-by: Fabiano Fidêncio <fabiano.fidencio@intel.com>
This commit is contained in:
Fabiano Fidêncio 2022-01-18 12:38:18 +01:00
parent db451f3c27
commit d87ab14fa7
6 changed files with 25 additions and 5 deletions

View File

@ -15,6 +15,7 @@ jobs:
name: WIP Check name: WIP Check
steps: steps:
- name: WIP Check - name: WIP Check
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
uses: tim-actions/wip-check@1c2a1ca6c110026b3e2297bb2ef39e1747b5a755 uses: tim-actions/wip-check@1c2a1ca6c110026b3e2297bb2ef39e1747b5a755
with: with:
labels: '["do-not-merge", "wip", "rfc"]' labels: '["do-not-merge", "wip", "rfc"]'

View File

@ -18,24 +18,26 @@ jobs:
name: Commit Message Check name: Commit Message Check
steps: steps:
- name: Get PR Commits - name: Get PR Commits
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
id: 'get-pr-commits' id: 'get-pr-commits'
uses: tim-actions/get-pr-commits@v1.0.0 uses: tim-actions/get-pr-commits@v1.0.0
with: with:
token: ${{ secrets.GITHUB_TOKEN }} token: ${{ secrets.GITHUB_TOKEN }}
- name: DCO Check - name: DCO Check
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
uses: tim-actions/dco@2fd0504dc0d27b33f542867c300c60840c6dcb20 uses: tim-actions/dco@2fd0504dc0d27b33f542867c300c60840c6dcb20
with: with:
commits: ${{ steps.get-pr-commits.outputs.commits }} commits: ${{ steps.get-pr-commits.outputs.commits }}
- name: Commit Body Missing Check - name: Commit Body Missing Check
if: ${{ success() || failure() }} if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') && ( success() || failure() ) }}
uses: tim-actions/commit-body-check@v1.0.2 uses: tim-actions/commit-body-check@v1.0.2
with: with:
commits: ${{ steps.get-pr-commits.outputs.commits }} commits: ${{ steps.get-pr-commits.outputs.commits }}
- name: Check Subject Line Length - name: Check Subject Line Length
if: ${{ success() || failure() }} if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') && ( success() || failure() ) }}
uses: tim-actions/commit-message-checker-with-regex@v0.3.1 uses: tim-actions/commit-message-checker-with-regex@v0.3.1
with: with:
commits: ${{ steps.get-pr-commits.outputs.commits }} commits: ${{ steps.get-pr-commits.outputs.commits }}
@ -44,7 +46,7 @@ jobs:
post_error: ${{ env.error_msg }} post_error: ${{ env.error_msg }}
- name: Check Body Line Length - name: Check Body Line Length
if: ${{ success() || failure() }} if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') && ( success() || failure() ) }}
uses: tim-actions/commit-message-checker-with-regex@v0.3.1 uses: tim-actions/commit-message-checker-with-regex@v0.3.1
with: with:
commits: ${{ steps.get-pr-commits.outputs.commits }} commits: ${{ steps.get-pr-commits.outputs.commits }}
@ -71,7 +73,7 @@ jobs:
post_error: ${{ env.error_msg }} post_error: ${{ env.error_msg }}
- name: Check Fixes - name: Check Fixes
if: ${{ success() || failure() }} if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') && ( success() || failure() ) }}
uses: tim-actions/commit-message-checker-with-regex@v0.3.1 uses: tim-actions/commit-message-checker-with-regex@v0.3.1
with: with:
commits: ${{ steps.get-pr-commits.outputs.commits }} commits: ${{ steps.get-pr-commits.outputs.commits }}
@ -82,7 +84,7 @@ jobs:
one_pass_all_pass: 'true' one_pass_all_pass: 'true'
- name: Check Subsystem - name: Check Subsystem
if: ${{ success() || failure() }} if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') && ( success() || failure() ) }}
uses: tim-actions/commit-message-checker-with-regex@v0.3.1 uses: tim-actions/commit-message-checker-with-regex@v0.3.1
with: with:
commits: ${{ steps.get-pr-commits.outputs.commits }} commits: ${{ steps.get-pr-commits.outputs.commits }}

View File

@ -19,11 +19,13 @@ jobs:
steps: steps:
- uses: actions/checkout@v2 - uses: actions/checkout@v2
- name: Install docker - name: Install docker
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
run: | run: |
curl -fsSL https://test.docker.com -o test-docker.sh curl -fsSL https://test.docker.com -o test-docker.sh
sh test-docker.sh sh test-docker.sh
- name: Build ${{ matrix.asset }} - name: Build ${{ matrix.asset }}
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
run: | run: |
make "${KATA_ASSET}-tarball" make "${KATA_ASSET}-tarball"
build_dir=$(readlink -f build) build_dir=$(readlink -f build)
@ -33,6 +35,7 @@ jobs:
KATA_ASSET: ${{ matrix.asset }} KATA_ASSET: ${{ matrix.asset }}
- name: store-artifact ${{ matrix.asset }} - name: store-artifact ${{ matrix.asset }}
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
uses: actions/upload-artifact@v2 uses: actions/upload-artifact@v2
with: with:
name: kata-artifacts name: kata-artifacts
@ -45,14 +48,17 @@ jobs:
steps: steps:
- uses: actions/checkout@v2 - uses: actions/checkout@v2
- name: get-artifacts - name: get-artifacts
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
uses: actions/download-artifact@v2 uses: actions/download-artifact@v2
with: with:
name: kata-artifacts name: kata-artifacts
path: build path: build
- name: merge-artifacts - name: merge-artifacts
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
run: | run: |
make merge-builds make merge-builds
- name: store-artifacts - name: store-artifacts
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
uses: actions/upload-artifact@v2 uses: actions/upload-artifact@v2
with: with:
name: kata-static-tarball name: kata-static-tarball
@ -63,6 +69,7 @@ jobs:
steps: steps:
- uses: actions/checkout@v2 - uses: actions/checkout@v2
- name: make kata-tarball - name: make kata-tarball
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
run: | run: |
make kata-tarball make kata-tarball
sudo make install-tarball sudo make install-tarball

View File

@ -16,6 +16,7 @@ jobs:
runs-on: ubuntu-latest runs-on: ubuntu-latest
steps: steps:
- name: Install hub - name: Install hub
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
run: | run: |
HUB_ARCH="amd64" HUB_ARCH="amd64"
HUB_VER=$(curl -sL "https://api.github.com/repos/github/hub/releases/latest" |\ HUB_VER=$(curl -sL "https://api.github.com/repos/github/hub/releases/latest" |\
@ -26,6 +27,7 @@ jobs:
sudo install hub /usr/local/bin sudo install hub /usr/local/bin
- name: Install hub extension script - name: Install hub extension script
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
run: | run: |
# Clone into a temporary directory to avoid overwriting # Clone into a temporary directory to avoid overwriting
# any existing github directory. # any existing github directory.
@ -35,9 +37,11 @@ jobs:
popd &>/dev/null popd &>/dev/null
- name: Checkout code to allow hub to communicate with the project - name: Checkout code to allow hub to communicate with the project
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
uses: actions/checkout@v2 uses: actions/checkout@v2
- name: Move issue to "In progress" - name: Move issue to "In progress"
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
env: env:
GITHUB_TOKEN: ${{ secrets.KATA_GITHUB_ACTIONS_TOKEN }} GITHUB_TOKEN: ${{ secrets.KATA_GITHUB_ACTIONS_TOKEN }}
run: | run: |

View File

@ -20,6 +20,7 @@ jobs:
runs-on: ubuntu-latest runs-on: ubuntu-latest
steps: steps:
- name: Install hub - name: Install hub
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
run: | run: |
HUB_ARCH="amd64" HUB_ARCH="amd64"
HUB_VER=$(curl -sL "https://api.github.com/repos/github/hub/releases/latest" |\ HUB_VER=$(curl -sL "https://api.github.com/repos/github/hub/releases/latest" |\
@ -30,6 +31,7 @@ jobs:
sudo install hub /usr/local/bin sudo install hub /usr/local/bin
- name: Checkout code to allow hub to communicate with the project - name: Checkout code to allow hub to communicate with the project
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
uses: actions/checkout@v2 uses: actions/checkout@v2
- name: Install porting checker script - name: Install porting checker script
@ -42,6 +44,7 @@ jobs:
popd &>/dev/null popd &>/dev/null
- name: Stop PR being merged unless it has a correct set of porting labels - name: Stop PR being merged unless it has a correct set of porting labels
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
env: env:
GITHUB_TOKEN: ${{ secrets.KATA_GITHUB_ACTIONS_TOKEN }} GITHUB_TOKEN: ${{ secrets.KATA_GITHUB_ACTIONS_TOKEN }}
run: | run: |

View File

@ -5,13 +5,16 @@ jobs:
runs-on: ubuntu-20.04 runs-on: ubuntu-20.04
steps: steps:
- name: Check out - name: Check out
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
uses: actions/checkout@v2 uses: actions/checkout@v2
with: with:
fetch-depth: 0 fetch-depth: 0
- name: Install Snapcraft - name: Install Snapcraft
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
uses: samuelmeuli/action-snapcraft@v1 uses: samuelmeuli/action-snapcraft@v1
- name: Build snap - name: Build snap
if: ${{ !contains(github.event.pull_request.labels.*.name, 'force-skip-ci') }}
run: | run: |
snapcraft -d snap --destructive-mode snapcraft -d snap --destructive-mode