Skip to content
This repository has been archived by the owner on Mar 5, 2024. It is now read-only.

Andymck/reactivation tweaks #1361

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

andymck
Copy link
Contributor

@andymck andymck commented May 23, 2022

related sibyl pr: helium/sibyl#53

  1. Centralises a couple of util functions ,is_gw_active/3 and max_activity_age rather than having this code living in various places.
  2. Check to see if a GW is asserted when processing the reactivation lists in a HB, if not then dont reactivate
  3. Check to see if a GW has already been reactivated when processing the reactivation list in a HB, for example a GW could have been bouncing around validators and been included in the HB from multiple validators. We now check to if it is within the max activity age and if so then dont update its last activity field

TODO: Point 3 of this list may require a chain var ?

@andymck andymck force-pushed the andymck/reactivation-tweaks branch 2 times, most recently from 01fe359 to 12f5b44 Compare May 23, 2022 13:40
@andymck andymck force-pushed the andymck/reactivation-tweaks branch 3 times, most recently from 7d8f7fb to dab3734 Compare May 24, 2022 09:45
@andymck andymck force-pushed the andymck/reactivation-tweaks branch from dab3734 to cd7dde3 Compare May 24, 2022 09:47
@andymck andymck marked this pull request as ready for review May 24, 2022 09:49
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.

1 participant