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

[PH] Transaction Generator support generating newaccount transactions #702

Merged
merged 26 commits into from
Feb 13, 2023

Conversation

oschwaldp-oci
Copy link
Contributor

@oschwaldp-oci oschwaldp-oci commented Feb 8, 2023

Expand transaction generator to support new account trx

  • Update user data transaction specification to support multiple actions and automatically generating an account namer per transaction to substitute into defined actions using the 'ACCT_PER_TRX' key word.
  • Removed arguments for action data pieces that have been moved into the larger transaction spec in the json files.
  • Added/Updated tests to exercise the new feature.

Additional work:

  • Add way to specify auth acct when using json trx description.
  • Add support for actionAuthAcct and actionAuthPrivKey for more explicit control when specifying transaction directly.
  • Add actionAuthAcct to json transaction description
  • Add trx generator id to allow differentiation of trx generators for use in create accounts.
  • Factored out set_transaction_headers into own helper function.
  • Add ability to configure log level for cluster via cli arg.

Add support for actionAuthAcct and actionAuthPrivKey for more explicit control when specifying transaction directly.

Add actionAuthAcct to json transaction description

Add trx generator id to allow differentiation of trx generators for use in create accounts.

Factored out set_transaction_headers into own helper function.
Update user data transaction specification to support multiple actions and automatically generating an account namer per transaction to substitute into defined actions using the 'ACCT_PER_TRX' key word.

Removed arguments for action data pieces that have been moved into the larger transaction spec in the json files.

Added/Updated tests to exercise the new feature.
Default is 90 sec, don't need that long for these tests.
@ClaytonCalabrese ClaytonCalabrese self-requested a review February 9, 2023 16:31
@oschwaldp-oci oschwaldp-oci requested a review from heifner February 9, 2023 21:42
tests/trx_generator/trx_generator.hpp Show resolved Hide resolved
tests/trx_generator/trx_generator.hpp Outdated Show resolved Hide resolved
tests/trx_generator/trx_generator.cpp Outdated Show resolved Hide resolved
tests/trx_generator/trx_generator.cpp Outdated Show resolved Hide resolved
tests/trx_generator/trx_generator.cpp Outdated Show resolved Hide resolved
tests/trx_generator/trx_generator.cpp Outdated Show resolved Hide resolved
tests/trx_generator/trx_generator.cpp Outdated Show resolved Hide resolved
tests/trx_generator/trx_generator.cpp Show resolved Hide resolved
tests/trx_generator/trx_generator.hpp Show resolved Hide resolved
tests/trx_generator/trx_generator.hpp Outdated Show resolved Hide resolved
tests/trx_generator/main.cpp Outdated Show resolved Hide resolved
tests/trx_generator/trx_generator.cpp Outdated Show resolved Hide resolved
tests/trx_generator/trx_generator.cpp Outdated Show resolved Hide resolved
tests/trx_generator/trx_generator.cpp Outdated Show resolved Hide resolved
tests/trx_generator/trx_generator.cpp Outdated Show resolved Hide resolved
tests/trx_generator/trx_generator.cpp Outdated Show resolved Hide resolved
tests/trx_generator/trx_generator.cpp Outdated Show resolved Hide resolved
tests/trx_generator/trx_generator.cpp Outdated Show resolved Hide resolved
tests/trx_generator/trx_generator.cpp Outdated Show resolved Hide resolved
@oschwaldp-oci oschwaldp-oci merged commit 8a1fecd into feature_performance_harness_stage_2 Feb 13, 2023
@oschwaldp-oci oschwaldp-oci deleted the ph-gen-newaccounts branch February 13, 2023 22:02
@oschwaldp-oci oschwaldp-oci added the OCI Work exclusive to OCI team label Apr 14, 2023
@oschwaldp-oci oschwaldp-oci self-assigned this Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCI Work exclusive to OCI team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants