Skip to content

Commit

Permalink
TESTONLY: Run matrix of concurrency options.
Browse files Browse the repository at this point in the history
To test the speed-up characteristics of different tuning params, and
to determine what fits on the worker.
  • Loading branch information
rousik committed Dec 1, 2023
1 parent 3bfce3b commit 60e6ab5
Showing 1 changed file with 9 additions and 1 deletion.
10 changes: 9 additions & 1 deletion .github/workflows/pytest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -121,6 +121,11 @@ jobs:
defaults:
run:
shell: bash -l {0}
strategy:
fail-fast: false
matrix:
dagster_workers: [0, 1, 2, 4]
xbrl_workers: [1, 2, 3, 4, 8]
steps:
- uses: actions/checkout@v4
with:
Expand Down Expand Up @@ -181,8 +186,11 @@ jobs:
run: |
coverage run ${{ env.COVERAGE_OPTIONS }} \
src/pudl/ferc_to_sqlite/cli.py \
--workers ${{ env.XBRL_WORKERS }} \
--dagster-workers ${{ matrix.dagster_workers }} \
--workers ${{ matrix.xbrl_workers }} \
--clobber ${{ env.ETL_COMMANDLINE_OPTIONS }} ${{ env.ETL_CONFIG }}
- name: Fail pipeline early
run: exit 1
- name: Run pudl_etl
env:
COVERAGE_FILE: .coverage.pudl_etl
Expand Down

0 comments on commit 60e6ab5

Please sign in to comment.