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

transport/manager: Investigate different state for a dial failure #238

Open
lexnv opened this issue Sep 5, 2024 · 0 comments
Open

transport/manager: Investigate different state for a dial failure #238

lexnv opened this issue Sep 5, 2024 · 0 comments

Comments

@lexnv
Copy link
Collaborator

lexnv commented Sep 5, 2024

Substrate panic triggered on debug_assert:

Thread 'tokio-runtime-worker' panicked at 'assertion failed: false', /usr/local/cargo/registry/src/index.crates.io-6f17d22bba15001f/litep2p-0.6.2/src/transport/manager/mod.rs:748

Code path:

state => {
tracing::warn!(
target: LOG_TARGET,
?peer,
?connection_id,
?state,
"invalid state for dial failure",
);
context.state = state;
debug_assert!(false);
Ok(())
}

Substrate issue: paritytech/polkadot-sdk#5595
Grafana link: https://grafana.teleport.parity.io/goto/sR83XkeSg?orgId=1

lexnv added a commit that referenced this issue Oct 28, 2024
This PR refactors the peer state. Previously the `AddressStore` and
`PeerState` where intertwined. The `AddressStore` contained an optional
`ConnectionID`, while the `PeerState` states contained a mandatory
score.

This PR separates the address store and the peer state, while moving the
logic of the peer state transitioning to a separate module. While at it,
have added documentation and testing for state machine transitions.

Changes include:
- Secondary connection from the PeerContext is merged into the
`PeerState`
- Connection Ids are removed from the Address Store
- Transport manager is refactored to use the new state machine, this
keeps a minimal code around in the manager

### Testing Done
- added extra tests to the peer state
- tested with subp2p-explorer for discovering kusama

This builds upon #250 for a
bigger refactor effort to track addresses in a healthier

Aims at improving:
- #239
- #238
- #237

---------

Signed-off-by: Alexandru Vasile <[email protected]>
Co-authored-by: Dmitry Markin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant