Skip to content

Commit

Permalink
Only run ARM collection tests in main branch (apache#45068)
Browse files Browse the repository at this point in the history
The ARM collection tests are testing if all tests can be properly
collected when packages not available on ARM are removed. We currently
do not have such packages, but since this is only valid for providers
and in version branches we never touch providers, we should just
skip the test on non-main branch.
  • Loading branch information
potiuk authored Dec 19, 2024
1 parent 35b927f commit d1245cd
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 0 deletions.
1 change: 1 addition & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -496,6 +496,7 @@ jobs:
needs.build-info.outputs.full-tests-needed == 'true')
with:
test-groups: ${{ needs.build-info.outputs.test-groups }}
default-branch: ${{ needs.build-info.outputs.default-branch }}
runs-on-as-json-default: ${{ needs.build-info.outputs.runs-on-as-json-default }}
image-tag: ${{ needs.build-info.outputs.image-tag }}
core-test-types-list-as-string: ${{ needs.build-info.outputs.core-test-types-list-as-string }}
Expand Down
5 changes: 5 additions & 0 deletions .github/workflows/special-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,10 @@ on: # yamllint disable-line rule:truthy
description: "The array of labels (in json form) determining default runner used for the build."
required: true
type: string
default-branch:
description: "The default branch for the repository"
required: true
type: string
test-groups:
description: "The json representing list of test test groups to run"
required: true
Expand Down Expand Up @@ -199,6 +203,7 @@ jobs:
include-success-outputs: ${{ inputs.include-success-outputs }}
run-coverage: ${{ inputs.run-coverage }}
debug-resources: ${{ inputs.debug-resources }}
if: ${{ inputs.default-branch == 'main' }}

tests-system:
name: "System test: ${{ matrix.test-group }}"
Expand Down

0 comments on commit d1245cd

Please sign in to comment.