ci: actionlint: Address issues and set as required

Address issues just introduced and set actionlint as a required by removing
the path filter.

Signed-off-by: Aurélien Bombo <abombo@microsoft.com>
This commit is contained in:
Aurélien Bombo
2025-10-03 11:18:42 -05:00
parent b3a551d438
commit 07645cf58b
7 changed files with 6 additions and 13 deletions

View File

@@ -3,17 +3,9 @@ name: Lint GHA workflows
on: on:
workflow_dispatch: workflow_dispatch:
pull_request: pull_request:
types:
- opened
- edited
- reopened
- synchronize
paths:
- '.github/workflows/**'
permissions: {} permissions: {}
concurrency: concurrency:
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }} group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
cancel-in-progress: true cancel-in-progress: true

View File

@@ -124,7 +124,7 @@ jobs:
echo "GITHUB_RUNNER_CI_NON_VIRT=true" >> "$GITHUB_ENV" echo "GITHUB_RUNNER_CI_NON_VIRT=true" >> "$GITHUB_ENV"
- name: Running `${{ matrix.command }}` for ${{ matrix.component.name }} - name: Running `${{ matrix.command }}` for ${{ matrix.component.name }}
run: | run: |
cd ${COMPONENT_PATH} cd "${COMPONENT_PATH}"
${COMMAND} ${COMMAND}
env: env:
COMMAND: ${{ matrix.command }} COMMAND: ${{ matrix.command }}

View File

@@ -128,7 +128,7 @@ jobs:
- name: Running `${{ matrix.command }}` for ${{ matrix.component.name }} - name: Running `${{ matrix.command }}` for ${{ matrix.component.name }}
run: | run: |
cd "${COMPONENT_PATH}" cd "${COMPONENT_PATH}"
eval ${COMMAND} eval "${COMMAND}"
env: env:
COMMAND: ${{ matrix.command }} COMMAND: ${{ matrix.command }}
COMPONENT_PATH: ${{ matrix.component.path }} COMPONENT_PATH: ${{ matrix.component.path }}

View File

@@ -40,7 +40,7 @@ jobs:
- name: Build runtime binaries - name: Build runtime binaries
run: | run: |
cd src/runtime cd src/runtime
make ${MAKE_TARGET} make "${MAKE_TARGET}"
env: env:
MAKE_TARGET: ${{ matrix.make_target }} MAKE_TARGET: ${{ matrix.make_target }}
SKIP_GO_VERSION_CHECK: "1" SKIP_GO_VERSION_CHECK: "1"

View File

@@ -108,7 +108,7 @@ jobs:
- name: Configure the ${{ matrix.snapshotter }} snapshotter - name: Configure the ${{ matrix.snapshotter }} snapshotter
env: env:
DEPLOY_CMD: ${{ matrix.deploy-cmd }} DEPLOY_CMD: ${{ matrix.deploy-cmd }}
run: bash tests/integration/kubernetes/gha-run.sh ${DEPLOY_CMD} run: bash tests/integration/kubernetes/gha-run.sh "${DEPLOY_CMD}"
if: ${{ matrix.snapshotter != 'overlayfs' }} if: ${{ matrix.snapshotter != 'overlayfs' }}
- name: Deploy Kata - name: Deploy Kata

View File

@@ -91,7 +91,7 @@ jobs:
run: | run: |
export PATH="$PATH:${HOME}/.cargo/bin" export PATH="$PATH:${HOME}/.cargo/bin"
cd "${COMPONENT_PATH}" cd "${COMPONENT_PATH}"
eval ${COMMAND} eval "${COMMAND}"
env: env:
COMMAND: ${{ matrix.command }} COMMAND: ${{ matrix.command }}
COMPONENT_PATH: ${{ matrix.component-path }} COMPONENT_PATH: ${{ matrix.component-path }}

View File

@@ -7,6 +7,7 @@ required_tests:
# TODO: cargo-deny-runner.yaml not yet treated as conditional # TODO: cargo-deny-runner.yaml not yet treated as conditional
- Cargo Crates Check Runner / cargo-deny-runner - Cargo Crates Check Runner / cargo-deny-runner
- GHA security analysis / zizmor - GHA security analysis / zizmor
- Lint GHA workflows / run-actionlint
required_regexps: required_regexps:
# Always required regexps # Always required regexps