search

helium / blockchain-core

Apache License 2.0
213 stars 85 forks source link

Always process gw reactivation list #1358

Closed jadeallenx closed 2 years ago

jadeallenx commented 2 years ago

Problem to solve: Currently, gateway "last activity" is updated only when a validator is not in consensus. But this behavior is incorrect. The reactivation list must always be processed so that gateways are properly marked as "active again" and thus may be considered for POC targeting in the future.

Solution: Always process the reactivation list, even if a node is in consensus.

Fixes #1357

andymck commented 2 years ago

@jadeallenx sorry i should have noted in the issue but the change will need to be behind a new var, otherwise sync will break due to targeting issues ( ie having an additional list of active hotspots to choose from )

jadeallenx commented 2 years ago

@jadeallenx sorry i should have noted in the issue but the change will need to be behind a new var, otherwise sync will break due to targeting issues ( ie having an additional list of active hotspots to choose from )

OK, I will update this behind a var