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

issues with hip-106 deployment #779

Open
nosmaster89 opened this issue Mar 31, 2024 · 6 comments
Open

issues with hip-106 deployment #779

nosmaster89 opened this issue Mar 31, 2024 · 6 comments
Assignees

Comments

@nosmaster89
Copy link

it has come to my attention that theres some issues still with what i think is the code for hip-106

in reference to the hotspot 112iFVTPaFnmah1idNqKDjT3Z7af3Vz6SfANbmQiwoqyutKeswAU

this hotspots witnesses are correctly being maked bad by hip 106 , but the oracles are validating all its beacons . on the 20th of march it validated 22 beacons in a 24 hour period

image

@andymck
Copy link
Contributor

andymck commented Apr 1, 2024

On initial look the beacons in question were indeed valid but they had zero witnesses. A valid beacon with zero witnesses does not count against reciprocity and thus its witnesses were declared invalid.

@nosmaster89
Copy link
Author

thats correct. but no hotspot on the network should have more than 4 valid beacons per epoch , the oracle should be invalidating any subsequent ones in the same 6 hour period .

@andymck
Copy link
Contributor

andymck commented Apr 3, 2024

This is a side effect of a change made in #759 where beacons with zero witnesses and which pass regular verifications are rendered valid but do not result in the gateway's last_beacon timestamp being updated in order to prevent a valid beacon with zero witnesses impacting reciprocity.

The verification which determines if a gateways beacons are as per the schedule also utilises the 'last_beacon' timestamp and as this does not get updated in the above scenario we end up rendering out of schedule zero witnessed beacons as valid....they will however not be rewarded ( due to having zero witnesses )

Will consider further and look at our options

@nosmaster89
Copy link
Author

thank you for the explination, i suspected as much that they are all only validated due to hip 106 and would not actually happen in a working setup

@andymck
Copy link
Contributor

andymck commented Apr 10, 2024

addressed via #784

@riobah
Copy link
Contributor

riobah commented Apr 21, 2024

#793 might be related

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

3 participants