-
Notifications
You must be signed in to change notification settings - Fork 548
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
PR #16186, split for re-review #16502
base: georgeee/redo-16186-for-better-review-base
Are you sure you want to change the base?
Changes from all commits
37fde44
33ce3c3
71bd217
e053563
788ab19
37e3584
f070475
ffbfe45
b2eb3df
34d7a98
d5a56e3
7860cdf
3062393
eaba812
09bd9bd
2249d75
10df606
39b7c74
eefcc50
bb8ebe1
c1d7fbd
b47cf86
cb395b2
b98f42f
60b5ab3
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -154,11 +154,10 @@ let there_and_back_again ~num_txn_per_acct ~txns_per_block ~slot_time ~fill_rate | |
~origin_sender_secret_key_path | ||
~(origin_sender_secret_key_pw_option : string option) | ||
~returner_secret_key_path ~(returner_secret_key_pw_option : string option) | ||
~graphql_target_node_option ~minimum_user_command_fee () = | ||
~graphql_target_node_option ~minimum_user_command_fee ~logger () = | ||
let open Deferred.Let_syntax in | ||
(* define the rate limiting function *) | ||
let open Logger in | ||
let logger = Logger.create () in | ||
let limit_level = | ||
let slot_limit = | ||
Float.( | ||
|
@@ -310,8 +309,6 @@ let there_and_back_again ~num_txn_per_acct ~txns_per_block ~slot_time ~fill_rate | |
return () | ||
|
||
let output_there_and_back_cmds = | ||
let genesis_constants = Genesis_constants.Compiled.genesis_constants in | ||
let compile_config = Mina_compile_config.Compiled.t in | ||
let open Command.Let_syntax in | ||
Command.async | ||
~summary: | ||
|
@@ -390,23 +387,32 @@ let output_there_and_back_cmds = | |
transactions, if this is not present then we use the env var \ | ||
MINA_PRIVKEY_PASS" | ||
(optional string) | ||
and config_file = Cli_lib.Flag.config_files | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We're calling a list of config files here There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is a typo, there are a few more places like this one, fixed in 6d26821 |
||
and graphql_target_node_option = | ||
flag "--graphql-target-node" ~aliases:[ "graphql-target-node" ] | ||
~doc: | ||
"URL The graphql node to send graphl commands to. must be in \ | ||
format `<ip>:<port>`. default is `127.0.0.1:3085`" | ||
(optional string) | ||
and minimum_user_command_fee = | ||
let default = compile_config.default_transaction_fee in | ||
Cli_lib.Flag.fee_common | ||
~minimum_user_command_fee:genesis_constants.minimum_user_command_fee | ||
~default_transaction_fee:default | ||
in | ||
there_and_back_again ~num_txn_per_acct ~txns_per_block ~txn_fee_option | ||
~slot_time ~fill_rate ~rate_limit ~rate_limit_level ~rate_limit_interval | ||
~origin_sender_secret_key_path ~origin_sender_secret_key_pw_option | ||
~returner_secret_key_path ~returner_secret_key_pw_option | ||
~graphql_target_node_option ~minimum_user_command_fee ) | ||
and minimum_user_command_fee_opt = Cli_lib.Flag.fee_common in | ||
fun () -> | ||
let open Deferred.Let_syntax in | ||
let logger = Logger.create () in | ||
let%bind minimum_user_command_fee = | ||
let%map conf = | ||
Runtime_config.Constants.load_constants ~logger config_file | ||
in | ||
Option.value | ||
~default: | ||
(Runtime_config.Constants.genesis_constants conf) | ||
.minimum_user_command_fee minimum_user_command_fee_opt | ||
in | ||
there_and_back_again ~num_txn_per_acct ~txns_per_block ~txn_fee_option | ||
~slot_time ~fill_rate ~rate_limit ~rate_limit_level | ||
~rate_limit_interval ~origin_sender_secret_key_path | ||
~origin_sender_secret_key_pw_option ~returner_secret_key_path | ||
~returner_secret_key_pw_option ~graphql_target_node_option | ||
~minimum_user_command_fee ~logger () ) | ||
|
||
let () = | ||
Command.run | ||
|
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.
Clarification: commit f070475 (Remove default_transaction_fee from node_config) also adds
--config-file
to thebatch_txn_tool
app