-
Notifications
You must be signed in to change notification settings - Fork 208
feat(ci): implement nextest and optimize Docker test builds #9435
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
46 commits
Select commit
Hold shift + click to select a range
6301c3b
ref(docker): improve cargo caching by aligning mounts with CARGO_HOME
gustavovalverde 683db84
chore: fix typo
gustavovalverde 6463d15
test: do not use cache mounts
gustavovalverde b3048dd
chore: trigger a build
gustavovalverde c53b392
imp(docker): revert to binds with `-Z checksum-freshness` Rust flag
gustavovalverde bf3605f
chore: trigger a build
gustavovalverde fec84d8
chore: reduce diff
gustavovalverde 72d967f
fix: remove extra steps
gustavovalverde 4b4ac6e
fix: reduce complexity
gustavovalverde f809abc
revert: use a better approach
gustavovalverde a5b0633
fix: create and own the cargo and cargo target folders
gustavovalverde ae5b63c
test building several features
gustavovalverde 30cabdb
use nextest and avoid rustup compilation at runtime
gustavovalverde 430579f
refactor: consolidate test features and migrate to cargo-nextest
gustavovalverde 0fd48e5
fix: use correct package management
gustavovalverde a62ac20
fix: better handle different test cases
gustavovalverde 3df79b9
refactor(ci): use nextest for triggering tests
gustavovalverde 391e580
fix: use nextest overrides
gustavovalverde fc934b0
fix: delete extra profiles
gustavovalverde 643452f
fix: use default filter instead
gustavovalverde cbf8507
refactor(testing): Use envars and nextest profiles for test control
gustavovalverde b766e68
refactor: remove compile-time feature gates from tests in favor of ru…
gustavovalverde e8cfc05
fix: build
gustavovalverde a5e10c9
feat: add indexer as a default feature
gustavovalverde 3a883f0
refactor: make lightwalletd-grpc-tests and indexer features optional
gustavovalverde 4a0755f
refactor(tests): remove unused import and use fully qualified path
gustavovalverde 93311d0
refactor(docker): simplify dockerfile
gustavovalverde ed0607c
fix: formatting
gustavovalverde d7e72e0
fix: use the default cargo directories
gustavovalverde f1cd560
fix: build
gustavovalverde e88ddf0
fix: cargo ownership
gustavovalverde 6da21f1
fix: use correct test filtering with nextest
gustavovalverde 74a2e3e
fix: avoid cargo cache invalidation
gustavovalverde 15a07ef
fix(ci): Correct Docker test execution and platform filtering
gustavovalverde ca325dc
fix: improve caching
gustavovalverde b844371
fix: segregate sync-past-mandatory test
gustavovalverde 06efc95
fix: show output for specific test
gustavovalverde d99090e
revert: most sync and lwd changes
gustavovalverde dd9aeaf
fix: remove indexer
gustavovalverde 45cf591
chore: reduce diff
gustavovalverde 59fd617
fix: nextest timeouts
gustavovalverde a364067
Apply suggestions from code review
gustavovalverde c90c674
fix: linting errors
gustavovalverde d06e0f1
chore: update documentation and add missing profiles
gustavovalverde 0e84e1e
chore: more documentation fixes
gustavovalverde 7463aa4
Merge branch 'main' into imp-caching
gustavovalverde 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,109 @@ | ||
| # Nextest configuration for Zebra | ||
| # This file centralizes all test execution configuration. | ||
|
|
||
| [profile.default] | ||
| fail-fast = true | ||
| status-level = "pass" | ||
|
|
||
| # --- All Tests profile --- | ||
| # Runs all tests except 'check_no_git_dependencies' | ||
| [profile.all-tests] | ||
| default-filter = 'not test(check_no_git_dependencies)' | ||
|
|
||
| # --- Individual Test Profiles --- | ||
|
|
||
| [profile.check-no-git-dependencies] | ||
| default-filter = 'test(check_no_git_dependencies)' | ||
|
|
||
| [profile.state-fake-activation-heights] | ||
| slow-timeout = { period = "10m", terminate-after = 1 } | ||
| default-filter = 'package(zebra-state) and test(~with_fake_activation_heights)' | ||
|
|
||
| [profile.sync-large-checkpoints-empty] | ||
| slow-timeout = { period = "60m", terminate-after = 2 } | ||
| default-filter = 'package(zebrad) and test(=sync_large_checkpoints_empty)' | ||
|
|
||
| [profile.sync-full-mainnet] | ||
| slow-timeout = { period = "30000m", terminate-after = 1 } | ||
| success-output = "immediate" | ||
| default-filter = 'package(zebrad) and test(=sync_full_mainnet)' | ||
|
|
||
| [profile.sync-full-testnet] | ||
| slow-timeout = { period = "1500m", terminate-after = 1 } | ||
| success-output = "immediate" | ||
| default-filter = 'package(zebrad) and test(=sync_full_testnet)' | ||
|
|
||
| [profile.sync-to-mandatory-checkpoint] | ||
| slow-timeout = { period = "1500m", terminate-after = 1 } | ||
| success-output = "immediate" | ||
| default-filter = 'package(zebrad) and test(~sync_to_mandatory_checkpoint_)' | ||
|
|
||
| [profile.sync-update-mainnet] | ||
| slow-timeout = { period = "30m", terminate-after = 2 } | ||
| success-output = "immediate" | ||
| default-filter = 'package(zebrad) and test(=sync_update_mainnet)' | ||
|
|
||
| [profile.sync-past-mandatory-checkpoint-mainnet] | ||
| slow-timeout = { period = "60m", terminate-after = 2 } | ||
| default-filter = 'package(zebrad) and test(=sync_past_mandatory_checkpoint_mainnet)' | ||
|
|
||
| [profile.sync-past-mandatory-checkpoint-testnet] | ||
| slow-timeout = { period = "60m", terminate-after = 2 } | ||
| default-filter = 'package(zebrad) and test(=sync_past_mandatory_checkpoint_testnet)' | ||
|
|
||
| [profile.generate-checkpoints-mainnet] | ||
| slow-timeout = { period = "90m", terminate-after = 1 } | ||
| success-output = "immediate" | ||
| default-filter = 'package(zebrad) and test(=generate_checkpoints_mainnet)' | ||
|
|
||
| [profile.generate-checkpoints-testnet] | ||
| slow-timeout = { period = "90m", terminate-after = 1 } | ||
| success-output = "immediate" | ||
| default-filter = 'package(zebrad) and test(=generate_checkpoints_testnet)' | ||
|
|
||
| [profile.lwd-rpc-test] | ||
| test-threads = 1 | ||
| slow-timeout = { period = "60m", terminate-after = 2 } | ||
| default-filter = 'package(zebrad) and test(=lwd_rpc_test)' | ||
|
|
||
| [profile.lwd-integration] | ||
| slow-timeout = { period = "30m", terminate-after = 2 } | ||
| default-filter = 'package(zebrad) and test(=lwd_integration)' | ||
|
|
||
| [profile.lwd-sync-full] | ||
| slow-timeout = { period = "30000m", terminate-after = 1 } | ||
| success-output = "immediate" | ||
| default-filter = 'package(zebrad) and test(=lwd_sync_full)' | ||
|
|
||
| [profile.lwd-sync-update] | ||
| slow-timeout = { period = "30m", terminate-after = 2 } | ||
| success-output = "immediate" | ||
| default-filter = 'package(zebrad) and test(=lwd_sync_update)' | ||
|
|
||
| [profile.lwd-grpc-wallet] | ||
| slow-timeout = { period = "60m", terminate-after = 2 } | ||
| default-filter = 'package(zebrad) and test(=lwd_grpc_wallet)' | ||
|
|
||
| [profile.lwd-rpc-send-tx] | ||
| slow-timeout = { period = "30m", terminate-after = 2 } | ||
| default-filter = 'package(zebrad) and test(=lwd_rpc_send_tx)' | ||
|
|
||
| [profile.rpc-get-block-template] | ||
| slow-timeout = { period = "30m", terminate-after = 2 } | ||
| default-filter = 'package(zebrad) and test(=rpc_get_block_template)' | ||
|
|
||
| [profile.rpc-submit-block] | ||
| slow-timeout = { period = "30m", terminate-after = 2 } | ||
| default-filter = 'package(zebrad) and test(=rpc_submit_block)' | ||
|
|
||
| [profile.indexer-has-spending-transaction-ids] | ||
| slow-timeout = { period = "30m", terminate-after = 2 } | ||
| default-filter = 'package(zebrad) and test(=has_spending_transaction_ids)' | ||
|
|
||
| [profile.lightwalletd-test-suite] | ||
| slow-timeout = { period = "60m", terminate-after = 2 } | ||
| default-filter = 'package(zebrad) and test(=lightwalletd_test_suite)' | ||
|
|
||
| [profile.rpc-z-getsubtreesbyindex-snapshot] | ||
| slow-timeout = { period = "30m", terminate-after = 2 } | ||
| default-filter = 'package(zebrad) and test(=fully_synced_rpc_z_getsubtreesbyindex_snapshot_test)' |
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 |
|---|---|---|
|
|
@@ -15,6 +15,7 @@ | |
| # Now un-exclude required files and folders: | ||
| # | ||
| !.cargo | ||
| !.config | ||
| !*.toml | ||
| !*.lock | ||
| !tower-* | ||
|
|
||
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
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
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.
Check warning
Code scanning / zizmor
code injection via template expansion Warning