-
Notifications
You must be signed in to change notification settings - Fork 251
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
v1.17: rpc-sts: add config options for stake-weighted qos (backport of #197) #340
Conversation
Cherry-pick of f41fb84 has failed:
To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
cc/ @sakridge for stable bp approval |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Conflicts need fixing
* rpc-sts: plumb options for swqos config * rpc-sts: send to specific tpu peers when configured (cherry picked from commit f41fb84) # Conflicts: # send-transaction-service/src/send_transaction_service.rs # validator/src/cli.rs # validator/src/main.rs
1724d8c
to
7e63edd
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## v1.17 #340 +/- ##
=======================================
Coverage 81.6% 81.6%
=======================================
Files 806 806
Lines 219237 219251 +14
=======================================
+ Hits 178998 179025 +27
+ Misses 40239 40226 -13 |
Problem
no simple way to configure an rpc node to send transactions to staked peer(s), as per stake-weighted qos
Summary of Changes
--rpc-send-transaction-tpu-peer HOST:PORT
to specify a specific tpu peer to send transctions to, instead of the current leader. this argument may be passed multiple times.--rpc-send-transaction-also-leader
to continue sending to the current leader even though tpu peers have been specified with--rpc-send-transaction-tpu-peer HOST:PORT
This is an automatic backport of pull request #197 done by [Mergify](https://mergify.com).