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

Improve node availability in beacon slots #569

Closed
Neylix opened this issue Sep 14, 2022 · 1 comment · Fixed by #574
Closed

Improve node availability in beacon slots #569

Neylix opened this issue Sep 14, 2022 · 1 comment · Fixed by #574
Assignees
Labels
beacon chain Involve BeaconChain bug Something isn't working self repair Involve SelfRepair mechanism

Comments

@Neylix
Copy link
Member

Neylix commented Sep 14, 2022

Describe the problem you discovered

Actually, to inform the availability of a node in a beacon slot, nodes ping the other node and if the node respond, it means that the node is available and if the node do not respond, it means that it is not available.
The problem with this implementation is that we have an availability bit at a specific time like a snapshot, but it may not be relevant as the node can be available all the time, but do not respond to this ping for some reason.

Describe the solution you'd like

An idea to have a more precise availability for a node is to monitor the time where the node is available during a slot time.
As so we have a more accurate value to determine if the node is really available or not as if a snapshot was created each second.

At summary time, we will have all slots from all beacon nodes so we can calculate a median on all available times for a specific node. This median will give us the available time and as we know the time between 2 slot, we can determine if the node is available or not.
We can also add a weighting to slots to prioritize the last slots as they are more relevant than the first slots.

@Neylix Neylix added bug Something isn't working beacon chain Involve BeaconChain self repair Involve SelfRepair mechanism labels Sep 14, 2022
@Neylix Neylix self-assigned this Sep 14, 2022
@samuelmanzanera
Copy link
Member

Closed by #574

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beacon chain Involve BeaconChain bug Something isn't working self repair Involve SelfRepair mechanism
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants