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.

Sign up for GitHub

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

Jump to bottom

Monitor connectivity to validators #964

Open
eskimor opened this issue Jul 5, 2021 · 0 comments
Open

Monitor connectivity to validators #964

eskimor opened this issue Jul 5, 2021 · 0 comments
Assignees
@eskimor
Labels
I5-enhancement An additional feature request.

Comments

@eskimor
Copy link
Member

eskimor commented Jul 5, 2021 •
edited
Loading

As of this PR we have some warning when authority lookup fails for too many peers over a longer period of time. Being connected to validators is essential for a validator, so we would like to know if there are problems.

The above mentioned PR does not go far enough unfortunately, as it only tracks whether an authority could be looked up, but does not track actual connections. For this gossip-support should keep track of PeerConnected and PeerDisconnected events as well and report warnings if there are not enough connections.

Another issue with that PR is, that the monitoring is not per session but for a couple of sessions combined. So the 2/3 threshold is not that meaningful here. Actual connection tracking should keep track of connectivity per session we are interested in and issue warnings whenever connectivity on one of those sessions goes below a threshold.
@eskimor eskimor changed the title Trace connectivity to validators Monitor connectivity to validators Jul 5, 2021
@eskimor eskimor mentioned this issue Jul 5, 2021
Merged
@eskimor eskimor mentioned this issue Sep 17, 2021
Closed
@eskimor eskimor self-assigned this Sep 17, 2021
@Sophia-Gold Sophia-Gold transferred this issue from paritytech/polkadot Aug 24, 2023
@the-right-joyce the-right-joyce added I5-enhancement An additional feature request. and removed J0-enhancement labels Aug 25, 2023
claravanstaden added a commit to Snowfork/polkadot-sdk that referenced this issue Dec 8, 2023
@claravanstaden @alistair-singh
Adds transfer token test (paritytech#964)
5ee2f71 
* adds transfer token test

* assert agent id is the source

* complete transfer token test

---------

Co-authored-by: claravanstaden <Cats 4 life!>
Co-authored-by: Alistair Singh <alistair.singh7@gmail.com>
helin6 pushed a commit to boolnetwork/polkadot-sdk that referenced this issue Feb 5, 2024
@nanocryk
v can only be 27 or 28 prefixed only with zeros (paritytech#964)
9f27e18
bkchr pushed a commit that referenced this issue Apr 10, 2024
@svyatonik @HCastano @bkchr
Serve multiple lanes with single complex relay instance (#964)
91c669e 
* serve multiple lanes with single complex relay instance

* Update relays/bin-substrate/src/cli/relay_headers_and_messages.rs

Co-authored-by: Hernando Castano <HCastano@users.noreply.github.com>

* revert default value removal

Co-authored-by: Hernando Castano <HCastano@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@eskimor eskimor

Labels
I5-enhancement An additional feature request.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@eskimor @the-right-joyce