-
Notifications
You must be signed in to change notification settings - Fork 387
tests: add initial patchbay tests #3986
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.
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
Merged
Merged
Changes from all commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
d38b3a7
tests: add patchbay netsim tests
Frando 722f174
tests: improve patchbay test error reporting
Frando 141cce5
improve patchbay test error reporting
Frando c5d5928
fixup timeouts
Frando ff384df
chore: bump patchbay
Frando 42c6001
update patchbay, enable parallelism, export metrics
Frando 26f4b83
split out degradation tests into individual tests
Frando e9b2dcb
fix: improve CI workflow
Frando ef8a3f4
fix: use nextest profiles instead of cfg directive
Frando 1849bbf
address review: remove obsolete helper fns
Frando 44a7ac5
fixup after rebase
Frando ea4aa51
chore: fmt
Frando bfd933f
remove netreport test draft (was not used yet), will be added back later
Frando 4778d37
docs fixes
Frando 400ecf2
fixup workflow
Frando 17cf5df
fix: ignore patchbay tests in cross ci
Frando 9153977
fix: reduce concurrency
Frando 6be3c7f
chore: update to patchbay 0.2.0
Frando 3b8a11a
refactor: remove macro for degrade ladder
Frando d8c8fc4
refactor: use enum for server/client side
Frando 9f7f09c
fix: use patchbay with fix for bidi link conditioning
Frando b8f2082
ignore extreme and absurd degradations for now
Frando 82f9d7a
fix: make test less flaky
Frando 8c243d8
deps: bump patchbay to 0.3
Frando 044ed4a
chore: remove patchbay publishing for now
Frando 7dbb85b
refactor: use server/client fns for pair constructions
Frando 3ebee11
refactor: remove some patchbay tests
Frando 207fa3f
refactor: improve and harden patchbay tests
Frando 4405898
fix: incongruency btw log and test
Frando a29eb7f
improve docs
Frando 91cfc40
fixup workflow
Frando 14ebbfe
fix: use makefile in ci
Frando 83a8d2a
fixup nits
Frando 09c545f
chore: change nextest version
Frando 8d91df1
fix: final cleanup, better docs, better errors
Frando File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,56 @@ | ||
| name: Patchbay Tests | ||
|
|
||
| on: | ||
| pull_request: | ||
| push: | ||
| branches: | ||
| - main | ||
|
|
||
| concurrency: | ||
| group: patchbay-${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }} | ||
| cancel-in-progress: true | ||
|
|
||
| env: | ||
| RUST_BACKTRACE: 1 | ||
| SCCACHE_CACHE_SIZE: "10G" | ||
| IROH_FORCE_STAGING_RELAYS: "1" | ||
| NEXTEST_VERSION: "0.9.80" | ||
|
|
||
| jobs: | ||
| patchbay_tests: | ||
| name: Patchbay Tests | ||
| timeout-minutes: 15 | ||
| runs-on: [self-hosted, linux, X64] | ||
| env: | ||
| RUSTC_WRAPPER: "sccache" | ||
| steps: | ||
| - name: Enable unprivileged user namespaces | ||
| run: sudo sysctl -w kernel.apparmor_restrict_unprivileged_userns=0 | ||
| continue-on-error: true | ||
|
|
||
| - uses: actions/checkout@v6 | ||
| - uses: dtolnay/rust-toolchain@stable | ||
| - uses: mozilla-actions/sccache-action@v0.0.9 | ||
| - name: Install cargo-make and cargo-nextest | ||
| uses: taiki-e/install-action@v2 | ||
| with: | ||
| tool: nextest@${{ env.NEXTEST_VERSION }},cargo-make | ||
|
|
||
| - name: Build patchbay tests | ||
| run: cargo make patchbay --no-run | ||
|
|
||
| - name: Run patchbay tests | ||
| id: tests | ||
| run: cargo make patchbay | ||
| env: | ||
| RUST_LOG: ${{ runner.debug && 'trace' || 'debug' }} | ||
| PATCHBAY_LOG: trace | ||
|
|
||
| - name: Upload test results | ||
| if: always() | ||
| uses: actions/upload-artifact@v4 | ||
| with: | ||
| name: patchbay-testdir-${{ github.sha }} | ||
| path: target/testdir-current/ | ||
| retention-days: 7 | ||
| if-no-files-found: ignore |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,4 @@ | ||
| /target | ||
| /logs | ||
| iroh.config.toml | ||
| /.patchbay | ||
flub marked this conversation as resolved.
Show resolved
Hide resolved
|
||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.