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

[Bug]: Stale Hash potential issues #2183

Closed
1 task
aramikm opened this issue Oct 11, 2024 · 0 comments · Fixed by #2184
Closed
1 task

[Bug]: Stale Hash potential issues #2183

aramikm opened this issue Oct 11, 2024 · 0 comments · Fixed by #2184
Assignees
Labels
bug Something isn't working

Comments

@aramikm
Copy link
Collaborator

aramikm commented Oct 11, 2024

Summary

It appears that the stale hash extension might be causing some issues with graph updates

Context

One of our providers has an issue with the graph updates. The exact reason is not currently known but it is highly likely the result of the new stale hash check extension added to ensure non valid graph transactions are going to get ignored early in the pool.

Acceptence Criterea

  • Revert the changes related to stale hash check

Relevant log output

[Nest] 35  - 10/11/2024, 7:31:18 PM   DEBUG [NonceService] nextNonce 2990897
2024-10-11 19:31:19        RPC-CORE: submitAndWatchExtrinsic(extrinsic: Extrinsic): ExtrinsicStatus:: 1014: Priority is too low: (757 vs 757): The transaction has too low priority to replace another transaction already in the pool.
[Nest] 35  - 10/11/2024, 7:31:19 PM   ERROR [ReconnectionGraphService] Error processing batch: RpcError: 1014: Priority is too low: (757 vs 757): The transaction has too low priority to replace another transaction already in the pool.
2024-10-11 19:31:19        RPC-CORE: submitAndWatchExtrinsic(extrinsic: Extrinsic): ExtrinsicStatus:: 1014: Priority is too low: (757 vs 757): The transaction has too low priority to replace another transaction already in the pool.
[Nest] 35  - 10/11/2024, 7:31:19 PM   ERROR [ReconnectionGraphService] Error updating graph for user 147584, provider 2: RpcError: 1014: Priority is too low: (757 vs 757): The transaction has too low priority to replace another transaction already in the pool.
    at checkError (/app/node_modules/@polkadot/rpc-provider/cjs/coder/index.js:23:15)
@aramikm aramikm added the bug Something isn't working label Oct 11, 2024
@aramikm aramikm self-assigned this Oct 11, 2024
@aramikm aramikm changed the title [Bug]: It appears that the stale hash extension might be causing some issues with graph updates [Bug]: Stale Hash potential issues Oct 11, 2024
aramikm added a commit that referenced this issue Oct 11, 2024
# Goal
The goal of this PR is <!-- insert goal here -->

Closes #2183 

Reverting #2137 


# Checklist
- [x] e2e Tests updated
- [X] Benchmarks updated
- [X] Spec version incremented?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant