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] Performance Harness Updates #1667

Merged
merged 15 commits into from
Sep 22, 2023
Merged

[PH] Performance Harness Updates #1667

merged 15 commits into from
Sep 22, 2023

Conversation

oschwaldp-oci
Copy link
Contributor

@oschwaldp-oci oschwaldp-oci commented Sep 22, 2023

Add command line config parameters for read-only-write-window-time-us and read-only-read-window-time-us.

  • When operating in testApiOpMode for testing read-only transactions, default read-only-write-window-time-us to 1000 and read-only-read-window-time-us to 165000.

Add configurable leap-target override. Allows the workflow to be run on a specific branch to get a specific version of the Performance Harness while allowing the tester to override which version of nodeos and cleos should be tested.

Fix when _acknowledged is incremented to allow completion of logging of ack data prior to signaling the acknowledgment.

Update ph_backward_compatibility.yaml workflow to use nodeos --full-version and cleos version full for logging the versions installed. This will help pinpoint which exact version in time was used for a given test run to allow better reproducibility of results. Also use full versions in performance_harness_run.yaml.

Work relates to ongoing issue: #1634

… and read-only-read-window-time-us.

Make default write window smaller and read window larger during read only tests (testApiOpMode).
Allows the workflow to be run on a specific branch to get a specific version of the Performance Harness while allowing the tester to override which version of nodeos and cleos should be tested.
@heifner heifner added the OCI Work exclusive to OCI team label Sep 22, 2023
@heifner heifner merged commit a448135 into main Sep 22, 2023
36 checks passed
@heifner heifner deleted the oschwaldp/perf-harness-updates branch September 22, 2023 18:48
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