Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace overall safety task in hotshot integration tests #2586

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Replace overall safety task in hotshot integration tests #2586

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
d4a6dae
init
ss-es Feb 12, 2025
0bd81c6
clippy
ss-es Feb 12, 2025
bcc013b
Merge branch 'main' into ss/replace-overall-safety-task
ss-es Feb 12, 2025
930168a
update lockfile
ss-es Feb 12, 2025
ec3ef3c
update cargo
ss-es Feb 12, 2025
bbc29b5
sort
ss-es Feb 13, 2025
a00f02a
disable fail fast
ss-es Feb 13, 2025
a1eceef
fix?
ss-es Feb 13, 2025
731c937
fix
ss-es Feb 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions .config/nextest.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
slow-timeout = { period = "2m", terminate-after = 3 }
default-filter = 'not (test(slow_) | package(tests))'
retries = 2
threads-required = "num-test-threads"

# More flaky tests
[[profile.default.overrides]]
Expand All @@ -18,7 +19,7 @@ retries = 10
filter = """
test(tests_4::test_with_builder_failures::test_with_builder_failures)
"""
threads-required = 2
# threads-required = 2

[[profile.default.overrides]]
# These tests are fast if they work (usually about 150ms) but sometimes they
Expand All @@ -34,7 +35,7 @@ retries = 10
# The restart tests run an entire sequencing network, and so are quite resource intensive.
[[profile.default.overrides]]
filter = 'test(slow_test_restart)'
threads-required = 'num-cpus'
# threads-required = 'num-cpus'

[profile.slow]
slow-timeout = "2m"
Expand Down
3 changes: 3 additions & 0 deletions .github/workflows/test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -133,6 +133,7 @@ jobs:
strategy:
matrix:
partition: [1, 2, 3, 4, 5, 6, 7, 8, 9, 10]
fail-fast: false
steps:
- name: Install Foundry
uses: foundry-rs/foundry-toolchain@v1
Expand All @@ -159,6 +160,7 @@ jobs:
strategy:
matrix:
partition: [1, 2, 3, 4, 5, 6, 7, 8, 9, 10]
fail-fast: false
steps:
- name: Fix submodule permissions check
run: |
Expand Down Expand Up @@ -198,6 +200,7 @@ jobs:

- version: 99
compose: "-f process-compose.yaml -f process-compose-mp.yml -D"
fail-fast: false
runs-on: ubuntu-latest
steps:
- name: Install Nix
Expand Down
Loading
Loading