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] Provide *PluginArgs dataclasses to capture and validate nodeos config options #546

Merged
merged 15 commits into from
Dec 7, 2022

Conversation

oschwaldp-oci
Copy link
Contributor

Provide *PluginArgs dataclasses to capture and validate nodeos config options.

Each *PluginArgs dataclass captures the command line argument and default for each option as specified in nodeos.

New test validate_nodeos_plugin_args.py uses nodeos's --help command to parse out each plugin's available options and defaults to validate current status vs. the *PluginArgs classes. Any addition, removal, update will cause a test failure with note of what has changed. This will allow easy maintenance going forward.

Consequently, quietly defaulted nodeos arguments will now have representation in the performance harness test reports.

… options.

Each *PluginArgs dataclass captures the command line argument and default for each option as specificed in nodeos.

New test validate_nodeos_plugin_args uses nodeos's --help command to parse out each plugin's available options and defaults to validate current status vs. the *PluginArgs classes.  Any addition, removal, update will cause a test failure with note of what has changed.  This will allow easy maintenance going forward.
tests/performance_tests/ChainPluginArgs.py Outdated Show resolved Hide resolved
tests/performance_tests/validate_nodeos_plugin_args.py Outdated Show resolved Hide resolved
@oschwaldp-oci oschwaldp-oci merged commit 54b1c4f into feature_performance_harness_stage_2 Dec 7, 2022
@oschwaldp-oci oschwaldp-oci deleted the ph-nodeos-defaults branch December 7, 2022 21:53
@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.

2 participants