Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

validator: --tpu-host-addr -> --public-tpu-address #31137

Merged
merged 1 commit into from
Apr 12, 2023

Conversation

diman-io
Copy link
Contributor

Problem

--tpu-host-addr is not very clear and does not match other similar arguments

See #30452 (comment) and below

Summary of Changes

--tpu-host-addr arg has been renamed to --public-tpu-address (with an added alias)

@codecov
Copy link

codecov bot commented Apr 10, 2023

Codecov Report

Merging #31137 (2eef42c) into master (861bca8) will increase coverage by 0.0%.
The diff coverage is 50.0%.

@@           Coverage Diff           @@
##           master   #31137   +/-   ##
=======================================
  Coverage    81.5%    81.5%           
=======================================
  Files         728      728           
  Lines      205947   205947           
=======================================
+ Hits       167973   168005   +32     
+ Misses      37974    37942   -32     

@diman-io diman-io requested a review from behzadnouri April 11, 2023 00:04
@behzadnouri behzadnouri merged commit b81b7eb into solana-labs:master Apr 12, 2023
jeffwashington pushed a commit to jeffwashington/solana that referenced this pull request Apr 14, 2023
diman-io added a commit to diman-io/solana that referenced this pull request May 3, 2023
@diman-io diman-io mentioned this pull request May 3, 2023
@diman-io diman-io deleted the diman/public-tpu-address branch May 13, 2023 21:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants