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

tx-generator: read path of node config file from CLI #3789

Merged
merged 7 commits into from
Apr 19, 2022

Conversation

MarcFontaine
Copy link
Contributor

@MarcFontaine MarcFontaine commented Apr 12, 2022

This is a combination of several changes, that used to be separate PRs:
#3710
#3726
#3776

The only user-visible change is a new CLI option:
Usage: tx-generator json_highlevel FILEPATH [-n|--nodeConfig FILENAME]
It allows one to set/overwrite the node config file.

@MarcFontaine MarcFontaine force-pushed the mafo/tx-generator-protocol-file-path branch 2 times, most recently from 14d48ac to 46f1da5 Compare April 13, 2022 10:04
@MarcFontaine MarcFontaine self-assigned this Apr 13, 2022
@MarcFontaine MarcFontaine added the enhancement New feature or request label Apr 13, 2022
For a while the tx-generator has been generating transaction just in time,
and the `TxSendQueue` had been used with a list of dummy transactions.
This PR removes the `TxSendQueue` altogether.
* Add a small built-in tx-generator-script.
* Add cabal-level criterion benchmark.
* Add new 'selftest' command: write some txs to a outfile.
@MarcFontaine MarcFontaine force-pushed the mafo/tx-generator-protocol-file-path branch from 46f1da5 to 6753283 Compare April 19, 2022 07:48
@MarcFontaine MarcFontaine marked this pull request as ready for review April 19, 2022 08:00
@MarcFontaine
Copy link
Contributor Author

bors r+

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Apr 19, 2022

Build succeeded:

@iohk-bors iohk-bors bot merged commit 72833b0 into master Apr 19, 2022
@iohk-bors iohk-bors bot deleted the mafo/tx-generator-protocol-file-path branch April 19, 2022 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants