Skip to content

Commit

Permalink
fix: Bump validation workflows to use large machine instances (Mobili…
Browse files Browse the repository at this point in the history
…tyData#1320)

* Bump validation workflows to use large machine instances to avoid running out of heap memory when processing large feeds.

Closes MobilityData#1304.

* Try using a label instead.

* label => labels

* Try yet another runner syntax.
  • Loading branch information
bdferris-v2 authored and KClough committed Mar 1, 2023
1 parent b904afb commit 15caa08
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
4 changes: 3 additions & 1 deletion .github/workflows/acceptance_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,9 @@ jobs:
matrix: ${{ steps.set-matrix.outputs.matrix }}
get-reports:
needs: [ fetch-urls, pack-master, pack-snapshot ]
runs-on: ubuntu-latest
# We use machines with more memory to run validation, as large feeds
# can consume too much heap for default machine instances (see #1304).
runs-on: ubuntu-latest-4-cores
strategy:
matrix: ${{ fromJson(needs.fetch-urls.outputs.matrix) }}
steps:
Expand Down
4 changes: 3 additions & 1 deletion .github/workflows/end_to_end.yml
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,9 @@ jobs:
matrix: ${{ steps.set-matrix.outputs.matrix }}
run-on-data:
needs: [ fetch-urls, pack-snapshot ]
runs-on: ubuntu-latest
# We use machines with more memory to run validation, as large feeds
# can consume too much heap for default machine instances (see #1304).
runs-on: ubuntu-latest-4-cores
strategy:
matrix: ${{ fromJson(needs.fetch-urls.outputs.matrix) }}
steps:
Expand Down

0 comments on commit 15caa08

Please sign in to comment.