Skip to content

Verify results via results_schema.py. and update test_tools handling#51

Merged
dvalinrh merged 2 commits into
masterfrom
validate
Feb 6, 2026
Merged

Verify results via results_schema.py. and update test_tools handling#51
dvalinrh merged 2 commits into
masterfrom
validate

Conversation

@dvalinrh

@dvalinrh dvalinrh commented Feb 4, 2026

Copy link
Copy Markdown
Contributor

Description

Adds verification of test results via pyantics, and fixes how test_tools is created.

Before/After Comparison

Before: No verification of run results
After: Verification of run results.

Clerical Stuff

This closes #50

Relates to JIRA: RPOPC-825

Test Results

Test command executed
/home/ec2-user/workloads/specjbb-wrapper-2.2/specjbb/specjbb_run --run_user ec2-user --home_parent /home --iterations 1 --tuned_setting tuned_none_sys_file_ --host_config "m5.xlarge" --sysname "m5.xlarge" --sys_type aws --use_pcp --java_version 21 --debug

csv file
Warehouses,Bops,Numb_JVMs,Start_Date,End_Date
2,115442,1,2026-02-04T20:02:46Z,2026-02-04T20:06:52Z
4,159470,1,2026-02-04T20:02:46Z,2026-02-04T20:06:52Z
6,156964,1,2026-02-04T20:02:46Z,2026-02-04T20:06:52Z
8,151070,1,2026-02-04T20:02:46Z,2026-02-04T20:06:52Z

Inducing an invalid warehouse of 0
0.Warehouses
Input should be greater than 0 [type=greater_than, input_value=0, input_type=int]
For further information visit https://errors.pydantic.dev/2.12/v/greater_than

--debug file
specjbb_out.txt

@dvalinrh dvalinrh requested a review from kdvalin February 4, 2026 20:27
Comment thread specjbb/specjbb_run Outdated
@dvalinrh dvalinrh requested a review from kdvalin February 5, 2026 11:06

@kdvalin kdvalin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dvalinrh dvalinrh merged commit 4f08a9b into master Feb 6, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

usep yantics to verify data

2 participants