-
Notifications
You must be signed in to change notification settings - Fork 12
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
chore: skip localnet specific tests in devnet 🤫 #3919
Conversation
PRO-793 Bouncer test that tests shutting down nodes to be re-instated for all networks?
For non-CI and non-local networks we had to disable the bouncer test that shuts down the connection to the external blockchains and then re-establishes connection. This is because networks like Sisyphos use a different, more production ready setup that means this method of shutting down containers can't be used. The question remains whether it's actually useful in these cases. Given we've seen it work in local and CI. The reason we might want to is to see its impact on a multi node network (which if one node's ETH connection goes down, should be none in a 3 node network). Original test ticket: PRO-700 |
Codecov Report
@@ Coverage Diff @@
## main #3919 +/- ##
=====================================
- Coverage 71% 71% -0%
=====================================
Files 366 366
Lines 56773 56773
Branches 56773 56773
=====================================
- Hits 40540 40522 -18
- Misses 14217 14231 +14
- Partials 2016 2020 +4 see 4 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
* origin/main: Fix: Correct Select Median Implementation (#3934) fix: independent witnessing startup (#3913) 🍒 cherry-pick: changes in release for CI and chainspec (#3933) refactor: Re-arrange Chains traits for better composability (#3912) fix: log error when we try to transfer *more* than we have fetched (#3930) chore: add checks and increase timeout (#3928) Add `bind_redeem_address` to CLI (#3908) 🍒 cherry-pick: add missing prod dockerfiles (#3926) chore: skip localnet specific tests in devnet 🤫 (#3919) fix: broadcast success should be witnessable after a rotation (#3921) # Conflicts: # state-chain/chains/src/eth/api.rs # state-chain/runtime/src/chainflip.rs
…on-integration * origin/main: Added CFE setting for logging span lifecycles (#3936) fix: only burn flip if non zero (#3932) Fix: Correct Select Median Implementation (#3934) fix: independent witnessing startup (#3913) 🍒 cherry-pick: changes in release for CI and chainspec (#3933) refactor: Re-arrange Chains traits for better composability (#3912) fix: log error when we try to transfer *more* than we have fetched (#3930) chore: add checks and increase timeout (#3928) Add `bind_redeem_address` to CLI (#3908) 🍒 cherry-pick: add missing prod dockerfiles (#3926) chore: skip localnet specific tests in devnet 🤫 (#3919) fix: broadcast success should be witnessable after a rotation (#3921) # Conflicts: # state-chain/cf-integration-tests/src/network.rs
* chore: skip localnet specific tests in devnet 🤫 * chore: flip logic * fix: condition * fix: simplify conditional
* chore: skip localnet specific tests in devnet 🤫 * chore: flip logic * fix: condition * fix: simplify conditional
No description provided.