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.

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

EPIC: Investigate and fix existing testnet network issues #1340

Closed
4 of 5 tasks
evan-forbes opened this issue Feb 6, 2023 · 3 comments
Closed
4 of 5 tasks

EPIC: Investigate and fix existing testnet network issues #1340

evan-forbes opened this issue Feb 6, 2023 · 3 comments
Assignees
Labels
epic item groups other items for easier tracking

Comments

@evan-forbes
Copy link
Member

evan-forbes commented Feb 6, 2023

We're currently observing some oddities around the stability of the mocha testnetwork. While we are producing blocks consistently, we are doing so at variable intervals (anywhere between 8-30s), block producers are producing back to back blocks very frequently, and it is common that peers in the network on different rounds. We need to investigate and fix these issues as soon as possible, and it will likely require further developments for our testing infrastructure #1256 and testnet data collection. It could also be the case that all of the observed issues are caused by the same underlying problem, and that thing problem is blocking us from coming to consensus on the first round.

Note: it's also possible that tendermint's floodsub is simply incapable of consistently producing 8MB blocks in the first round of consensus, which might mean that we are not able to have consistent block times until we improve the gossiping protocol with celestiaorg/celestia-core#884 and celestiaorg/celestia-core#883

@evan-forbes evan-forbes added the epic item groups other items for easier tracking label Feb 6, 2023
@evan-forbes evan-forbes added this to the Mainnet milestone Feb 6, 2023
@musalbas
Copy link
Member

musalbas commented Feb 6, 2023

simply incapable of consistently producing 8MB blocks

aren't the vast majority of blocks in mocha much smaller than 8mb though?

@evan-forbes
Copy link
Member Author

aren't the vast majority of blocks in mocha much smaller than 8mb though?

yes, that is correct. I do not believe that this is the reason why we are seeing inconsistent block times, that still requires investigation, and likely changing the timeout commit to be dynamic. I just wanted to note that even after we investigate and implement things like dynamic timeout commits, it is still a possibility that consistently coming to consensus within the first round 95+% of the time for 8MB blocks might not be possible using the existing implementation in a normal network setting.

@evan-forbes
Copy link
Member Author

removing the mainnet milestone as 2MB and 8MB blocks function fine. We still want to fix block prop, but I think that can be separated from this epic

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
epic item groups other items for easier tracking
Projects
Development

No branches or pull requests

4 participants