Skip to content

Commit 9814a58

Browse files
committed
removing tests needed check that skips container builds for tag runs
Signed-off-by: Sebastian Schmittner <[email protected]>
1 parent 339d307 commit 9814a58

File tree

1 file changed

+1
-12
lines changed

1 file changed

+1
-12
lines changed

.github/workflows/build-all.yml

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -18,27 +18,18 @@ jobs:
1818
outputs:
1919
repo_owner: ${{ steps.repo_owner.outputs.lowercase }}
2020
os_matrix: "{\"os_version\":[\"debian10\",\"debian11\",\"ubuntu16\",\"ubuntu18\",\"ubuntu20\"]}"
21-
testsNeeded: ${{ steps.testsNeeded.outputs.testsNeeded }}
2221
steps:
2322
- name: Check out the repo
2423
uses: actions/checkout@v3
2524
- name: Lowercase repo owner
2625
id: repo_owner
2726
run: echo "lowercase=$(echo ${{ github.repository_owner }} | tr \"[:upper:]\" \"[:lower:]\")" >>$GITHUB_OUTPUT
2827
shell: bash
29-
- name: testsNeeded
30-
id: testsNeeded
31-
uses: dorny/paths-filter@v2
32-
with:
33-
filters: |
34-
testsNeeded:
35-
- '!**.md'
3628

3729
build_nodes:
3830
name: Build node images
3931
runs-on: ubuntu-latest
4032
needs: workflow_setup
41-
if: ${{ needs.workflow_setup.outputs.testsNeeded == 'true' }}
4233
strategy:
4334
matrix: ${{ fromJson(needs.workflow_setup.outputs.os_matrix) }}
4435
steps:
@@ -83,7 +74,6 @@ jobs:
8374
name: Build controller image
8475
runs-on: ubuntu-latest
8576
needs: workflow_setup
86-
if: ${{ needs.workflow_setup.outputs.testsNeeded == 'true' }}
8777
steps:
8878
- name: Check out the repo
8979
uses: actions/checkout@v3
@@ -120,7 +110,6 @@ jobs:
120110
test_node_and_controller:
121111
runs-on: ubuntu-latest
122112
needs: [workflow_setup, build_controller, build_nodes]
123-
if: ${{ needs.workflow_setup.outputs.testsNeeded == 'true' }}
124113
strategy:
125114
matrix: ${{ fromJson(needs.workflow_setup.outputs.os_matrix) }}
126115
fail-fast: false
@@ -199,7 +188,7 @@ jobs:
199188
name: statusCheck
200189
runs-on: ubuntu-latest
201190
needs: [workflow_setup, test_node_and_controller]
202-
if: ${{ needs.workflow_setup.outputs.testsNeeded == 'false' || success() }}
191+
if: ${{ success() }}
203192
steps:
204193
- run: 'echo "Just a status Check (Always true, when executed) for branch protection rules(blocks merging while test are running and if tests fail)." '
205194

0 commit comments

Comments
 (0)