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

escalator drop test #28

Open
wants to merge 4 commits into
base: hyperlane
Choose a base branch
from
Open

Conversation

daniel-savu
Copy link

@daniel-savu daniel-savu commented Jan 7, 2025

drops every 3rd tx to make sure the noncemanager works correctly. Since the noncemanager resyncs its internal nonce every 10 txs, by using 3 here (which is co-prime with 10) we're guaranteed to eventually drop every nth tx in the interval of 10. Example counters when a tx is dropped:

  • 3 % 10 = 3 => 4th tx in range of 10 gets dropped
  • 6 % 10 = 6 => 7th tx in range of 10 gets dropped
  • 9 % 10 = 9 => 10th tx in range of 10 gets dropped
  • 12 % 10 = 2 => 3rd tx in range of 10 gets dropped
  • 15 % 10 = 5 => 6th tx in range of 10 gets dropped
    ...
  • 30 % 10 = 0 => 1st tx in range of 10 gets dropped
    etc

@daniel-savu daniel-savu force-pushed the dan/escalator-drop-test branch from ac6b057 to cb711c5 Compare January 7, 2025 14:40
@daniel-savu daniel-savu changed the base branch from dan/noncemanager-fix to hyperlane January 7, 2025 14:41
@daniel-savu daniel-savu force-pushed the dan/escalator-drop-test branch from a16d7d2 to ff9c2ae Compare January 7, 2025 17:34
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

Successfully merging this pull request may close these issues.

3 participants