Skip to content

Commit

Permalink
account for non-parallel-izable integration test runs
Browse files Browse the repository at this point in the history
  • Loading branch information
mikealfare committed Oct 10, 2024
1 parent 6c85db5 commit e9dd6b4
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 9 deletions.
15 changes: 10 additions & 5 deletions .github/workflows/merge-checks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,12 +10,17 @@ concurrency:
cancel-in-progress: true

jobs:
integration-tests:
integration-tests-dbt-athena:
uses: ./.github/workflows/integration-tests.yml
with:
package: ${{ matrix.package }}
branch: ${{ github.ref }}
secrets: inherit
strategy:
matrix:
package: ["dbt-athena", "dbt-athena-community"]

# integration test runs can't run in parallel for now, so run them in serial
integration-tests-dbt-athena-community:
needs: integration-tests-dbt-athena
uses: ./.github/workflows/integration-tests.yml
with:
package: "dbt-athena-community"
branch: ${{ github.ref }}
secrets: inherit
6 changes: 2 additions & 4 deletions .github/workflows/pull-request-checks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,13 +30,11 @@ jobs:
if: contains(github.event.pull_request.labels.*.name, 'ci:integration-tests')
uses: ./.github/workflows/integration-tests.yml
with:
package: ${{ matrix.package }}
# integration test runs can't run in parallel for now, so only run dbt-athena
# both will run post merge and unit tests run for both, so this is sufficient
branch: ${{ github.event.pull_request.head.ref }}
repository: ${{ github.event.pull_request.head.repo.full_name }}
secrets: inherit
strategy:
matrix:
package: ["dbt-athena", "dbt-athena-community"]

# This job does nothing and is only used for branch protection
results:
Expand Down

0 comments on commit e9dd6b4

Please sign in to comment.