Skip to content

Commit

Permalink
[Backport 1.0.latest] Fix #9907: Add retry to tox to reduce flaky tes…
Browse files Browse the repository at this point in the history
…ts due to network failures (#10178)

* [Backport 1.0.latest] Fix #9907: Add retry to tox to reduce flaky tests due to network failures

* Update main.yml
  • Loading branch information
aranke authored May 21, 2024
1 parent 8389845 commit 1786f99
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 6 deletions.
16 changes: 12 additions & 4 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -68,8 +68,12 @@ jobs:
python -m pip --version
tox --version
- name: Run tox
run: tox
- name: Run integration tests
uses: nick-fields/retry@v3
with:
timeout_minutes: 30
max_attempts: 3
command: tox

unit:
name: unit test / python ${{ matrix.python-version }}
Expand Down Expand Up @@ -103,8 +107,12 @@ jobs:
python -m pip --version
tox --version
- name: Run tox
run: tox
- name: Run unit tests
uses: nick-fields/retry@v3
with:
timeout_minutes: 10
max_attempts: 3
command: tox -e unit

- name: Get current date
if: always()
Expand Down
8 changes: 6 additions & 2 deletions .github/workflows/structured-logging-schema-check.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# This Action checks makes a dbt run to sample json structured logs
# and checks that they conform to the currently documented schema.
#
#
# If this action fails it either means we have unintentionally deviated
# from our documented structured logging schema, or we need to bump the
# version of our structured logging and add new documentation to
Expand Down Expand Up @@ -61,7 +61,11 @@ jobs:
# integration tests generate a ton of logs in different files. the next step will find them all.
# we actually care if these pass, because the normal test run doesn't usually include many json log outputs
- name: Run integration tests
run: tox -e py38-postgres -- -nauto
uses: nick-fields/retry@v3
with:
timeout_minutes: 30
max_attempts: 3
command: tox -e py38-postgres -- -nauto

# apply our schema tests to every log event from the previous step
# skips any output that isn't valid json
Expand Down

0 comments on commit 1786f99

Please sign in to comment.