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.

Sign up for GitHub

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

Jump to bottom

[NONEVM-984][SOAK] - Reorg Detection #969

Draft
wants to merge 106 commits into
base: develop
Choose a base branch
from
Draft

[NONEVM-984][SOAK] - Reorg Detection #969

wants to merge 106 commits into from

Conversation

Farber98
Copy link
Contributor

@Farber98 Farber98 commented Dec 10, 2024 •
edited
Loading

Latest Soak Test (Devnet)

Soak tests are being run using a local set up.

Grafana Logs

Grafana Stats Latest Test

Metric Query Link Count (last 48h)
Failed Transaction Sends Query 1
Failed Retry Transaction Sends Query 0
Failed Retry Transaction Builds Query 0
Invariant Violations in Retries Query 0
Failed Enqueue Transactions Query 0
Successfully Processed Transactions Query 1.41k
Successfully Confirmed Transactions Query 20.5k
Not Found Transactions Query 25.2k
Transaction Timeouts Query 0
Failed to Move Beyond 'Processed' Query 0
Bumped Transactions Query 72
Transactions Sent Query 20.6k
Expiration Rebroadcast Query 0
Send Errors Query 0
Rebroadcast Errors Query 0
Re-org Detected Query 0
Failed to handle Re-org Query 0

@Farber98
refactor so txm owns blockhash assignment
2d1a82d
@Farber98
lastValidBlockHeight shouldn't be exported
50dfef0
@Farber98
better comment
4e545e2
@Farber98
refactor sendWithRetry to make it clearer
4ded53c
@Farber98
confirm loop refactor
9e1be6d
@Farber98
fix infinite loop
7dd2028
@Farber98
move accountID inside msg
6c675f2
@Farber98
lint fix
b0d9426
@Farber98
base58 does not contain lower l
1b38665
@Farber98
fix hash errors
6923ddf
@Farber98
fix generate random hash
462844b
@Farber98
remove blockhash as we only need block height
fd785d0
@Farber98
expired tx changes without tests
cf958a4
@Farber98
add maybe to mocks
c5e957b
@Farber98
expiration tests
a505993
@Farber98
send txes through queue
adc8b1c
@Farber98
revert pendingtx leakage of information. overwrite blockhash
7d77f99
@Farber98
fix order of confirm loop and not found signature check
92a280b
@Farber98
fix mocks
2598e19
@Farber98
prevent confirmation loop to mark tx as errored when it needs to be r…
42b3da1 
…ebroadcasted
@Farber98
fix test
89af1f3
@Farber98
fix pointer
5e8a0da
@Farber98
add comments
75c1dcd
@Farber98
reduce rpc calls + refactors
4ff2d23
@Farber98
tests + check to save rpc calls
84e423e
@Farber98
address feedback + remove redundant impl
7d8319e
@Farber98
iface comment
68f3a3e
@Farber98
address feedback on compute unit limit and lastValidBlockHeight assig…
780179f 
…nment
@Farber98
blockhash assignment inside txm.sendWithRetry
98f0246
@Farber98
address feedback
cbf55f6
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube

Base automatically changed from nonevm-984-reorg to develop January 10, 2025 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Farber98