Skip to content
This repository was archived by the owner on Jan 20, 2026. It is now read-only.

Self kill process if there are no available peers#58

Merged
philipsu522 merged 11 commits intomainfrom
p2p-self-kill-no-peers
Feb 4, 2023
Merged

Self kill process if there are no available peers#58
philipsu522 merged 11 commits intomainfrom
p2p-self-kill-no-peers

Conversation

@philipsu522
Copy link
Collaborator

@philipsu522 philipsu522 commented Feb 2, 2023

Describe your changes and provide context

Add a channel to notify the node class when there are no more peers. When this happens, the node exits

Testing performed to validate your change

Ran 4 node tendermint cluster where I injected failures, verified that node was shut down.

@philipsu522 philipsu522 requested a review from codchen February 3, 2023 04:30
@philipsu522 philipsu522 changed the title Have router be restarted if there are no available peers Self kill process if there are no available peers Feb 3, 2023
@philipsu522 philipsu522 force-pushed the p2p-self-kill-no-peers branch from 26bac9f to bce1991 Compare February 3, 2023 22:12
philipsu522 added a commit to sei-protocol/sei-cosmos that referenced this pull request Feb 4, 2023
## Describe your changes and provide context
Includes sei-protocol/sei-tendermint#58
## Testing performed to validate your change
@philipsu522 philipsu522 merged commit 54b0a40 into main Feb 4, 2023
codchen pushed a commit to sei-protocol/sei-cosmos that referenced this pull request Feb 12, 2023
## Describe your changes and provide context
Includes sei-protocol/sei-tendermint#58
## Testing performed to validate your change
Timwood0x10 pushed a commit to Timwood0x10/sei-tendermint that referenced this pull request Jun 7, 2023
## Describe your changes and provide context
Relevant PR:
sei-protocol/sei-chain#313

We are moving the module deposits to the end of the block and
transferring them all at once. This is to prevent the gas fee deposit
from being a single source of a bottleneck when it comes to concurrency.

## Testing performed to validate your change
Ran 3 iterations of load tests with 5 rounds, and 400 orders per block 

![image](https://user-images.githubusercontent.com/18161326/196336249-f150b940-f356-4d9b-a76f-c5cd270a979e.png)


![image](https://user-images.githubusercontent.com/18161326/196336238-3ad5f8a3-afc9-42f1-8b4d-185bad78d8bb.png)
masih pushed a commit to sei-protocol/sei-chain that referenced this pull request Sep 29, 2025
## Describe your changes and provide context
Includes sei-protocol/sei-tendermint#58
## Testing performed to validate your change
masih pushed a commit to sei-protocol/sei-chain that referenced this pull request Sep 30, 2025
## Describe your changes and provide context
Includes sei-protocol/sei-tendermint#58
## Testing performed to validate your change
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants