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

Fixes for the deep scenarios #54

Merged
merged 10 commits into from
Apr 24, 2024
Merged

Fixes for the deep scenarios #54

merged 10 commits into from
Apr 24, 2024

Conversation

daria305
Copy link
Contributor

  • Spam now is working for deep scenarios correctly.
  • For non-conflicting scenario like peace there are no errors.
  • For conflicting scenarios like guava we see that some portion of blocks is not posted due to error but that is expected behavior.
    e.g 1532/11356 rejected block for guava
2024-04-23T18:16:20+02:00       INFO    EvilTools.Spammer       Blocks issued so far: 11356, faucet outputs left: 5361, errors encountered: 1532 

Issue: Deeper scenarios were not working properly, e.g. guava was eating up 5 fresh faucet outputs but shuld use only 1 and reuse 4 other created in the same batch.

After fixing this bug (tempID was calculated on the older output struct, outputID was set after tx was built) the deeper spams were no longer working.
Node returns:

2024-04-23T16:09:57+02:00       ERROR   EvilTools.accounts      failed to post block: internal server error: url  
http://localhost:8050/api/core/v3/blocks, error message: Internal Server Error, error: code=500, message=Internal  
Server Error: failed to attach block: error issuing model block: context deadline exceeded whilst waiting for ev  
ent on block BlockID(0x6adec2e818a2693c140361a78cc5f0c830b24f044e5b23239072f1e01c69b21015000000:21)

this message is unclear, but the problem is that blocks were dropped because we tried to spend not yet accepted outputs.
Now we added, await for acceptance of the transaction after each batch is sent, this leads to many more go rutines spammed and awating but has solved the problem.

@jkrvivian jkrvivian merged commit aa7fff0 into develop Apr 24, 2024
4 checks passed
@jkrvivian jkrvivian deleted the fix/fruit-spam branch April 24, 2024 06:45
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.

2 participants